1. PHP / Говнокод #1868

    +145.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if($record->getTitle()) {
  $request->setAttribute('title', $record->getTitle());
}else{
  $request->setAttribute('title', $record->getName());
}

    определение существования метода у объекта $record

    Запостил: seroy, 18 Сентября 2009

    Комментарии (13) RSS

    • ava guest 18.09.2009 17:12 # +1

      Ознакомьтесь сначала с ООП в PHP. Таким способом не определить существование метода у объекта. Если метода нет, то вывалится фатал.
      Ответить

    • ava guest 18.09.2009 17:15 # +3

      Чо за бред? Просто у бъекта record может либо быть свойство title - и тогда нужно использовать его, либо не быть - и тогда использовать св-во name.
      Аффтар поста, учи матчасть
      Ответить

      • ava Shock 19.09.2009 19:16 # +1

        $title = $record->getTitle() ? $record->getTitle() : $record->getName();
        $request->setAttribute(\'title\', $title);
        Ответить

        • ava guest 19.09.2009 19:53 # 0

          Думаю автор просто не знал про конструкцию \"? :\"
          Имхо, не такой уж и говнокод.
          Ответить

          • ava WGH 19.09.2009 22:00 # 0

            Не знаю, мне кажется, использовать if-then-else даже нагляднее.
            Ответить

          • ava Shock 20.09.2009 13:48 # 0

            ну на крайняк так, чтобы избежать копипасты.
            но не критично — согласен. думаю, достаточно часто в рабочих проектах такое встречается.
            if($record->getTitle()) {
            $title = $record->getTitle();
            }else{
            $title = $record->getName();
            }
            $request->setAttribute('title', $title);
            Ответить

            • ava Shock 20.09.2009 13:49 # +1

              хе. или даже так:
              $title = $record->getTitle();
              if(!$title) {
              $title = $record->getName();
              }
              $request->setAttribute('title', $title);
              единственный вариант, при котором метод getTitle не вызывается дважды)
              Ответить

              • ava TWINc 20.09.2009 20:09 # +2

                можно еще так
                if (!($title = $record->getTitle()) {
                $title = $record->getName();
                }
                ...
                Ответить

                • ava Shock 26.09.2009 02:17 # 0

                  одной скобки не хватает, но да, можно)
                  Ответить

    • ava Bartelby 20.09.2009 10:47 # 0

      А где говнокод?
      Ответить

    • ava 0m3r 22.09.2009 17:50 # 0

      в контексте __get и __set не говнокод
      Ответить

    Добавить комментарий