Rolf B: Auf einen bestimmten Inhalt prüfen?

Beitrag lesen

Hallo Bernd,

darf ich Dir noch einen Vorschlag machen? Folgende Codeblöcke tun das gleiche:

// Umständliche Version, schlecht lesbar wegen aussagelosem Variablennamen "h"
h = $("#test").text();  

if(h == 'Bitte wählen ....') {
	$( "#insert" ).prop("disabled", true);
} else {
	$( "#insert" ).prop("disabled", false);
}

Würdest Du jemandem folgende Erklärung geben? "Wenn Du eine rote Karte hast, lege die rote Karte in den Kasten. Wenn Du eine grüne Karte hast, lege die grüne Karte in den Kasten". Würdest du nicht eher sagen: "Lege deine Karte in den Kasten"? Die Karte ist das Ergebnis der Bedingung, der Kasten ist eine Variable oder der 2. Parameter für prop(). Also:

// Kompakter und besser lesbar wegen beschreibender Variablennamen
var selectedText = $("#text").text();
var isDefault = (selectedText == 'Bitte wählen ....');
$("#insert").prop("disabled", isDefault);
// Einzeiler ohne Temp-Variablen
$("#insert").prop("disabled", $("#text").text() == 'Bitte wählen ....');

Rolf

--
sumpsi - posui - clusi