Javascript Syntaxprüfung mit JSLint, Unused variable usw.

[dedlfix]

Tach!

Unnecessary 'else' after disruption. Was ist an der Schreibweise mit dem "else" so schlimm?

if(anzahl === ""){
  alert('Mach was...');
  return false;
}else{
  //weiterer code hier.
}

Es ist nicht schlimm, sonst wäre es ein Fehler und nicht nur ein Hinweis vom Linter. Das else ist schlicht nicht nötig, wenn der if-Zweig mit return beendet wird. Der "weiterer code" muss nicht mehr ein einem eigenen else-Block stehen, weil es keinen Unterschied gibt zwischen Code im else und Code, der nach dem kompletten if-Statement kommt.

Dann meldet JSLint bei diesem Code:

Unused 'erledigt'.

var erledigt = "";
if(val_defekt == '1'){
   erledigt = "Anwender";
}
else{
  erledigt = "";
}

Was ist daran falsch?

In dem Ausschnitt ist nur eine Zuweisung zu einer lokalen Variable zu sehen, aber keine Nutzung derselben. Vielleicht ist das ein Tippfehler oder ein logischer, und du solltest das mal genauer untersuchen.

dedlfix.