Javaコードスタイルの質問 - forループ

Question

次のコードははっきりと読みやすいですか？

public void createDatabase() throws SQLException, IOException { 
    SQLiteDatabase database = dbStore.getDatabase(); 
    LineNumberReader scriptInputReader = new LineNumberReader(new InputStreamReader(getClass().getResourceAsStream(SCRIPT_CREATE))); 
    for(String line; (line = scriptInputReader.readLine()) != null;) { 
     database.execSQL(line); 
    } 
} 

私は上記のような "for"ループをたくさん書きます。私にとっては、ループ内で使用される一時変数（ "行"）を示し、ループの終了時（ "readLine"が "null"を返す）にスコープを制限します。他のプログラマは、これらの...

またはこのいずれかのために私を嫌いになるのだろうか：

SQLiteDatabase database = dbStore.getDatabase(); 
    Cursor cursor = database.query("PINS", new String [] {"ID", "X", "Y"}, null, null, null, null, "ID"); 
    if(cursor.moveToFirst()) { 
     for(; !cursor.isAfterLast(); cursor.moveToNext()) { 
      (...) 
     } 
    } 
    cursor.close(); 

は、上記だけで「きちんとした」またはすでにJavaにパズルのようなものはありますか？

Answer 1

私はあなたがやったように、しかし、私は1つの小さな変更になるだろう：

for(String line = scriptInputReader.readLine(); line != null; line = scriptInputReader.readLine()) { 
    database.execSQL(line); 
} 

これは、ループの終了条件から反復アクションを分離します。また、 "while"バージョンとは異なり、line変数のスコープをループ狭化スコープに制限するのは、できるだけコーディングを適切に行うことです。

また、コードスタイルチェッカーは、通常、テスト内にネストされた割り当てを「不良スタイル」と見なします。明確にするために、コードは次のように少しです：

for (int i = -1; ++i < max;) { // don't do this: increment action inside condition section 
    // some code 
} 

Answer 2

私は私が選ぶだろうwhileループ

String line = scriptInputReader.readLine(); 
while(line != null){ 
    //do stuff 
    line = scriptInputReader.readLine(); 
} 

Answer 3

を使用します。これは、明確かつ簡単です

String line = null; 
while((line = scriptInputReader.readLine()) != null) { 
    ... do stuff with line 
} 

。

Answer 4

私は気楽にもっと気分を味わえます。

ループが何をするのか分かりやすいので最初のものはそれほど悪くはありませんが、ループの途中にロジックを追加して操作が複雑になると、 「ちょっと、ファイルを読んでほしいと思ったら、しばらく使っていたと思うので、ちょっとしたトリックが必要だ」と思うだろう。

2番目のコード（ループ内のコードはありません）は恐ろしいものですが、それほど遠くない将来に誰かが言うでしょう：「ねえ、ここにループがあり、内容が削除されました。彼らはループを削除することを忘れてしまった！私はそれを完全に除去することによってそれを最適化することができます '）。

Answer 5

ここには2つの考え方があります。 Jarek Potiukが示すように、/ whileループを異なる目的で使用することを好むもの、つまり、ループの範囲を事前に知っているときはforループを使用しなければなりません（for（; i < arr.length（）; i ++））。

しかし、もう1つの種類のループを使用する考え方があります。これは、より汎用性が高いためです。例えば、Java SDKは無制限の状況（例：リンクリスト）でかなり頻繁にforループを使用します。 しかし、あなたは本当にボヘミアンのようにforループを書くべきです - はるかに明確です。

Answer 6

ループの終了時期がわからないときは、whileループを使用する方が好きです。例えば、ファイルを読み込む。

Jarek Potiukのソリューションが動作する

は、しかし、私はこのようなものを好む：

String line = scriptInputReader.readLine(); 
while(line != null) 
{ 
    ... do stuff with line 
    line = scriptInputReader.readLine(); 
} 

をそれは少しより多くのコードですが、私はボヘミアンに同意する必要があります：スタイルチェッカーが通常考える

コード テスト内にネストされた割り当てを 「貧弱なスタイル」