ЙажаСценарий / Говнокод #3567 Ссылка на оригинал

0

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

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

Запостил: mozg_raka mozg_raka, (Updated )

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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


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

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

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

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

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

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

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

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

  • ava Wеbkill # Ссылка на оригинал 0

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

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