Benutzer:PerfektesChaos/js/ReferencesCreator

aus Wikipedia, der freien Enzyklopädie
Zur Navigation springen Zur Suche springen

Notizen zum Add-on für Firefox: wikipedia-references-creator

  • WSTM für ISBN (und ggf. mehr): WSTMwrc

Anmerkungen, Stand Ende Februar 2015

Zeile Bemerkung
3 var Evaluate = require("./Evaluate"); – warum sich selbst einbinden?
4 Timer verwendest du nicht (mehr).
56+ Einfügen: var result;
44 Use '===' to compare with "". Siehe #trim.
66 Missing semicolon
123 Zuweisung ohne var, das result wurde schon Zeile 66 verwendet.
170 Use '===' to compare with "".
191 Missing semicolon
198 Use '===' to compare with "".
205 Use '===' to compare with 'true'.
253 Unnecessary semicolon

Die trim-Funktion ginge einfacher zu schreiben:

function trim(s) 
{
    if (s)
    {
        return s.replace(/^\s+/, '').replace(/\s+$/, '');
    }
    return "";
}

Aber man könnte auch ganz darauf verzichten; bei neueren Browsern ist es eine normale Funktion wie auch eben .replace().

  • developer.mozilla.org
  • Wenn man Code für unbekannte Browser und Benutzer schreibt, ist es sinnvoll, .trim() noch nicht zu verwenden.
  • MediaWiki hängt eigentlich an das String-Objekt eine .trim() an, so dass es in jeder Online-Seite unter MediaWiki benutzt werden kann.
  • Du bist in deinem Add-On sicher, dass unter Firefox ausgeführt wird, und der FF 3.5 ist schon ein paar Jahre alt und die Benutzer wurden -zigmal zur Aktualisierung aufgefordert.