Как устранить ошибку Uncaught TypeError: btnLeft.onclick is not a function?

@AlexKOR5 · Регистрация: 15.02.2019

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

Как устранить ошибку Uncaught TypeError: btnLeft.onclick is not a function без потери правильной работы слайдера?

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
let init = () => {
  blocks = document.querySelectorAll('.slider__item');
  Slider = document.querySelector('.slider');
  btnLeft = document.getElementById('btnLeft');
  btnRight = document.getElementById('btnRight');
  SafeSrc = [];
  gapRight = 0;
  step = 0;
  CoordsBlocks = [...blocks].map((curentV, index, array) => {
    return array[index].getBoundingClientRect();
  });
 
  for (let i = 0; i < blocks.length; i++) {
    blocks[i].remove();
  }
  for (let i = 0; i < blocks.length; i++) {
    blocks[i].remove();
  }
  Slider.append(blocks[0]);
 
  btnLeft.onclick = () => {
    drawRight();
    moveLeft();
 
    let y = btnRight.onclick;
    btnRight.onclick = null;
    setTimeout(() => {
      btnRight.onclick = y;
    }, 1000);
  }
 
  btnRight.onclick = () => {
    drawLeft();
    moveRight();
 
    let y = btnLeft.onclick;
    btnLeft.onclick = null;
    setTimeout(() => {
      btnLeft.onclick = y;
    }, 1000);
  }
  let ty = setInterval(() => {
    btnLeft.onclick();
  }, 1000);
 
}//end of init
let drawRight = () => {
  ++step;
  if (step >= blocks.length) {
    step = 0;
  }
  let gapRight = CoordsBlocks[step].width;
  blocks[step].style.left = gapRight + 'px';
  Slider.append(blocks[step]);
}
let drawLeft = () => {
  --step;
  if (step < 0) {
    step = blocks.length - 1;
  }
  let gapRight = CoordsBlocks[step].width;
  blocks[step].style.left = -gapRight + 'px';
  Slider.append(blocks[step]);
}
let moveLeft = () => {
  let x = btnLeft.onclick;
  btnLeft.onclick = null;
  let twoElems = document.querySelectorAll('.slider__item');
  let offset = -1;
 
  CoordstwoElems = [...twoElems].map((curentV, index, array) => {
    return array[index].getBoundingClientRect();
  });
 
  for (let i = 0; i < twoElems.length; i++) {
    twoElems[i].style.left = offset * CoordstwoElems[i].width + 'px';
    offset++;
  }
  setTimeout(() => {
    twoElems[0].remove();
    btnLeft.onclick = x;
  }, 1000);
}
 
let moveRight = () => {
  let x = btnRight.onclick;
  btnRight.onclick = null;
  let twoElems = document.querySelectorAll('.slider__item');
  let offset = 0;
  twoElems = Array.from(twoElems);
  twoElems = twoElems.reverse();
 
  CoordstwoElems = [...twoElems].map((curentV, index, array) => {
    return array[index].getBoundingClientRect();
  });
  CoordstwoElems.reverse();
 
  for (let i = 0; i < twoElems.length; i++) {
    twoElems[i].style.left = offset * CoordstwoElems[i].width + 'px';
    offset++;
  }
  setTimeout(() => {
    twoElems[twoElems.length - 1].remove();
    btnRight.onclick = x;
  }, 1000);
}
window.addEventListener('load', init);

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<body>
    <div class="slider">
        <div class="slider__item slider__img1"></div>
        <div class="slider__item slider__img2"></div>
        <div class="slider__item slider__img3"></div>
        <div class="slider__item slider__img4"></div>
        <div class="slider__item slider__img5"></div>
        <div class="slider__item slider__img6"></div>
    </div>
    <div class="buttons">
        <button id="btnLeft">left</button>
        <button id="btnRight">Right</button>
    </div>
 
    <script src="script.js"></script>
</body>

CSS
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
*{
    margin:0px;
    padding:0px;
    box-sizing: border-box;
}
.slider{
    width:700px;
    height:400px;
    border:1px solid blue;
    margin:100px auto 0px auto;
    position: relative;
    /* overflow: hidden; */
}
.slider__item{
    width: 700px;
    height:400px;
    position: absolute;
    top:0px;
    left:0px;
    background-repeat: no-repeat;
    background-size: cover;
    transition:all 1s ease-in-out 0s;
}
.slider__item > img{
    width:100%;
    height:100%;
}
.slider__img1{
    background-image: url(imgs/1.jpg);
}
.slider__img2{
    background-image: url(imgs/2.jpg);
}
.slider__img3{
    background-image: url(imgs/3.jpg);
}
.slider__img4{
    background-image: url(imgs/4.jpg);
}
.slider__img5{
    background-image: url(imgs/5.jpg);
}
.slider__img6{
    background-image: url(imgs/6.jpg);
}

