1. JavaScript / Говнокод #3567

    +144

    1. 1
    var oneWeekAgo = dateObj.valueOf() - 604800000;

    "Ура! у меня есть калькулятор!!1"

    Запостил: mozg_raka, 25 Июня 2010

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

    • ava telnet 25.06.2010 11:04 # 0

      В чём говнокод? Прокомментировать магическое число, и нормально будет. Ну ещё getTime() вместо valueOf() для большей очевидности. Или я что-то не улавливаю?
      Ответить

      • ava 3.14159265 25.06.2010 11:47 # 0

        ну кагбе создать final int ms_WEEK = 1000*3600*24*7;
        было бы кошернее
        Ответить

        • ava telnet 25.06.2010 11:56 # +1

          > final int
          Это JS, а не Java.
          > 1000*3600*24*7
          Зло, имхо.
          Ответить

          • ava 3.14159265 25.06.2010 12:00 # 0

            >>Это JS, а не Java.
            на автомате написал ))

            в чем именно зло?
            в умножении или в числах вместо констант?
            Ответить

            • ava telnet 25.06.2010 12:06 # 0

              В умножении. Нет смысла при каждом запуске вычислять одно и то же. Выносить же здесь магическое число в константу смысла не вижу, если кусок скрипта изначально пишется под недельный интервал без прикидки на будущее изменение. Достаточно просто короткого комментария, откуда взялось число 604800000.
              Ответить

              • ava turdman 25.06.2010 12:11 # 0

                >Нет смысла при каждом запуске вычислять одно и то же
                Javascript портировали на Mark I?

                >Достаточно просто короткого комментария
                Зачем писать
                - 604800000; // 1000*3600*24*7
                когда можно написать
                - 1000*3600*24*7
                Ответить

                • ava telnet 25.06.2010 12:15 # 0

                  Корпение над вопросами производительности мелочей - годная тема для холиваров, да. По-моему, тут каждый строчит как хочет. В общем, предлагаю не разводить демагогии, тем более что мы до сих пор не услышали мнения автора, в чём, собственно, говнокод-то.
                  Ответить

                • ava Анонимус 25.06.2010 12:42 # +1

                  >>Javascript портировали на Mark I?

                  Сюда:
                  http://ru.wikipedia.org/wiki/%D0%A4%D0%B0%D0%B9%D0%BB:British_Mark_I_ Tank.jpg

                  ?
                  Ответить

                  • ava 3.14159265 25.06.2010 12:59 # 0

                    доставило ))
                    но есть еще такая древняя ЭВМ с произодительностью ниже калькулятора
                    Ответить

        • ava HyperGeek 25.06.2010 12:12 # 0

          А почему не так
          const MS_IN_WEEK = MS_IN_SEC * SEC_IN_HOUR * HOURS_IN_DAY * DAYS_IN_WEEK
          Ответить

          • ava 3.14159265 25.06.2010 12:23 # 0

            >>const MS_IN_WEEK = MS_IN_SEC * SEC_IN_HOUR * HOURS_IN_DAY * DAYS_IN_WEEK
            я так обычно и пишу, когда константы используются в проекте в нескольких местах это было для краткости и понятности
            Ответить

          • ava Анонимус 25.06.2010 12:43 # 0

            const в JS же не работает в половине браузеров, нет?
            Ответить

            • ava telnet 25.06.2010 12:51 # 0

              Так точно.
              Ответить

            • ava HyperGeek 25.06.2010 12:57 # 0

              Не работает только в ИЕ, который многие и за браузер не считают )) Т.е. если бы не var, а const, то код нормальный?
              Ответить

              • ava Анонимус 25.06.2010 13:00 # −4

                Может и не считают, да только 85% юзверей на нем сидит.

                Я бы сделал префикс, что бы показать что это константа.

                var c_MS_IN_WEEK = ....
                как-то так
                Ответить

                • ava Анонимус 25.06.2010 22:03 # 0

                  судя по тому, как меня минусуют, всем нравится писать
                  i = 42 * 100500 - 3;
                  Ответить

                  • ava koodeer 25.06.2010 22:31 # 0

                    отминусовали 85% юзверей, сидящих на осле
                    Ответить

                  • ava Lure Of Chaos 25.06.2010 22:44 # +2

                    честно говоря, меня устраивает решение
                    const MS_IN_WEEK=604800000; // 1000*3600*24*7


                    а плодить ради одной константы еще кучу констант
                    >MS_IN_SEC * SEC_IN_HOUR * HOURS_IN_DAY * DAYS_IN_WEEK
                    как существо крайне ленивое, считаю совершенно излишним

                    з.ы. хоть заминусуйтесь, ниипет, чесна-чесна-ё.
                    Ответить

                • ava WGH 27.06.2010 12:30 # 0

                  Зачем префикс? Разве заглавных букв мало, чтобы было понятно, что это константа?
                  Ответить

      • ava HyperGeek 25.06.2010 12:06 # +2

        Да, может даже и комментировать не надо — oneWeekAgo далеко не двусмысленно. В последнее время на Говнокоде совсем на константах помешались. Чтоб сделать простейшую операцию нужно констант наплодить и 10 строчек комментариев накатать.
        Этот код прост и понятен, проще уже некуда.
        Ответить

        • ava telnet 25.06.2010 12:12 # +5

          На чём тут реально помешались, это на заливке своих поделий и просьбах прокомментировать. Ладно хоть сессия к концу подходит, а у кого-то кончилась уже.
          Ответить

    • ava Wеbkill 25.06.2010 23:31 # −10

      показать все, что скрытоЯ размножаюсь почкованием и гермофрадитными отростками. Держите меня семеро! Начинаю...
      Ответить

    • ava Wеbkill 28.06.2010 09:05 # −1

      Ой говнокод... Тут нужна константа.
      Ответить

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