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

    +4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    /**
  * @param int $day
  * @return bool
  */
protected function isWorkingDay($day)
{
    return $day % 8 <= 5;
}

    Cпособ определить, является ли указанный день (передается номер дня в неделе от 1 до 7) рабочим. Чем автора не устроил return $day <= 5 - непонятно..

    Запостил: feversocial, 11 Июля 2016

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

    • ava Hubactep 11.07.2016 19:12 # 0

      Наверное тем что предполагалось что будет передаваться день месяца, а потом забили?
      Ответить

      • ava 3_14dar 11.07.2016 21:46 # 0

        А может день с начала года или что-то в этом духе.
        Ответить

      • ava feversocial 11.07.2016 22:30 # 0

        передается именно номер дня в неделе, т.е. date('N'). в данном методе можно хоть $day % 8, хоть $day % 20 - результат при $day <= 7 все равно один и тот же :-)
        Ответить

    • ava 3_dar 11.07.2016 19:44 # 0

      Тогда уж (day - 1) % 7 < 5
      Ответить

    • ava bormand 11.07.2016 20:04 # +2

      > @param int $day
      > @return bool
      Ахуенный докстринг.
      Ответить

      • ava 3_dar 11.07.2016 20:40 # 0

        я сначала подумал что % 8 - это чтобы к инту привести, но потом посмотрел на phpDoc
        Ответить

        • ava bormand 11.07.2016 20:43 # +3

          А вдруг они там правда работают 6 дней из 8?
          Ответить

      • ava 3_14dar 11.07.2016 21:46 # 0

        Что не так?
        Ответить

        • ava Soul_re@ver 11.07.2016 23:07 # +3

          Из него просто офигенно понятно, что делает данная функция, какие параметры она ждёт и какие на них есть ограничения.
          Ответить

          • ava 1024-- 12.07.2016 10:28 # 0

            Ну раз всё понятно, тогда что не так⸮
            Ответить

          • ava 3_14dar 13.07.2016 17:12 # 0

            Ну разве что можно добавить описание $day. А в ообще докстринг сделан на скорую руку для того чтобы указать типы, кому не нравится пусть допишет :)
            Ответить

          • ava Bobik 15.07.2016 00:47 # 0

            Вот меня на работе заставляют как-то так делать:
            // This class holds debug information.
class DebugInformation {
    public:
        // Returns the link to see debug information.
        string link() const { return m_link; }
        // Sets the link to see debug information.
        void set_link(string link) { m_link = std::move(link); }
    private:
        // Holds the link to see debug information.
        string link;
}

            НАХУЯ?
            Ответить

            • ava bormand 15.07.2016 00:50 # +1

              > string link
              > m_link = std::move(link);
              И правда, нахуя?
              Ответить

              • ava Bobik 15.07.2016 00:56 # +1

                void set_link(string link) показывает, что класс DebugInformation скопирует к себе строку, переданную как параметр. Если бы передавался const string&, caller мог бы подумать, что мы берём ссылку на строку, и исходную строку нельзя освобождать.
                А подход c std::move позволяет иметь столько же копирований, сколько и при приёме const string&, но быть более понятным в сигнатуре.
                Ответить

                • ava Soul_re@ver 15.07.2016 02:08 # +2

                  А если оригинальная строка не нужна и вызывающий сделал move сам, то получается очень быстро. Данный подход позволяет избавиться от 2х перегрузок для l и r-values
                  Ответить

              • ava roman-kashitsyn 15.07.2016 10:34 # +3

                > И правда, нахуя?

                Борманд, где твоё C++-чутьё? Так же можно сыкономить одно копирование, выжать каплю ПИРФОМАНСА, если в качестве входного параметра передаётся временный объект! Мув же почти бесплатный.
                // Временный объект, созданный из const char *,
// не будет скопирован, его содержимое переедет
// внутрь debugInfo без копирования!
debugInfo.set_link("http://lmgtfy.com/");

// Аналогично, если вызывающему строка
// больше не нужна, и он мувает её сам.
debugInfo.set_link(std::move(rtfm_link));
                Ну и так меньше унылых констрефов писать.
                Ответить

                • ava 3_dar 15.07.2016 22:35 # 0

                  А почему в первом примере не произойдет копирование по значению при передаче в set_link?
                  Ответить

                  • ava Soul_re@ver 15.07.2016 22:40 # 0

                    Потому что его переместят а не скопируют.
                    Ответить

                    • ava 3_dar 15.07.2016 22:49 # 0

                      так а почему его переместят?
                      метод ожидает string, и я думал const char* преобразуется в string, потом по значению передается в set_link
                      Ответить

                      • ava Soul_re@ver 15.07.2016 22:51 # 0

                        У string есть конструктор, принимающий const char* то есть параметр set_link будет создан напрямую из const char*
                        Ответить

      • ava Fike 14.07.2016 23:37 # 0

        да это автогенерация, к которой проебали написать описание
        Ответить

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