Javascript, объясните свойство конструкции?

Question

[deleted-tnorman]

Добрый день, я в js начинающий, вот так мой код изменил мой верховный главнокомандующий, но я с ним сейчас связаться не могу и мне интересно, что это значит? Он сказал, что исправил, чтоб избежать какой-то ошибку. Но я не могу понять какой ошибки можно было избежать такой реконструкцией кода? Всё же js не самый очевидный ЯП.

было:

document.getElementById("defaultForm").appendChild(this.containers.trigger);

стыло:

document.getElementById("defaultForm") {
            document.getElementById("defaultForm").appendChild(this.containers.trigger);
        }

Делают ли так? И если делают то с какой целью? Спасибо большое за ответ.

Answer 1

[Евгений Петров]

var form = document.getElementById('defaultForm');

if (form) {
	form.appendChild(this.containers.trigger);
}

Смысл в проверке существования элемента. И во втором случае у вас синтаксис неверный.
Javascript — язык нормальный и вполне очевидный.

Comments

[deleted-tnorman]

Хм, вопрос себя исчерпал. В моем представлении это выглядело так.

if ( document.getElementById("aspnetForm") ){ document.getElementById("aspnetForm").appendChild(this.containers.trigger);
};

Но глядя на код, который был на сервере, я подумал, что мой Мастер придумал что-то эдакое. Однако эта штука корявая находилась только на локальном сервере. На основном же сервере находится вот такая конструкция.

if ( document.getElementById("aspnetForm") ){ document.getElementById("aspnetForm").appendChild(this.containers.trigger);
};

Получается то, что я нашел на локальном никто ни с какой целью не делал.

Спасибо за ответ.

[Евгений Петров]

Не нужно искать повторно уже найденный элемент. Это — очень плохая практика. Присмотритесь к введению переменной в моём примере.

[deleted-tnorman]

Да, я переосмыслил суть быитя. Пользуемся единожды-найденным. Бритва Оккама и всё такое. Переписал по вашей подсказке.