Variable zuweisen in ob Bedingungsanweisung, gute Praxis oder nicht? [geschlossen]

Ich bin vor einem Jahr von klassischen OO-Sprachen wie Java zu JavaScript gewechselt. Der folgende Code wird in Java definitiv nicht empfohlen (oder sogar nicht korrekt):

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;
}

Grundsätzlich habe ich gerade herausgefunden, dass ich einem Wert in einer if-Bedingungsanweisung eine Variable zuweisen und den zugewiesenen Wert sofort überprüfen kann, als ob er boolesch wäre.

Für eine sicherere Wette trenne ich das normalerweise in zwei Codezeilen, weise es zuerst zu und überprüfe dann die Variable. Jetzt, wo ich das gefunden habe, frage ich mich nur, ob es in den Augen erfahrener JavaScript-Entwickler eine gute Praxis ist oder nicht.

Ich würde es nicht empfehlen. Das Problem ist, dass es wie ein häufiger Fehler aussieht, bei dem Sie versuchen, Werte zu vergleichen, aber einen einzelnen =anstelle von ==oder verwenden ===. Zum Beispiel, wenn Sie dies sehen:

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

Sie wissen nicht, ob sie das wollten oder ob sie das schreiben wollten:

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

Wenn Sie die Aufgabe wirklich an Ort und Stelle ausführen möchten, würde ich auch einen expliziten Vergleich empfehlen:

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

Ich sehe keinen Beweis dafür, dass es keine gute Praxis ist. Ja, es mag wie ein Fehler aussehen, aber das kann leicht durch vernünftige Kommentare behoben werden. Nimm zum Beispiel:

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

Warum sollte diese Funktion ein zweites Mal ausgeführt werden dürfen mit:

alert(processorIntensiveFunction());

Weil die erste Version schlecht aussieht? Ich kann dieser Logik nicht zustimmen.

Ich habe es oft gemacht. Um die JavaScript-Warnung zu umgehen, füge ich zwei Parens hinzu:

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

Sie sollten es vermeiden, wenn Sie es wirklich verwenden möchten, schreiben Sie einen Kommentar darüber und sagen Sie, was Sie tun.

Sie können dies auch in Java tun. Und nein, es ist keine gute Praxis. :) :)

(Und verwenden Sie das ===in Javascript für typisierte Gleichheit. Lesen Sie Crockfords The Good Parts-Buch über JS.)

Es gibt einen Fall, in dem Sie dies tun, mit while-loops.
Wenn Sie Dateien lesen, gehen Sie normalerweise folgendermaßen vor:

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
        }
    }
}

Schauen Sie sich die whileSchleife in Zeile 9 an. Dort wird eine neue Zeile gelesen und in einer Variablen gespeichert, und dann wird der Inhalt der Schleife ausgeführt. Ich weiß, dass dies keine ifAussage ist, aber ich denke, eine while-Schleife kann auch in Ihre Frage aufgenommen werden.

Der Grund dafür ist, dass bei Verwendung von a FileInputStreambei jedem Aufruf FileInputStream.readLine()die nächste Zeile in der Datei gelesen wird. Wenn Sie sie also aus der Schleife aufgerufen hätten, fileIn.readLine() != nullohne die Variable zuzuweisen, anstatt sie aufzurufen (currentLine = fileIn.readLine()) != null, und dann von Auch innerhalb der Schleife würden Sie nur jede zweite Zeile erhalten.

Hoffe du verstehst und viel Glück!

Sie können Zuweisungen auch innerhalb von if-Anweisungen in Java ausführen. Ein gutes Beispiel wäre, etwas einzulesen und aufzuschreiben:

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

Der Code:

// 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();
}

Wenn Sie sich auf Martin Fowlers Buch Refactoring beziehen, das das Design des vorhandenen Codes verbessert ! Dann gibt es mehrere Fälle, in denen es eine gute Praxis wäre, z. lange komplexe Bedingungen, um einen Funktions- oder Methodenaufruf zu verwenden, um Ihren Fall zu bestätigen:

"Motivation

Einer der häufigsten Komplexitätsbereiche in einem Programm liegt in der komplexen bedingten Logik. Wenn Sie Code schreiben, um Bedingungen zu testen und abhängig von verschiedenen Bedingungen verschiedene Dinge zu tun, erhalten Sie schnell eine ziemlich lange Methode. Die Länge einer Methode ist an sich ein Faktor, der das Lesen erschwert, aber die Bedingungen erhöhen die Schwierigkeit. Das Problem liegt normalerweise in der Tatsache, dass der Code sowohl in den Bedingungsprüfungen als auch in den Aktionen Ihnen sagt, was passiert, aber leicht verdecken kann, warum es passiert.

Wie bei jedem großen Codeblock können Sie Ihre Absicht klarer machen, indem Sie sie zerlegen und Codestücke durch einen Methodenaufruf ersetzen, der nach der Absicht dieses Codeblocks benannt ist. > Mit Bedingungen können Sie weitere Vorteile erhalten, indem Sie dies für den bedingten Teil und jede der Alternativen tun. Auf diese Weise markieren Sie die Bedingung und machen deutlich, worauf Sie verzweigen. Sie markieren auch den Grund für die Verzweigung. "

Und ja, seine Antwort gilt auch für Java-Implementierungen. In den Beispielen wird die bedingte Funktion jedoch keiner Variablen zugewiesen.

Es ist keine gute Praxis. Sie werden bald verwirrt sein. Es sieht einem häufigen Fehler ähnlich: Missbrauch der Operatoren "=" und "==".

Sie sollten es in 2 Codezeilen aufteilen. Dies hilft nicht nur, den Code klarer zu gestalten, sondern lässt sich auch in Zukunft leicht umgestalten. Stellen Sie sich vor, Sie ändern die IF-Bedingung? Sie können die Zeile versehentlich entfernen und Ihre Variable erhält nicht mehr den ihr zugewiesenen Wert.

Ich würde dies eher als C-Stil der alten Schule betrachten. Es ist keine wirklich gute Praxis in JavaScript, daher sollten Sie es vermeiden.

Sie könnten so etwas tun:

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

Ich bin aus Golang hierher gekommen, wo es üblich ist, so etwas zu sehen

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

In dem errist nur auf diesen ifBlock beschränkt. Als solches mache ich Folgendes in es6, was ziemlich hässlich erscheint, aber meine ziemlich strengen Eslint-Regeln nicht durcheinander bringt und dasselbe erreicht.

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

Die zusätzlichen Klammern definieren einen neuen, "lexikalischen Bereich"? Das heißt, ich kann verwenden constund errsteht dem äußeren Block nicht zur Verfügung.