Pourquoi voudriez-vous d'utiliser une assignation dans une condition?

Dans de nombreuses langues, les affectations sont légales dans les conditions. Je n'ai jamais compris la raison derrière cela. Pourquoi voudriez-vous écrire:

if (var1 = var2) {
  ...
}

au lieu de:

var1 = var2;
if (var1) {
  ...
}
  • Cette question est vraiment indépendant de la langue? Les langues qui sont touchés par cette? Je sais que le C et le C++ sont, quoi d'autre?
InformationsquelleAutor lajos | 2008-09-30
  1. 99

    C'est plus utile pour les boucles que si des instructions.

    while( var = GetNext() )
{
  ...do something with var 
}

    Qui autrement auraient du être écrit

    var = GetNext();
while( var )
{
 ...do something
 var = GetNext();
}
    • Je suis d'accord - mais ce serait d'écrire: while ((var = GetNext()) != 0) { ... } sans réfléchir à deux fois, en partie parce que GCC se plaint si je ne suis pas avec mon défaut les options de compilation.
    • Vrai. En supposant que vous êtes en utilisant un langage qui prend en charge pour en boucle les déclarations, et vous n'êtes pas déjà en utilisant var à l'extérieur de la boucle.
    • dans Certains langages, comme le C ANSI, les règles de portée ne permet pas cette approche, @KerrekSB
    • Tu veux dire le C ANSI à partir de ce qui, de 1989? Comme quand ils avaient des moteurs à vapeur et des cartes perforées? Ce pourrait être le cas, mais est-ce pertinent?
    • Est-ce une mauvaise pratique? Mon javascript linter me donne l'avertissement: "attendue expression conditionnelle et a plutôt connu une cession" quand je ne while(item = stream.read()) { ... }
    • Cet avertissement n'est pas parce qu'il n'y a rien de mal avec une affectation dans un certain temps, mais parce que de faire une attribution dans le cas où vous vraiment le but de faire une comparaison est une commune de bug. Si vous voulez vous débarrasser de l'avertissement que vous pouvez faire la JS équivalent de ce qu' @Jonathan Leffler suggère. Personnellement, je serais probablement le faire de toute façon parce que de la façon dont beaucoup je me méfie de l'JavaScript truthy règles 🙂 Il convient également de noter dans certains langages comme C# et Java c'est le seul moyen.
    • et c'est ainsi que nous arrivons à imgur.com/a/N7Zs6
    • Je ne comprends pas la première ligne: C'est plus utile pour les boucles que si des instructions. Il n'existe pas et si les déclarations.

    InformationsquelleAutor Gerald
  2. 29

    Il est plus utile si vous êtes l'appel d'une fonction:

    if (n = foo())
{
    /* foo returned a non-zero value, do something with the return value */
} else {
    /* foo returned zero, do something else */
}

    Bien sûr, vous pouvez simplement mettre les n = foo(); sur une déclaration séparée, alors si (n), mais je pense que le ci-dessus est assez lisible idiome.

    • Je pense que vous devriez inclure dans la parenthèse ou gcc va se plaindre avec: avertissement: suggérer des parenthèses autour de cession utilisé comme valeur de vérité [-Wparentheses] Comme ceci: if ((n = foo())) { // faire quelque chose avec la valeur de retour } else { // foo retourné zéro/NULL, faire autre chose }
    InformationsquelleAutor Chris
  3. 29

    Je le trouve très utile dans les chaînes d'actions qui impliquent souvent de détection d'erreur, etc.

    if ((rc = first_check(arg1, arg2)) != 0)
{
    report error based on rc
}
else if ((rc = second_check(arg2, arg3)) != 0)
{
    report error based on new rc
}
else if ((rc = third_check(arg3, arg4)) != 0)
{
    report error based on new rc
}
else
{
    do what you really wanted to do
}

    L'alternative (et non à l'aide de la cession en l'état) est:

    rc = first_check(arg1, arg2);
if (rc != 0)
{
    report error based on rc
}
else
{
    rc = second_check(arg2, arg3);
    if (rc != 0)
    {
        report error based on new rc
    }
    else
    {
        rc = third_check(arg3, arg4);
        if (rc != 0)
        {
            report error based on new rc
        }
        else
        {
            do what you really wanted to do
        }
    }
}

    Sur une longue vérification des erreurs, l'alternative peut fonctionner sur le membre de droite de la page tandis que la mission-dans-à condition que la version ne le fait pas.

    Les contrôles d'erreur peut également être "actions"—first_action(), second_action(), third_action() — bien sûr, plutôt que de simplement les contrôles. Qui est, ils pourraient être vérifiées par étapes dans le processus que la fonction est la gestion. (Le plus souvent dans le code que je travaille avec, les fonctions sont le long des lignes de pré-contrôles des conditions, ou de l'allocation mémoire nécessaire pour la fonction de travail, ou similaire).

    • Ouais, en fait, je l'a fait pour éviter l'excès de nidification et ensuite effectué une recherche en ligne pour voir si c'était une pratique courante. En fin de compte j'ai décidé contre elle parce que je n'avais d'autre cas, mais cela semble être une raison légitime de mettre de la cession en l'état.
    • Des collègues déteste! Pour moi, au moins dans la plupart des cas, est beaucoup plus facile à gérer qu'un arbre géant, une ou plusieurs déclarations. Je préfère unique de retour d'une fonction...
    • raisonnables exemple, le compagnon de blocs de faire le patron évident, auto-explicatif. Mais bon, le bloc ("pseudo") code ressemble un peu moche, et à l'inverse de style (en plaçant ce que tu voulais vraiment faire à la fin) n'est pas grande.
    InformationsquelleAutor Jonathan Leffler
  4. 23

    Il peut être utile si vous appelez une fonction qui retourne des données pour des travaux ou un drapeau pour indiquer une erreur (ou que vous avez terminé).

    Quelque chose comme:

    while ((c = getchar()) != EOF) {
    //process the character
}

