Ist dies die eleganteste/sicherste/empfohlene Lösung für diese Situation?

Question

Ich hätte gerne die else Anweisung für die erste if. Etwas wie folgt aus:

if (vowels_counter >= consonants_counter) 
    for (i = 0; ntca[i]; i++) 
     for (j = 0; vowels[j]; j++) 
      if (ntca[i] == vowels[j]) 
      { ntca[i] += 1; 
       break; 
      } 
else 
    for (i = 0; ntca[i]; i++)... 

Aber Sie sehen, wie die else mit dem zweiten if verbunden ist:

if (vowels_counter >= consonants_counter) 
     for (i = 0; ntca[i]; i++) 
      for (j = 0; vowels[j]; j++) 
       if (ntca[i] == vowels[j]) 
       { ntca[i] += 1; 
        break; 
       } 

       else 
        for (i = 0; ntca[i]; i++)... 

Ich löste diese Situation durch die verschachtelte if setzen und die for in Klammern wie folgt aus:

if (vowels_counter >= consonants_counter) 
     for (i = 0; ntca[i]; i++) 
      {for (j = 0; vowels[j]; j++) 
       {if (ntca[i] == vowels[j]) 
        {ntca[i] += 1; 
        break; 
        } 
       } 
      } 

    else 
     for (i = 0; ntca[i]; i++)... 

Ist dies die eleganteste/sicherste/empfohlene Lösung? Vielen Dank!

Answer 1

Gibt es eine andere Lösung für diese Situation ohne Klammern zu verwenden?

Das ist die falsche Frage. Die eigentliche Frage ist: Gibt es jemals einen Grund, KEINE Klammern zu benutzen? NEIN. Verwenden Sie einfach Klammern. Ihre Kollegen und Ihre Zukunft werden Ihnen später danken.

Answer 2

Eine Lösung besteht darin, den Zustand des inneren if zu negieren und den Prozess auf zu setzen.

if (vowels_counter >= consonants_counter) 
    for (i = 0; ntca[i]; i++) 
     for (j = 0; vowels[j]; j++) 
      if (!(ntca[i] == vowels[j])); else 
      { ntca[i] += 1; 
       break; 
      } 
else 
    for (i = 0; ntca[i]; i++)... 

Answer 3

Was Sie versuchen, ist nicht so einfach: die else bindet an die nächste ausstehende if Anweisung. Sie müssen mindestens Klammern um den Körper einer der for Aussagen legen, oder schlimmer noch, fügen Sie eine leere else; Klausel zu verschachtelten if.

Aber das ist NICHT was Sie tun sollten!

Beachten Sie, dass es schlechter Stil nicht Klammern von if, for, while kommandierte für Anweisungen verwenden do usw., die auf einer einzigen Zeile nicht beschränkt sind. Es ist fehleranfällig und nicht sehr lesbar. Verwenden Sie großzügig Klammern und halten Sie Ihren Code vor späteren Änderungen durch weniger versierte (oder weniger nüchterne) Programmierer, die es zu leicht brechen würden.

Klassische Kernighan und Ritchie Stil sieht wie folgt aus:

if (vowels_counter >= consonants_counter) { 
     for (i = 0; ntca[i]; i++) { 
      for (j = 0; vowels[j]; j++) { 
       if (ntca[i] == vowels[j]) { 
        ntca[i] += 1; 
        break; 
       } 
      } 
     } 
    } else { 
     for (i = 0; ntca[i]; i++) { 
      ... 
     } 
    } 

Answer 4

I Klammern um den Körper der if Anweisung verwenden würde, dann ist es die einfachste und lesbar Option:

if (vowels_counter >= consonants_counter) 
{ 
    for (... 
} 
else 
{ 
    for (... 
} 

aber nur zum Spaß:

if (vowels_counter >= consonants_counter) 
    for (i = 0; ntca[i]; i++) 
    { 
    for (j = 0; vowels[j] && ntca[i] != vowels[j]; j++) 
     ; 

    ntca[i] += (vowels[j] && ntca[i] == vowels[j]); 
    } 
else 
    for (i = 0; ntca[i]; i++) // ... 

Answer 5

Der naheliegende Weg ist, ein weitereshinzuzufügen, gefolgt von einer leeren Anweisung (d. H. sofort von einer ;)

if (vowels_counter >= consonants_counter) 
    for (i = 0; ntca[i]; i++) 
     for (j = 0; vowels[j]; j++) 
      if (ntca[i] == vowels[j]) 
      { ntca[i] += 1; 
       break; 
      } 
      else ;  // note presence of ; here 
else 
    for (i = 0; ntca[i]; i++) 

Doch obwohl es um die Frage, wie gefragt antwortet, dann ist dies eine Technik, nicht das ich je empfehlen würde.Es gibt so viele Möglichkeiten, dass solche Hacker Programmierer - einschließlich sich selbst in Zukunft - verwirren könnten, dass es sich einfach nicht lohnt. Code, der leicht zu missverstehen ist, ist leicht falsch zu verstehen.

in die Gewohnheit, die Leichen von if des Vergeben, for, else, etc in geschweiften Klammern, wenn Sie eine sehr spezifische und überzeugend haben (an jemanden andere) Rechtfertigung nicht. Solche Dinge werden in den meisten Styleguides aus gutem Grund dringend empfohlen. Und denken Sie daran, dass die Einrückung die Bedeutung von Code in keiner Weise ändert.