Zależnie od wyboru w select-ach pobierana jest wartość z inputa[JAVA-SCRIPT]

Question

kod całej aplikacji w issue https://forum.pasja-informatyki.pl/562339/czy-jest-funkcja-sprawdzajaca-czy-input-miejsci-sie-w-wartosc-tablicy

Zależnie od wyboru w select-ach pobierana jest wartość z inputa[JAVA-SCRIPT] i wypisywana jako wynik w AX =f1,BX =F3

Tu jest funkcja od inputów

window.addEventListener("load", load);

function load() {
	const register = document.querySelector("#rejestry");
	const answer = document.querySelector(".Instructions");
	register.addEventListener("click", buttonAction);
	register.addEventListener("input", checkInput);
	answer.addEventListener("click", buttonCounter);
}

function buttonAction(e) {
	if (e.target.nodeName == "BUTTON") {
		switch (e.target.className) {
			
			case "save-button":
				const p = e.target.closest("div").querySelector("p").textContent,
					input = e.target.closest("div").querySelector("input");
				input.value = getRandomHex();
				input.classList.remove("hex-valid-value", "hex-invalid-value");
				console.log(p + " " + input.value.trim());
				break;
		}
	}
}

A tu początek tego co chciał bym osignać

function buttonCounter(e) {
	if (e.target.nodeName == "BUTTON") {
		const p = e.target.closest("div").querySelector("p");
		const select1 = e.target
			.closest("div")
			.querySelector(".select-1")
			.value.trim();
		const select2 = e.target
			.closest("div")
			.querySelector(".select-2")
			.value.trim();
		const inputs = document.querySelectorAll("#rejestry input");
		switch (e.target.className) {
			case "MOV-button": {
				console.log(select1, select2);

				p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[1].value}`;
				break;
			}
			case "XCHG-button": {
				console.log(select1, select2);

				p.textContent = `${select1} ${this.input} ${select2} null`;
				break;
			}
		}
	}
}

myślaem nad switch z 16 casami i pytanie dotyczy czy jest lepsze rozwiazanie niż dla każdego przypadku rozpatrywać warunek

Answer 1

if (select1[0] && select2[0]) {
		console.log(select1, select2);
		p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[0].value}`;
	}
	if (select1[0] && select2[1]) {
		console.log(select1, select2);
		p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[1].value}`;
	}
	if (select1[0] && select2[2]) {
		console.log(select1, select2);
		p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[2].value}`;
	}
	if (select1[0] && select2[3]) {
		console.log(select1, select2);
		p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[3].value}`;
	}

z if-ami wygladało by tak czy jest jakaś możliwść skrócenia tego poniważ kod zajumje 50 linijek

Comments

// i<10 przykładowy zakres ustaw jaki wymagasz
for (let i=0; i<10; i++)
    for (let j=0; j<4; j++)
        if (select1[i] && select2[j]) {
            console.log(select1, select2);
            p.textContent = `Rejestr ${select1}:${inputs[0].value} , Rejestr ${select2}:${inputs[j].value}`;
        }