//end of file reached...

    Personnellement c'est un langage que je ne suis pas très friand, mais parfois, l'alternative est la plus laide.

    InformationsquelleAutor Michael Burr
  5. 10

    GCC peut vous aider à détecter (avec un Mur) si vous accidentellement essayez d'utiliser une affectation en tant que valeur de vérité, dans le cas où il recommande de vous écrire

    if ((n = foo())) {
   ...
}

    I. e. utiliser des parenthèses pour indiquer que c'est vraiment ce que vous voulez.

    • Bon à savoir. J'espère que vos collègues de le lire de la même manière.
    • Moi? La bonté, je n'aurais jamais essayé de l'utiliser à l'affectation d'une valeur de vérité. Je faisais allusion à ce que GCC recommande.
    InformationsquelleAutor JesperE
  6. 8

    L'idiome est plus utile lorsque vous écrivez un while boucle au lieu d'un if déclaration. Pour un if déclaration, vous pouvez le diviser comme vous le décrivez. Mais sans cette construction, il faudrait répéter:

    c = getchar();
while (c != EOF) {
    //...
    c = getchar();
}

    ou l'utilisation d'une boucle-et-un-demi-structure:

    while (true) {
    c = getchar();
    if (c == EOF) break;
    //...
}

    Je préfère habituellement la boucle-et-un-moitié formulaire.

    • Pour un while boucle, préférez l'utilisation de: do { c = getchar(); ... } while (c != EOF);
    • Et for boucles peuvent être utilisés à la place: for (char c = getchar(); c != EOF; c= getchar()) { /* do something with c */ } -- c'est la plus concise, et for toujours dit "en boucle" pour moi, du point de vue stylistique. Petit inconvénient est que vous devez spécifier la fonction de retour c deux fois.
    InformationsquelleAutor Greg Hewgill
  7. 4

    La réponse courte est que Expression orientée langages de programmation permettent plus succincte code. Ils ne vous force pas à séparer les commandes à partir de requêtes.

    • C'est une lutte constante pour faire respecter le codage sans affectations en si(), alors(), et d'autres déclarations similaires. C/C++ sont bien connus pour les effets secondaires d'un tel qui se révèlent souvent être des bugs, surtout quand on ajoute le ++ et -- des opérateurs.
    • Le premier lien est inutile: C ou C++ ne semblent pas être une telle langue.
    InformationsquelleAutor Hugh Allen
  8. 3

    En PHP, par exemple, c'est utile pour faire une boucle à travers la base de données SQL résultats:

    while ($row = mysql_fetch_assoc($result)) {
    //Display row
}

    C'est beaucoup mieux que d':

    $row = mysql_fetch_assoc($result);
while ($row) {
    //Display row
    $row = mysql_fetch_assoc($result);
}
    • Cela a aussi le pur effet secondaire de $row ne vivent pas au-delà de cette boucle et sont donc éligibles pour la collecte des ordures tôt (dans les langues qui n'GC de toute façon).
    InformationsquelleAutor Jeremy Ruten
  9. 2

    L'autre avantage vient lors de l'utilisation de gdb.
    Dans le code suivant le code d'erreur n'est pas connu si nous étions à une seule étape.

    while (checkstatus() != -1) {
    //process
}

    Plutôt

    while (true) {
    int error = checkstatus();
    if (error != -1)
        //process
    else
        //fail
}

    Maintenant, lors de l'étape que l'on peut savoir ce qu'était le code d'erreur retourné à partir de la checkstatus().

    • Tu veux dire qu'il peut être utile pour les changements temporaires lors du débogage de code?
    InformationsquelleAutor plasmixs
  10. -2

    La raison en est :

    1. L'amélioration de la Performance (Parfois)

    2. Moindre Code (Toujours)

    Prendre un exemple: Il y a une méthode someMethod() et dans un if la condition que vous voulez vérifier si la valeur de retour de la méthode est null. Si vous allez utiliser la valeur de retour de nouveau. 

    If(null != someMethod()){
    String s = someMethod();
    ......
    //Use s
}

    Il va entraver la performance depuis que vous appelez la même méthode à deux reprises. Au lieu d'utilisation:

    String s;
If(null != (s = someMethod())) {
    ......
    //Use s
}
    InformationsquelleAutor Gangadhar