if条件ステートメントで変数を割り当てますか、良い習慣かどうか？[閉まっている]

私は1年前に、Javaなどの古典的なOO言語からJavaScriptに移行しました。次のコードは、Javaでは絶対に推奨されません（または正しくありません）。

if(dayNumber = getClickedDayNumber(dayInfo))
{
    alert("day number found : " + dayNumber);
}
function getClickedDayNumber(dayInfo)
{
    dayNumber = dayInfo.indexOf("fc-day");
    if(dayNumber != -1) //substring found
    {
        //normally any calendar month consists of "40" days, so this will definitely pick up its day number.
        return parseInt(dayInfo.substring(dayNumber+6, dayNumber+8));
    }
    else return false;
}

基本的に、if条件ステートメントの値に変数を割り当てて、割り当てられた値をブール値であるかのようにすぐにチェックできることがわかりました。

より安全な賭けのために、私は通常それを2行のコードに分けて、最初に割り当ててから変数をチェックしますが、これを見つけたので、経験豊富なJavaScript開発者の目には良い習慣かどうか疑問に思っていますか？

私はそれをお勧めしません。問題は、値を比較しようとする一般的なエラーのように見えますが=、==またはの代わりに単一を使用することです===。たとえば、次のような場合：

if (value = someFunction()) {
    ...
}

それが彼らがやろうとしていたことなのか、あるいは彼らがこれを書くつもりだったのかはわかりません。

if (value == someFunction()) {
    ...
}

割り当てを適切に行う必要がある場合は、明示的な比較も行うことをお勧めします。

if ((value = someFunction()) === <whatever truthy value you are expecting>) {
    ...
}

それが良い習慣ではないという証拠はありません。はい、それは間違いのように見えるかもしれませんが、それは賢明なコメントによって簡単に修正されます。例えば：

if (x = processorIntensiveFunction()) { // declaration inside if intended
    alert(x);
}

なぜその関数が2回目の実行を許可されるのか：

alert(processorIntensiveFunction());

最初のバージョンが悪いので？その論理には同意できません。

何度もやった。JavaScriptの警告をバイパスするには、2つの括弧を追加します。

if ((result = get_something())) { }

使用を避けたい場合は、その上にコメントを書き込んでください。

Javaでもこれを行うことができます。いいえ、それは良い習慣ではありません。:)

（そして===、型付きの等価性のためにJavaScriptでinを使用します。JSに関するCrockfordのThe Good Partsの本を読んでください。）

while-loopsを使用する場合、1つのケースがあります。
ファイルを読み取るとき、通常は次のようにします。

void readFile(String pathToFile) {
    // Create a FileInputStream object
    FileInputStream fileIn = null;
    try {
        // Create the FileInputStream
        fileIn = new FileInputStream(pathToFile);
        // Create a variable to store the current line's text in
        String currentLine;
        // While the file has lines left, read the next line,
        // store it in the variable and do whatever is in the loop
        while((currentLine = in.readLine()) != null) {
            // Print out the current line in the console
            // (you can do whatever you want with the line. this is just an example)
            System.out.println(currentLine);
        }
    } catch(IOException e) {
        // Handle exception
    } finally {
        try {
            // Close the FileInputStream
            fileIn.close();
        } catch(IOException e) {
            // Handle exception
        }
    }
}

while9行目の-loopを見てください。そこで、新しい行が読み取られて変数に格納され、次にループの内容が実行されます。これはifステートメントではないことはわかっていますが、whileループも質問に含めることができると思います。

これは、を使用するとFileInputStream、を呼び出すたびFileInputStream.readLine()に、ファイル内の次の行を読み取るため、をfileIn.readLine() != null呼び出す代わりに、変数を割り当てずにループから呼び出した(currentLine = fileIn.readLine()) != null場合、ループの内側でも、2行おきにしか取得できません。

ご理解のほどよろしくお願いいたします。

Javaのifステートメント内でも割り当てを行うことができます。良い例は何かを読み書きすることです：

http://www.exampledepot.com/egs/java.io/CopyFile.html?l=new

コード：

// Copies src file to dst file.
// If the dst file does not exist, it is created
void copy(File src, File dst) throws IOException 
{
    InputStream in = new FileInputStream(src);
    OutputStream out = new FileOutputStream(dst);

    // Transfer bytes from in to out
    byte[] buf = new byte[1024];
    int len;
    while ((len = in.read(buf)) > 0) {
        out.write(buf, 0, len);
    }
    in.close();
    out.close();
}

Martin Fowlersの本「既存のコードのデザインを改善するリファクタリング」を参照するとしたら！次に、それが良い習慣になるいくつかのケースがあります。関数またはメソッドの呼び出しを使用してケースをアサートする長い複雑な条件：

"動機

プログラムの複雑さの最も一般的な領域の1つは、複雑な条件付きロジックにあります。条件をテストし、さまざまな条件に応じてさまざまなことを行うコードを書くと、すぐにかなり長い方法になってしまいます。メソッドの長さはそれ自体が読みにくくする要因ですが、条件によってはさらに難しくなります。問題は通常、条件チェックとアクションの両方のコードが何が起こっているかを通知しますが、なぜそれが起こるのかを簡単に覆い隠すことができるという事実にあります。

大きなコードブロックと同様に、コードを分解し、コードのチャンクをそのコードブロックの意図に基づいて命名されたメソッド呼び出しに置き換えることで、意図をより明確にすることができます。>条件を使用すると、条件部分と代替のそれぞれに対してこれを実行することで、さらにメリットを得ることができます。このようにして、状態を強調表示し、分岐しているものを明確にします。分岐の理由も強調します。」

そして、はい、彼の答えはJava実装にも有効です。例では、変数に条件関数を割り当てません。

それは良い習慣ではありません。あなたはすぐにそれについて混乱するでしょう。これは一般的なエラーに似ています。「=」および「==」演算子の誤用です。

2行のコードに分割する必要があります。コードをより明確にするだけでなく、将来のリファクタリングも容易になります。IF条件を変更することを想像してみてください。誤って行を削除すると、変数に値が割り当てられなくなります。

私はこれを昔ながらのCスタイルと考えています。JavaScriptでは実際には良い方法ではないため、回避する必要があります。

あなたはそのようなことをすることができます：

if (value = /* sic */ some_function()){
  use_value(value)
}

私はgolangから来ました。

if (err := doSomething(); err != nil) {
    return nil, err
}

これではerrそれにスコープされるifブロックのみ。そのため、これはes6で実行していることです。これはかなり見苦しいように見えますが、かなり厳密なeslintルールを変更せず、同じことを実現します。

{
  const err = doSomething()
  if (err != null) {
    return (null, err)
  }
}

余分な中括弧は、新しい、ええと、「字句スコープ」を定義しますか？これは私が使用できることを意味しconst、そしてerr外側のブロックには使用できません。