From dd410812025d80ff4a2a56bc2ce855ff8ebf8cf9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juhani=20Krekel=C3=A4?= <juhani@krekelä.fi>
Date: Tue, 21 May 2024 14:25:30 +0300
Subject: [PATCH] Yksinkertaista elementtien poistamista

---
 käyttöliittymä.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/käyttöliittymä.js b/käyttöliittymä.js
index 36454a9..83aa5ba 100644
--- a/käyttöliittymä.js
+++ b/käyttöliittymä.js
@@ -63,8 +63,7 @@ function suoritaMuutos(tietokanta, muutos) {
 		luokatLista.insertBefore(luoLuokka(id, uusi), seuraava);
 	} else if (taulu === taulut.luokat && uusi === undefined) {
 		// Luokka poistettu
-		const luokka = document.getElementById(`luokka-${id}`);
-		luokka.parentElement.removeChild(luokka);
+		poistaElementti(document.getElementById(`luokka-${id}`));
 	// TODO: luokka muutos
 	} else if (taulu === taulut.opettajat && vanha === undefined) {
 		// Uusi opettaja
@@ -77,8 +76,7 @@ function suoritaMuutos(tietokanta, muutos) {
 		opettajatLista.insertBefore(luoOpettaja(id, uusi), seuraava);
 	} else if (taulu === taulut.opettajat && uusi === undefined) {
 		// Opettaja poistettu
-		const opettaja = document.getElementById(`opettaja-${id}`);
-		opettaja.parentElement.removeChild(opettaja);
+		poistaElementti(document.getElementById(`opettaja-${id}`));
 	// TODO: opettaja muutos
 	} else if (taulu === taulut.tilat && vanha === undefined) {
 		// Uusi tila
@@ -88,8 +86,7 @@ function suoritaMuutos(tietokanta, muutos) {
 		tilatLista.insertBefore(luoTila(id, uusi), seuraava);
 	} else if (taulu === taulut.tilat && uusi === undefined) {
 		// Tila poistettu
-		const tila = document.getElementById(`tila-${id}`);
-		tila.parentElement.removeChild(tila);
+		poistaElementti(document.getElementById(`tila-${id}`));
 	// TODO: tila muutos
 	} else {
 		throw new Error(`Ei toteutettu ${taulu} ${id} ${vanha} ${uusi}`);
@@ -106,6 +103,10 @@ function vertaa(a, b) {
 	return a.localeCompare(b);
 }
 
+function poistaElementti(elementti) {
+	elementti.parentElement.removeChild(elementti);
+}
+
 function luoLuokka(id, nimi) {
 	const li = document.createElement('li');
 	li.id = `luokka-${id}`;