"PHP" / Говнокод #7098 Ссылка на оригинал

0

  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 cartman, (Updated )

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

  • ava cartman # Ссылка на оригинал 0

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

    • ava yasenhui # Ссылка на оригинал 0 ⇈

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

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

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

      • ava guest # Ссылка на оригинал 0 ⇈

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

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

      • ava yasenhui # Ссылка на оригинал 0 ⇈

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

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

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

      • ava TarasB # Ссылка на оригинал 0 ⇈

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

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

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

    return true;

}
else
{

    return false;

}


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

        • ava nislova # Ссылка на оригинал 0 ⇈

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

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

          • ava Vasiliy # Ссылка на оригинал 0 ⇈

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

            • ava 3.14159265 # Ссылка на оригинал 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 # Ссылка на оригинал 0 ⇈

                Гуру!

                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 # Ссылка на оригинал 0 ⇈

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

                  • ava guest # Ссылка на оригинал 0 ⇈

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

  • ava warider # Ссылка на оригинал 0

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

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