Говнокод или есть смысл?

Question

[lightalex]

Доброго времени суток, коллеги!
Наткнулся на такую вот конструкцию:
parseInt( Math.floor( num ) )
Есть ли какой-то смысл в том что мы к числу которое вернуло Math.floor пытаемся применить функцию parseInt?
Или это говнокод?
Я просто понять не могу - зачем?
Ведь Math.floor уже возвращает целое число!
Заранее благодарен за ваши предположения :)

Comments

[Иван Сергеев]

Скорее есть смысл так: Math.floor( parseInt(num) )

[Дмитрий Беляев]

Иван Сергеев: тоже нет смысла, parseInt возвращает всегда целое

[Дмитрий Беляев]

Иван Сергеев: если быть точным, либо целое либо NaN

[lightalex]

Иван Сергеев: ну в исходном коде (рабочем коде) была именно моя конструкция
Я так понимаю это поддержка кроссбраузерности

Answer 1

[iBird Rose]

1) ну начнем с того что это точно не является говнокодом. потому что он не мешает работоспособности кода, не несет в себе костыли или баги, а также не вводит в заблуждение другого программиста.
2) да. этот код не несет в себе какой то пользы, но скорее всего он просто вырван из контекста. и внутри parseInt или была или планировалась формула, которая держала бы в себе не только int значения.

Comments

[lightalex]

1) Как минимум ввело в заблуждение программиста (то есть меня)
+ Зачем нужна лишняя операция если она не имеет смысла?
Это тоже самое что num = num + 1 - 1
2) Вот полная функция - https://jsfiddle.net/6kk9wydw/
Была там какая-то формула? Планировалась ли?
Но это рабочий код который грузится на компы миллионов юзеров Стима

[Ильдар Гизетдинов]

lightalex: Если вас это ввело заблуждение, вы плохой кодер, либо неопытный, на практике такой код можно периодически встретить. Пример: как написано выше могла быть какая то большая формула в другом месте, а человек который делал конкретно это просто скопировал и вырезал то что ему мешало. Другой пример: к примеру в каком либо из браузеров просто напросто Math.floor независимо от спецификации берет и возвращает не число (а стим, я уверен, старается поддерживать все что может), в результате, может возникнуть ошибка - самый простой способ, обернуть в такую конструкцию. Угадать вы этого не сможете, да, есть вероятность что программист просто ошибся, и добавил лишнию операцию, но на практике это никак не скажется на производительности.

[lightalex]

Ильдар Гизетдинов: про поддержку браузеров тоже подумал
Но что-то мне кажется прогер действительно ошибся
И зря вы так думаете про производительность
Есть такая пословица - копейка рубль бережет
Конечно же одна строка вообще никак не повлияет на производительность
Но лично я уже не одну подобную строку обнаружил в коде Стима
Да и то что он все время лагает, мне кажется вполне весомое доказательство плохого кода