Javascript-форум (https://javascript.ru/forum/)
-   Элементы интерфейса (https://javascript.ru/forum/dom-window/)
-   -   Проверить код (https://javascript.ru/forum/dom-window/71272-proverit-kod.html)

MyNameIsCode 06.11.2017 10:34

Проверить код
 
Привет!
Проверьте пожалуйста мой код, а то я с javascript не очень дружу пока.
Код работает, но все же хотел бы удостовериться что потом не будет головной боли

function () {
            var seoData = $("#regionsWithSeo")[0].innerHTML;
            var seoWithRegions = JSON.parse(seoData);
            var currData;

            $("#regionsId").change(function () {

                $("#regionsId option:selected").each(function () {
                    var rowSeo = $(this).text();
                    $.each(seoWithRegions, function() {
                        if (rowSeo === this.forSelect) {
                            currData = this;
                        }
                    });
                });
                $("#inputSeoTextBelow").val(currData.textBelow);
                $("#inputRegionId").val(currData.code);
            })
                .change();
        }

Dilettante_Pro 07.11.2017 11:13

MyNameIsCode,
Вызывает подозрение один момент:
Цитата:

Сообщение от MyNameIsCode
var seoData = $("#regionsWithSeo")[0].innerHTML;

Что, в HTML может быть несколько элементов с таким id?
Если так, то головная боль будет.

laimas 07.11.2017 12:21

Цитата:

Сообщение от Dilettante_Pro
Что, в HTML может быть несколько элементов с таким id?

Нет, это получение DOM, просто надо было $("#regionsWithSeo").html(), а если DOM, то это тоже самое что и $("#regionsWithSeo").get(0).innerHTML.

Dilettante_Pro 07.11.2017 13:46

laimas,
Я же и говорю - подозрение... Зачем-то ТС понадобилась такая конкретизация, хотя в остальных случаях при поиске по идентификатору он этого не делает.


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