Christian Kruse: Unübersichtlich und „aufgeblähter" Code?

Beitrag lesen

Hallo Gunnar,

ist auch schrecklich, weil man derlei gut auf einer Zeile unterbringen kann.

> if(empty($_POST['hersteller'])) { $errors[] = "Geben Sie bitte einen Hersteller an"; }
> if(empty($_POST['url']))        { $errors[] = "Geben Sie bitte eine Hersteller URL an"; }

Wozu dann bitte die Klammern?

Das weglassen von Klammern ist potentiell immer gefährlich, Bugs wie goto fail zeigen das sehr deutlich. Bitte nicht, auch nicht wenn man den if-Body auf der gleichen Zeile hat.

LG,
CK

0 69

Unübersichtlich und „aufgeblähter" Code?

Jnnbo
  • php
  1. 0
    frankx
  2. 0
    dedlfix
    1. 0
      Auge
      • meinung
      • php
      1. 0
        dedlfix
  3. 0
    Jörg Reinholz
    1. 0
      frankx
    2. 0
      Jörg Reinholz
      • programmiertechnik
      1. 0
        Jnnbo
        1. 0
          Jörg Reinholz
          1. 0
            Gunnar Bittersmann
            1. 0
              dedlfix
            2. 0
              Jörg Reinholz
      2. 0
        frankx
        1. 0
          Jörg Reinholz
        2. 0
          Auge
          1. 0
            Gunnar Bittersmann
      3. 1
        dedlfix
      4. 0
        Gunnar Bittersmann
        1. 2
          Christian Kruse
    3. 1
      Auge
      • meinung
      • php
    4. 0
      Gunnar Bittersmann
      1. 0
        Jörg Reinholz
        1. 0

          Curly Braces und Coding Style

          frankx
          1. 1
            Der Martin
            1. 1
              Sven Rautenberg
              1. 0
                Tabellenkalk
                1. 0

                  Curly Braces und Coding Style - Leerzeichen oder neue Zeile?

                  frankx
                  1. 0
                    Tabellenkalk
                    • php
                    • programmiertechnik
                    1. 0
                      frankx
                      1. 0

                        Curly Braces und Coding Style - PHP Coding Standards Fixer

                        frankx
                    2. 1
                      Sven Rautenberg
                      1. 0
                        dedlfix
                        1. 0
                          Sven Rautenberg
                          1. 0
                            dedlfix
  4. 0
    Auge
    1. 0
      Jnnbo
      1. 1
        Christian Kruse
        1. 0
          Jnnbo
          1. 1
            Christian Kruse
        2. 0
          Gunnar Bittersmann
          • programmiertechnik
          1. 0
            Christian Kruse
      2. 0
        Jörg Reinholz
        1. 0
          Jnnbo
          1. 0
            Jörg Reinholz
            1. 0
              Jnnbo
              1. 0
                Sven Rautenberg
                1. 0
                  BaBa
                  1. 0
                    Gunnar Bittersmann
                    1. 2
                      Christian Kruse
                    2. 0
                      Sven Rautenberg
                      1. 0
                        dedlfix
                        1. 0
                          Gunnar Bittersmann
                          1. 0
                            dedlfix
                        2. 0
                          Gunnar Bittersmann
                          1. 0
                            dedlfix
                          2. 0
                            Gunnar Bittersmann
                            • menschelei
                            1. 1
                              Der Martin
                              1. 0
                                Gunnar Bittersmann
    2. 0
      frankx
  5. 0
    frankx
    1. 0
      Jnnbo
      1. 0
        dedlfix
      2. 0
        frankx
  6. 0
    MudGuard
    1. 0
      Jnnbo
      1. 0
        MudGuard
    2. 0
      dedlfix
      1. 0
        frankx