@Iverycool · 10.08.2020, 20:19

AlexKOR5, нельзя просто так взять и вызвать onclick!
Замените строку 43 на btnLeft.click();

Не по теме:

У вас в коде полно и других ошибок, например нажатие кнопок смены кадра "перекрывается" постоянным листанием

@AlexKOR5 · 10.08.2020, 20:37 **[ТС]**

Спасибо за ответ.
Уточните пожалуйста за ошибки.

@Iverycool · 10.08.2020, 20:59

AlexKOR5, причина ошибок кроется в "перенакосячении" логики работы вашей программы.
Ошибку исправлять у меня сейчас нет времени, обращу внимание лишь на явно видимые "проблемы":
1. В функции init переменные не объявлены, тем самым загрязняя глобальную область
2. SafeSrc и CoordsBlocks с большой буквы, зачем? Только конструкторы классов должны начинаться с большой буквы
Про ваши переменные говорить можно много, лучше почитайте https://learn.javascript.ru/variables

Сообщение от AlexKOR5

}//end of init

Вспоминаю ооочень старые добрые времена

лучше просто после окончании функции ставить пустую строку, а ещё лучше разбить проект на отдельные файлы.

Сообщение от AlexKOR5

[...blocks].map((curentV, index, array) => {
return array[index].getBoundingClientRect();
});

Зачем так мудрить?

JavaScript
1
[...blocks].map(block => block.getBoundingClientRect());

https://learn.javascript.ru/array-methods#map
Постарайтесь прочитать учебник

@AlexKOR5 · 10.08.2020, 22:06 **[ТС]**

Спасибо большое.
Если все же будет время то напишите в чем именно ошибка, я бы хотел это исправить сам.

@j2FunOnly · 11.08.2020, 09:33

Сообщение от AlexKOR5

в чем именно ошибка

Вы управляете состоянием своего слайдера используя атрибуты элементов.
При клике на btnRight(32), вы устанавливаете btnLeft.onclick = null и вызываете этот null в интервале btnLeft.onclick();(43), там много у вас таких мест.

В ~~двух словах~~: ваш слайдер должен быть независим от кнопок, то есть у него есть слайды, он знает как и в какую сторону их менять (или не менять, то есть слайдер сам "должен решать" что ему делать, в зависимости от своего состояния), и есть у него "интерфейс" с методами, которые вы можете вызывать в любое время (в данном случае при нажатии на кнопку), а тут ваши кнопки "слишком много знают" и получается мешанина, что тоскливо становится в этой простыне разбираться.

@AlexKOR5 · 12.08.2020, 08:53 **[ТС]**

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

Я не знаю как сделать независимый слайдер от кнопок чтобы все прекрасно работало.
Помогите.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
let leftBtn = document.getElementById('btnLeft');
let rightBtn = document.getElementById('btnRight');
const slider = document.querySelector('.slider');
let slides = document.querySelectorAll('.slider__item');
let slidesWidth = [...slides].map(currentV => { return currentV.getBoundingClientRect() });
let step = 0;
let interval;
 
let init = () => {
    for (let i = 1; i < slides.length; i++) {
        slides[i].remove();
    }
    leftBtn.addEventListener('click', actionLeftbtn);
    rightBtn.addEventListener('click', actionRightbtn);
    interval = setInterval(() => {
        leftBtn.click();
    }, 1000);
    window.onblur = () => {
        clearInterval(interval);
    }
    window.onfocus = () => {
        interval = setInterval(() => {
            leftBtn.click();
        }, 1000);
    }
}//end of init
 
let actionLeftbtn = () => {
    drawRight(moveLeft);
    rightBtn.removeEventListener('click', actionRightbtn);
    setTimeout(() => {
        rightBtn.addEventListener('click', actionRightbtn);
    }, 500);
}
 
