После проверки на наличие элемента ошибка Cannot read property?

Question

[valitskiydmitriy]

Здесь подсказали как сделать проверку на наличие элемента, ошибка была решена но появилась новая, вот полный код скрипта:

window.onload = function () {
    var day = new Date,
        md = (new Date(day.getFullYear(), day.getMonth() + 1, 0, 0, 0, 0, 0)).getDate(),
        $month_name = "января февраля марта апреля мая июня июля августа сентября октября ноября декабря".split(" ");

        function set_select(a, c, d, e) {
            var el = document.getElementsByName(a)[0];

            if (!el) {
                console.warn('Element "' + a + '" not found');
                return;
            }

            for (var b = el.options.length = 0; b < c; b++) {
                el.options[b] = new Option(a == 'month' ? $month_name[b] : b + d, b + d);
            }

            el.options[e] && (el.options[e].selected = !0)
        }
    set_select("day", md, 1, day.getDate() - 1);
    set_select("month", 12, 1, day.getMonth());
    set_select("year", 50, day.getFullYear()-50, 50);


    var year = document.getElementById('year');
    var month = document.getElementById("month");

    function check_date() {
        var a = year.value | 0,
            c = month.value | 0;
        md = (new Date(a, c, 0, 0, 0, 0, 0)).getDate();
        a = document.getElementById("day").selectedIndex;
        set_select("day", md, 1, a)
    };

    if (document.addEventListener) {
        year.addEventListener('change', check_date, false);
        month.addEventListener('change', check_date, false);

    } else {
        year.detachEvent('onchange', check_date);
        month.detachEvent('onchange', check_date);
    }

}

Ошибка:

Uncaught TypeError: Cannot read property 'addEventListener' of null

Answer 1

[Алексей Уколов]

Тоже самое же - вы не проверяете есть ли на странице элементы с id year и month прежде чем навешивать на них обработчики.
Алгоритм решения очень прост:
1. Ищете все места, где используется функция addEventListener (ну а правильнее, конечно, прямо в консоли посмотреть конкретный файл и строку).
2. Смотрите, при каких условиях переменная, на которой вызывается этот метод может быть null.
3. Добавляете необходимые проверки.

Comments

[valitskiydmitriy]

Алексей Уколов А на каком этапе нужно проверять year и month?

[valitskiydmitriy]

Алексей Уколов Где set_select или var?

[Алексей Уколов]

valitskiydmitriy: попробуйте подумать самостоятельно, алгоритм я вам написал - там же всего три шага. Подумайте, что означает эта ошибка, что может к ней привести и вам сразу станет ясно, как её избежать.

[valitskiydmitriy]

Алексей Уколов: Никак не получается. Ошибок новых нет, но и эта не решается, проверял и day и month

[Алексей Уколов]

valitskiydmitriy: во-первых, при чем тут day, я же написал в каких элементах проблема? А во-вторых, покажите код.

[valitskiydmitriy]

Алексей Уколов Извиняюсь перепутал элементы, вот к примеру:

function check_date() {
        var a = year.value | 0,
            c = month.value | 0;
            var ch = document.getElementsByName('month','year');
            if (!ch) {
        console.warn('Element "' +cha + '" not found');
        return;
    }

        md = (new Date(a, c, 0, 0, 0, 0, 0)).getDate();
        a = document.getElementById("day").selectedIndex;
        set_select("day", md, 1, a)
    };

    if (document.addEventListener) {

        year.addEventListener('change', check_date, false);
        month.addEventListener('change', check_date, false);

    } else {
        year.detachEvent('onchange', check_date);
        month.detachEvent('onchange', check_date);
    }

}

[Алексей Уколов]

А зачем вы эту проверку вставили в check_date? Какое отношение по вашему эта функция имеет к указанной ошибке? Вы вообще код читать умеете? Вы видите в какой строке возникает ошибка? Вы понимаете почему она возникает?

[valitskiydmitriy]

Алексей Уколов: Разве ошибка не возникает в этой функции? Ошибка возникает как я полагаю в строке

year.addEventListener('change', check_date, false);
month.addEventListener('change', check_date, false);
 <code>
Ошибка возникает потому что он не видит этих элементов, поэтому и вставил туда

[Алексей Уколов]

Ошибка действительно возникает там и по этой причине, но установка обработчиков событий происходит вне функции check_date. Попробуйте нормально отформатировать тот кусок кода, который вы приложили и вы всё сами увидите (включая лишнюю фигурную скобку).