Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Покритикуйте код пожалуйста. (https://javascript.ru/forum/project/32541-pokritikujjte-kod-pozhalujjsta.html)

vchmuzh 19.10.2012 23:37

Покритикуйте код пожалуйста.
 
1.5 месяца назад выкладывал сода свою работу
С того времени, прочитал уже почти весь learn, и решил все переделать, учитывая прошлую критику и советы.

У кого есть время и желание покритикуйте код, укажите на ошибки.
задание

Quiz

play/Quiz

PashPP 21.10.2012 23:26

Расскажи, что ты переделывал, исправлял?
Это может быть полезным послушать.

Код чей-то я еще не могу критиковать, но там не понятно, где, собственно твой собственный код, а где костыли. И во вкладке анимейт лишние функции неиспользуемых дельт. Копипаст для обучения это как-то не ок.

vchmuzh 22.10.2012 00:00

Если пройти по ссылке на тему с предыдущим вариантом, там можно почитать замечания которые мне сделали опытные пользователи сайта.
Вот я и старался их учитывать. То есть,старался лучше структурировать код, применял шаблоны, ООП (если я правильно понял что это :) ) и т.д.
хотелось бы попробовать с MVC но до этого я еще не дошел.

Мой код это все кроме helpers.js, animate.js, и tmpl.js
Я все эти функции уже не раз применял решая задачки с учебника, и не только, и первые разы старался писать их по памяти,честно :), но сейчас уже решил скопипастить, не думаю что это страшно

K313 26.10.2012 00:38

Цитата:

Я все эти функции уже не раз применял решая задачки с учебника, и не только, и первые разы старался писать их по памяти
не забивайте голову ненужными вещами. в любом развитом языке програмирования ОЧЕНЬ много функций. Всех их зазубрить - не хватит RAM-а даже для одного языка. Помнить нужно только то, где быстро найти подсказку, описание нужной функции (справочник, google, и тд) Часто используемые функции сами остануться в голове

vchmuzh 26.10.2012 16:26

Цитата:

Сообщение от K313 (Сообщение 212357)
не забивайте голову ненужными вещами. в любом развитом языке програмирования ОЧЕНЬ много функций. Всех их зазубрить - не хватит RAM-а даже для одного языка. Помнить нужно только то, где быстро найти подсказку, описание нужной функции (справочник, google, и тд) Часто используемые функции сами остануться в голове

ну да, вы правы.
Хотя когда я написал "по памяти", то неправильно выразился.
Я не хотел сказать что зазубривал их, а просто первые разы старался писать функцию сам, не подглядывая в учебник, а потом смотрел как ее написал автор в учебнике и сверялся.


Часовой пояс GMT +3, время: 01:11.