let actionRightbtn = () => {
    drawLeft(moveRight);
    leftBtn.removeEventListener('click', actionLeftbtn);
    setTimeout(() => {
        leftBtn.addEventListener('click', actionLeftbtn);
    }, 500);
}
 
let drawRight = (callback) => {
    step++;
    step > slides.length - 1 ? step = 0 : false;
    slides[step].style.left = slidesWidth[step].width + 'px';
    slider.append(slides[step]);
    callback();
}
 
let drawLeft = (callback) => {
    step--;
    step < 0 ? step = slides.length - 1 : false;
    slides[step].style.left = -slidesWidth[step].width + 'px';
    slider.append(slides[step]);
    callback();
}
 
let moveLeft = () => {
    leftBtn.removeEventListener('click', actionLeftbtn);
    let twoElems = document.querySelectorAll('.slider__item');
    let twoElemsWidth = [...twoElems].map(currentV => { return currentV.getBoundingClientRect() });
    let offset = -1;
    for (let i = 0; i < twoElems.length; i++) {
        twoElems[i].style.left = offset * twoElemsWidth[i].width + 'px';
        ++offset;
    }
    setTimeout(() => {
        twoElems[0].remove();
        leftBtn.addEventListener('click', actionLeftbtn);
    }, 500);
}
 
let moveRight = () => {
    rightBtn.removeEventListener('click', actionRightbtn);
    let twoElems = document.querySelectorAll('.slider__item');
    twoElems = Array.from(twoElems);
    twoElems = twoElems.reverse();
    let twoElemsWidth = [...twoElems].map(currentV => { return currentV.getBoundingClientRect() });
    let offset = 0;
    for (let i = 0; i < twoElems.length; i++) {
        twoElems[i].style.left = offset * twoElemsWidth[i].width + 'px';
        ++offset;
    }
    setTimeout(() => {
        twoElems[twoElems.length - 1].remove();
        rightBtn.addEventListener('click', actionRightbtn);
    }, 500);
}
 
window.addEventListener('load', init, false);

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<body>
    <div class="slider">
        <div class="slider__item slider__img1"></div>
        <div class="slider__item slider__img2"></div>
        <div class="slider__item slider__img3"></div>
        <div class="slider__item slider__img4"></div>
        <div class="slider__item slider__img5"></div>
        <div class="slider__item slider__img6"></div>
    </div>
    <div class="buttons">
        <button id="btnLeft" class="button-style">Left</button>
        <button id="btnRight" class="button-style">Right</button>
    </div>
 
    <script src="script.js"></script>
</body>

CSS
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
*{
    margin:0px;
    padding:0px;
    box-sizing: border-box;
}
.buttons{
    width:200px;
    margin:20px auto 0px auto;
    display: flex;
    flex-flow: row wrap;
}
.button-style{
    width:50%;
    height:50px;
}
.slider{
    width:1141px;
    height:331px;
    border:1px solid blue;
    margin:10px auto;
    position: relative;
}
.slider__item{
    width:1141px;
    height:331px;
 
    background-repeat: no-repeat;
    background-size: cover;
    position: absolute;
    top:0px;
    left:0px;
    transition:all 0.5s ease-in-out 0s;
}
.slider__img1{
    background-image: url('imgs/img-1.png');
}
.slider__img2{
    background-image: url('imgs/img-2.png');
}
.slider__img3{
    background-image: url('imgs/img-3.png');
}
.slider__img4{
    background-image: url('imgs/img-4.png');
}
.slider__img5{
    background-image: url('imgs/img-5.png');
}
.slider__img6{
    background-image: url('imgs/img-6.png');
}

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

@Iverycool 1586 / 796 / 362 Регистрация: 01.02.2019 Сообщений: 1,047
	10.08.2020, 20:19
	AlexKOR5, нельзя просто так взять и вызвать onclick! Замените строку 43 на btnLeft.click(); Не по теме: У вас в коде полно и других ошибок, например нажатие кнопок смены кадра "перекрывается" постоянным листанием 1

@AlexKOR5 50 / 14 / 3 Регистрация: 15.02.2019 Сообщений: 514
	10.08.2020, 20:37 [ТС]
	Спасибо за ответ. Уточните пожалуйста за ошибки. 0

@AlexKOR5 50 / 14 / 3 Регистрация: 15.02.2019 Сообщений: 514
	10.08.2020, 22:06 [ТС]
	Спасибо большое. Если все же будет время то напишите в чем именно ошибка, я бы хотел это исправить сам. 0