Ошибка Uncaught TypeError: Cannot read property 'toggle' of undefined

@ThePhoenix · Регистрация: 17.11.2018

Author24 — интернет-сервис помощи студентам

В js файле есть код:

Javascript

$( document ).ready(function() {
    var value = $("#s_theme").text();
    var obj = document.getElementsByClassName('settings');
    function main () {
    if (value != 'Ok') {
        obj.classList.add('theme_hide');
    }
    else {
        obj.classList.remove('theme_hide');
    }
    }
    var inter = setInterval(main,500);
});

Но в консоли ошибка:
script.js:6 Uncaught TypeError: Cannot read property 'add' of undefined
at main (script.js:6)

Можете указать, в чем ошибка?

@amr-now · 02.03.2019, 12:20

Надо

Javascript

1	var obj = document.getElementsByClassName('settings')[0];

или

Javascript

1	var obj = document.querySelector('.settings');

Добавлено через 16 минут
А вообще в jQuery надо хотя бы так написать:

PHP/HTML

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
</head>
 
<body>
    <div id="s_theme">текст</div>
    <div class="settings"></div>
    <script>
        $(function () {
            function main() {
                var value = $("#s_theme").text();
                var obj = $('.settings');
                if (value != 'Ok') {
                    obj.addClass('theme_hide');
                }
                else {
                    obj.removeClass('theme_hide');
                }
            }
            var inter = setInterval(main, 500);
        });   
    </script>
</body>
</html>

Не надо делать лишние замыкания на переменные.
-----
Да и проверка значения по интервалу тоже говорит о нехорошем проектировании программы...

@ThePhoenix · 02.03.2019, 12:29 **[ТС]**

Понял. Принял. Я еще только начал изучение этого

@amr-now · 02.03.2019, 14:15

ThePhoenix, если есть опыт программирования в других языках и есть желание поиграться с наблюдением за изменением объекта,
то посмотрите MutationObserver за UI-элементом,

для POJO-классов - Object.observe(), класс Proxy и классическую реализацию Наблюдателя на геттерах/сеттерах.

Сейчас вообще мода играться с data binding. Так что да. Актуально. А там и до Vue.js c Реактом скатитесь...

@ThePhoenix 4 / 5 / 0 Регистрация: 17.11.2018 Сообщений: 23
	02.03.2019, 12:29 [ТС]	3
	Понял. Принял. Я еще только начал изучение этого 0

@amr-now 6492 / 3903 / 2005 Регистрация: 14.06.2018 Сообщений: 6,781
	02.03.2019, 14:15	4
	ThePhoenix, если есть опыт программирования в других языках и есть желание поиграться с наблюдением за изменением объекта, то посмотрите MutationObserver за UI-элементом, для POJO-классов - Object.observe(), класс Proxy и классическую реализацию Наблюдателя на геттерах/сеттерах. Сейчас вообще мода играться с data binding. Так что да. Актуально. А там и до Vue.js c Реактом скатитесь... 0

Ошибка Uncaught TypeError: Cannot read property 'toggle' of undefined

Решение