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

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    private function _fileExists($file)
{
    if(file_exists(self::FILE_PATH . $file)) {
        return true;
    }
    return false;
}

    Нашел в одном из проектов.

    Запостил: cartman, 28 Июня 2011

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

    • ava sonic 28.06.2011 20:40 # 0

      откройте мне глаза. в чем говно?
      Ответить

    • ava cartman 28.06.2011 20:43 # +2

      4 строки вместо одной
      return file_exists(self::FILE_PATH . $file));
      Ответить

      • ava yasenhui 28.06.2011 21:37 # +1

        Пиздец, кошмар-то какой, 4 строки вместо одной, вы только представьте!!.
        Иди в перл, однострочники писать.

        Разница в нагрузке практически нулевая. Читабельность чуть лучше. Как итог, один хуй.

        Может человек расширять собирался? А может торопился и не слазил в справку чтобы убедится, что функция всегда возвращает boolean (а не как некоторые в похапе)?
        Нет блять, великий cartman узрит гавно и запостит. Предложи еще все обращения к функции в классе заменить на file_exists(self::FILE_PATH . $file)), Целых семь строчек сэкономишь, охуеть как круто!
        Ответить

        • ava guest 28.06.2011 21:42 # 0

          Не вру у меня капча гетовая выпала 9999!!!(могу скрин выложить, но всем пох)
          Загадаю желние, посмотрим исполнится или нет

          Что касается картманов тут 90-95% такого
          Ответить

        • ava yasenhui 28.06.2011 22:56 # 0

          >Целых семь строчек сэкономишь, охуеть как круто!

          Обсчитался, учитывая переносы строк между функциями - все восемь.

          Кажется, у меня БАЙТКОД от таких мега-постов на говнокоде.
          Ответить

        • ava TarasB 29.06.2011 09:30 # 0

          > Читабельность чуть лучше.

          Нет, надо так:

          if(file_exists(self::FILE_PATH . $file)) {

    return true;

}
else
{

    return false;

}


          Так намного читабельнее!
          Ответить

          • ava bugmenot 29.06.2011 10:02 # −3

            if(file_exists(self::FILE_PATH . $file)==true) {
            Fixed
            Ответить

          • ava nislova 29.06.2011 10:45 # −3

            if ( file_exists ( self::FILE_PATH . $file ) === TRUE )
            {
            return ( TRUE );
            }
            else
            {
            return ( FALSE );
            }

            Во как надо ))
            Ответить

            • ava Vasiliy 29.06.2011 11:05 # +5

              Ну мой вариант
              switch (file_exists ( self::FILE_PATH . $file ))
              {
              case true: return true;
              case false: return false;
              default : return false;
              }
              Ответить

              • ava 3.14159265 29.06.2011 11:39 # 0

                неплохо. но нет предела совершенству.
                switch (true)
                {
                case file_exists ( self::FILE_PATH . $file ): return true;
                }
                switch (false)
                {
                case file_exists ( self::FILE_PATH . $file ): return false;
                }
                Ответить

                • ava Int 29.06.2011 17:30 # −1

                  Гуру!

                  if (file_exists ( self::FILE_PATH . $file )) $flag=1;
                  elseif (!file_exists ( self::FILE_PATH . $file )) $flag=0;
                  switch ($flag)
                  {
                  case 1: return true;
                  case 0: return false;
                  }

                  Ну да, я не так просветлён :(
                  Ответить

              • ava TarasB 29.06.2011 12:09 # −1

                Вообще-то на default надо кидать исключение.
                Ответить

                • ava guest 29.06.2011 12:42 # 0

                  Надеюсь не в ПХП?
                  Ответить

                  • ava TarasB 29.06.2011 13:21 # −1

                    Я имею в виду, в данном случае, когда выбор по логическому типу, если не ИСТИНА и не ЛОЖЬ, значит исключение.
                    (ещё вариант: (не ИСТИНА) и (не ЛОЖЬ)).
                    Ответить

                    • ava guest 29.06.2011 18:03 # −2

                      Хотя в пхп, с её системой типов метка default вполне может отработать в некоторых случаях...
                      switch (f())
{
case true: return true;
case false: return false;
default : return false;
}
                      Ответить

                    • ava LegionDead 30.06.2011 17:14 # 0

                      третье состояние кота шредингера?
                      Ответить

      • ava sonic 28.06.2011 22:06 # 0

        кододрочер
        *по доброму*
        Ответить

    • ava warider 28.06.2011 22:58 # −3

      хороший код. почему он здесь?
      1 сточка против четырех - ты тоже пишешь процедурно, а классы и объекты медленней работают?
      Ответить

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