|
|
|
это нормально?
|
|||
|---|---|---|---|
|
#18+
Я тестер, знания в Javascript у меня начальные, то есть писать могу. Меня просили оценить чужие Javascripts. Ну, я уже сделал список тех, что не выполняются. Заметил еще пару халтурных приемов, причем специалисты со мной согласились. Хочу узнать мнение сообщества, считается ли нормальным или допустимым то, что я выделил в ошибки 1) try lala=123; catch (e){}; //пустой блок обработки ошибки 2)return null // зачем возвращать null ?! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 23.03.2013, 13:29 |
|
||
|
это нормально?
|
|||
|---|---|---|---|
|
#18+
1) Подавление не критичной ошибки. 2) Чтобы где-то проверить, что это именно null, а не undefined какой-нибудь. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 23.03.2013, 14:20 |
|
||
|
это нормально?
|
|||
|---|---|---|---|
|
#18+
и еще, там одна функция js, длиной в килобайт, скопипащена на двух десятках страниц, то есть не вынесена в отдельный файл. Это нормально ?Тяжек ли грех ?То есть ухудшается удобство поддержки, например. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 23.03.2013, 15:01 |
|
||
|
это нормально?
|
|||
|---|---|---|---|
|
#18+
В принципе да, это уже айяйяй. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 23.03.2013, 16:03 |
|
||
|
это нормально?
|
|||
|---|---|---|---|
|
#18+
и еще, закомментированные блоки по полкилобайта, это временное решение допустимое в продакшене или тоже аяяй? причем комментированные операторы,а не пояснения. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 23.03.2013, 17:14 |
|
||
|
|
start [/forum/topic.php?fid=22&gotonew=1&tid=1448553]: |
0ms |
get settings: |
12ms |
get forum list: |
18ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
70ms |
get topic data: |
14ms |
get first new msg: |
8ms |
get forum data: |
4ms |
get page messages: |
62ms |
get tp. blocked users: |
2ms |
| others: | 242ms |
| total: | 440ms |
| 0 / 0 |
