2016-09-15 6 views
-3

いくつかのフィードバックが必要でしたが、バグを修正し、下のロジック、if、else-ifのロジックに問題があるかどうか尋ねました。それが正しいかどうか。新しいコードベースを見て、このロジックが正しいことを確認したかったのです。私がコメントしたif文とelse文は、わかりません。これらのステートメントはどちらも当てはまるでしょうか?私に知らせてください、または他の問題があれば教えてください。おかげプログラミングロジック

try { 
    if (data.length == 0 && _routeOnEmptyBytearray) 
     _receivedBytearrayZero = true; 
    } 

else if (data.length >= 0) { 
     if (_outStr == null){ 
      _receivedFileContent = true; 
      fileCreate(); 
     } 
     _outStr.write(data); 
    } 
} 
+0

以下のようにtryブロックの後にキャッチを追加するには、ロジックが正しいかではないと言うことはできません方法。しかし、データがnullの場合、コードでNullPointerExceptionがスローされ、filCreateのコードで初期化されていない場合は_outstrがnullであることがわかります。 –

+1

この方法の要件を知らなければ、どのようにして誰がその論理が正しいかどうかをどのように語ることができますか?これは、「サイズ10の靴をいくつか買った、これは私にとって正しいサイズですか?私の足を見たことのない人に。 –

答えて

0

ifとelse文は、オープンとクローズかっこ を持っているか、

0

上記のコードはコンパイルされませんを持っていないでなければならないの両方、_receivedBytearrayZero = true;は試してブロックを閉じた後ではありません「}」ので、次にcatchが続きます。 elseステートメントには届きません。私たちは、あなたがそれで何をすべきかわからないので

0

かの最初の条件の後にブレースを追加し、

try { 
    if (data.length == 0 && _routeOnEmptyBytearray) { 
     _receivedBytearrayZero = true; 
    } else 
     if (data.length >= 0) { 
      if (_outStr == null) {  
       _receivedFileContent = true; 
       fileCreate(); 
      } 
      _outStr.write(data); 
     } 
} catch(Exception e) { 
} 
関連する問題