Как оптимизировать повторяющийся js код?

Question

[Антон]

Есть подозрение, что это можно улучшить

function fun1() {
var chbox;
chbox=document.getElementById('tab5');
	if (chbox.checked) {
		document.querySelector(".next-slide-text").click();
	}
}
function fun2() {
var chbox;
chbox=document.getElementById('dvd1');
	if (chbox.checked) {
		document.querySelector(".next-slide-text").click();
	}
}
function fun3() {
var chbox;
chbox=document.getElementById('dvd2');
	if (chbox.checked) {
		document.querySelector(".next-slide-text").click();
	}
}
function fun4() {
var chbox;
chbox=document.getElementById('dvd3');
	if (chbox.checked) {
		document.querySelector(".next-slide-text").click();
	}
}
function fun5() {
var chbox;
chbox=document.getElementById('dvd4');
	if (chbox.checked) {
		document.querySelector(".next-slide-text").click();
	}
}

Answer 1

[Arman]

function fun(id, selector) {
  selector || (selector = ".next-slide-text");

  var chbox = document.getElementById(id);
  if (chbox.checked) {
    document.querySelector(selector).click();
  }
}

fun('tab5');
fun('dvd1');

Comments

[Антон]

так не работает.

[Петр Мухуров]

anton99zel, он не может не работать) Он не будет работать только в том случае, если не работает ваш код, который вы написали выше, либо вы не передаете id в функцию
Arik

function fun(id, selector = ".next-slide-text") {
var chbox = ...
}

[Антон]

Сейчас у меня так:

<input type="radio"  onchange="fun1()" id="dvd1"/>
<input type="radio"  onchange="fun2()" id="dvd2"/>
<input type="radio"  onchange="fun3()" id="dvd3"/>
<input type="radio"  onchange="fun4()" id="dvd4"/>
<input type="radio"  onchange="fun5()" id="tab5">
<script>
function fun1() {
var chbox;
chbox=document.getElementById('tab5');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun2() {
var chbox;
chbox=document.getElementById('dvd1');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun3() {
var chbox;
chbox=document.getElementById('dvd2');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun4() {
var chbox;
chbox=document.getElementById('dvd3');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun5() {
var chbox;
chbox=document.getElementById('dvd4');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
</script>

Я сделал так:

<input type="radio"  onchange="fun1()" id="dvd1"/>
<input type="radio"  onchange="fun2()" id="dvd2"/>
<input type="radio"  onchange="fun3()" id="dvd3"/>
<input type="radio"  onchange="fun4()" id="dvd4"/>
<input type="radio"  onchange="fun5()" id="tab5">
<script>
function fun(id, selector) {
  selector || (selector = ".next-slide-text");

  var chbox = document.getElementById(id);
  if (chbox.checked) {
    document.querySelector(selector).click();
  }
}

fun('tab5');
fun('dvd1');
fun('dvd2');
fun('dvd3');
fun('dvd4');

</script>

Что я не так сделал?

[Dmitry Alekseev]

anton99zel, на onChange идёт попрежнему fun1, fun2

[Антон]

Dmitry Alekseev, ничего не понял. Как правильно?

[Dmitry Alekseev]

На событие input.onChange вызывается fun1, fun2 и тд., когда определена функция fun.

[levchak0910]

<input type="radio"  onchange="fun(this)" id="dvd1">
<input type="radio"  onchange="fun(this)" id="dvd2">
<input type="radio"  onchange="fun(this)" id="dvd3">
<input type="radio"  onchange="fun(this)" id="dvd4">
<input type="radio"  onchange="fun(this)" id="tab5">

function fun(chbox) {
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}