- 1
- 2
- 3
- 4
- 5
if (!field.parentElement.childNodes[0].checked)
{
getElementByFieldName("Original").parentNode.childNodes[0].disabled = true;
getElementByFieldName("Duplicate").parentNode.childNodes[0].disabled = true;
...-
JavaScript / Говнокод #8612
+153
Далее идет ещё примерно 20 аналогичных обращений, аля parentNode.childNodes[0]
Очень читабельный и понятный код! :)Запостил:
LexeY4eg,
23 Ноября 2011
Follow us!
getElementByFieldName("Original").parent Node.childNodes[0].disabled = true;
getElementByFieldName("Duplicate").paren tNode.childNodes[0].disabled = true;
Тут как минимум есть смысл сделать метод, чтобы не писать каждый раз XXX.parentNode.childNodes[0], а конкретно в моем случае, искомый элемент можно было получить вообще другим способом, сформировав нужное имя (суть в том что рисуется форма, и на ней по 2 элемента, 1-й для отображения пользователю, 2-й для хранения данных, и они отлачаются только тем что у 1-го в имени есть префикс label). Но даже если не рассматривать мой конкретный случай, считаю этот код кривым, т.к. всегда можно найти способ нормального доступа к элементам а не через такие дурацкие цепочки.
Говорить с набитым ртом - неприлично
Нет, я не пьян и не под воздействием препаратов
- Мадам, я уже пять дней ничего не ем.
- Завидую вам! - вздохнула дама. - Если бы у меня была такая сила воли!
// А чтобы получить моральное право учить терпению, не помешало бы закутаться в простыню, положить мокрую тряпку на голову и пройти по границе Индии с Пакистаном, иногда наступая на подкладываемые осколки стекол и коровьи лепешки... ;)