String-Manipulation in javascript

[Cheatah]

Hi,

Geht das nicht kürzer als so:

mittels substr() bzw. substring() geht es in einer Zeile - wobei ich zwei empfehle, da Du das Ergebnis von lastIndexOf() mehrmals brauchst und somit zwischenspeichern solltest. Hingegen brauchst Du slice() IMHO gar nicht.

Einige Tipps noch zum Coding Style, wie es so schön auf Neudeutsch heißt:

var Laenge = Sound.length;

Mit Großbuchstaben beginnende Bezeichner sollten für Klassen und Konstanten reserviert sein; letztere werden üblicherweise komplett groß geschrieben. Ferner empfiehlt es sich, mit den Bezeichnern in einer einzigen Sprache zu bleiben - ich empfehle das Englische. Dies ergäbe hier 'var length', was wiederum vermieden werden sollte, weil 'length' einfach zu üblich ist. Die Folge wäre hier ein 'var sndLength', auch in Anlehnung an Dein 'sndName'.

var Suche = Sound.lastIndexOf ("/");

Schließe die öffnende Funktionsklammer direkt an den Methodennamen an. Übernimm dies aber bitte nicht zu den Bedingungsklammern bei if, for, while u.ä.

var Pfad = Sound.slice (0, Suche + 1);

Leerzeichen lockern auf. Achte aber darauf, dass es nicht _zu_ locker wird. Dieser Code wirkt, als gäbe es drei Funktionsargumente.

wmpSound = Sound.slice (0, Suche + 1) + "wmp" + Sound.slice (Suche + 1, Laenge);

wmpSound = Sound.slice(0, Suche+1) + "wmp" + Sound.slice(Suche+1, Laenge);

Dies liest sich IMHO sehr viel flüssiger, man erkennt die Zugehörigkeiten mit einem Blick.

Cheatah