|
|
|
это нормально?
|
|||
|---|---|---|---|
|
#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&msg=38195975&tid=1448553]: |
0ms |
get settings: |
9ms |
get forum list: |
18ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
175ms |
get topic data: |
12ms |
get forum data: |
3ms |
get page messages: |
60ms |
get tp. blocked users: |
1ms |
| others: | 253ms |
| total: | 537ms |
| 0 / 0 |
