Re: More on the cosmetics (if-statements)

From:

desktop <fff@sss.com>

Newsgroups:

comp.lang.c++

Date:

Thu, 31 May 2007 15:01:44 +0200

Message-ID:

<f3mgvl$med$1@news.net.uni-c.dk>

terminator wrote:

On May 31, 3:13 pm, desktop <f...@sss.com> wrote:

Which would you prefer:

/* A */

        int k = 402;
        if ( k == 40) {
                std::cout << "40\n";
        } else {
                std::cout << "Not 40\n";
        }

/* B */
        if ( k == 40) std::cout << "40\n";
        else std::cout << "Not 40\n";

I know it depends on taste but I am just curious and would like to hear
your pro's and con's.

for the sake of maintanance and later correction/improvement 'A' is
better ;it prevents algorithmic errors due to later editing and
forggeting braces.

Ok but what about this (assuming that there is no such thing as a switch):

    int k = 40;

      /* Case 1. */
      if (k == 10) {
          std::cout << "1\n";
      } else
          /* Case 2. */
          if (k == 20) {
              std::cout << "2\n";
      } else
          /* Case 3. */
          if (k == 30) {
              std::cout << "3\n";
      } else
          /* Case 4. */
          if (k == 40) {
              std::cout << "4\n";
      }

/* INSTEAD OF:*/

      /* Case 1: Only 1 element in the list. */
      if (k == 10) {
          std::cout << "1\n";
      } else {
          if (k == 20) {
              std::cout << "2\n";
          } else {
              if (k == 30) {
                  std::cout << "3\n";
              } else {
                  if (k == 40) {
                      std::cout << "4\n";
                  }
              }
          }
      }