Hallo fischlak,
Jetzt sag mir mal bitte ganz objektiv, was daran verkehrt sein soll
Verschiedenes.
Zunächst mal wird Dir verborgen, dass der Inhalt der Variablen textContent modifiziert wird. Das gehört unbedingt in den Kommentar hinein. Denn ansonsten ist das eine böse Tretmine.
Sodann noch dreierlei:
- Der Funktionsname und der Kommentar lügen: der Text wird nicht decodiert, sondern codiert.
- Der Text wird nicht nur codiert, sondern auch noch von Anführungszeichen am Anfang und Ende befreit. Der angemessene Name für die Funktion könnte also
unquoteAndUrlEncodeSeparatorslauten, sofern Du ihre Aktivität beschreiben willst. Alternativ kann sie auch in fachlichen Begriffen benannt werden, d.h. der Name beschreibt, welchem Zweck diese Manipulation dient. Da ich den nicht kenne, kann ich dafür keinen Vorschlag machen. - Die Implementierung könnte ggf. durch Standardfunktionen ersetzt werden
Sobald die Funktion einen sprechenden und korrekten Namen hat, kannst Du sie auch in den Funktionszoo abschieben und faltungsfrei
textContent = unquoteAndUrlEncodeSeparators(textContent); // encode ,;
schreiben.
Je nach Datenlage brauchst Du das aber vielleicht auch gar nicht. Könnte
textContent = encodeURIComponent(textContent.trim('"'));
vielleicht auch den Job tun?
Rolf
--
sumpsi - posui - obstruxi
sumpsi - posui - obstruxi