Habe gerade einen Knoten im Kopf. JavaScript string mit Quotes

Thallius · 13. August 2013, 16:35

Hi,

irgendwie weiß ich gerade nicht wie ich das Problem lösen soll.

Ich habe ein html input feld in das ich einen Text eingeben kann. Natürlich kann ich da auch Hochkommas und Anführungszeichen eingeben. Wenn ich das Formular dann abschicke baue ich mir in PHP einfach ein Array zusammen so in der Art

Quellcode

$params= array('id' => -1,'name1' => $_POST['myInputField'], ...

Das gebe ich dann an meine PHP Klase welches das wiederum per PDO in eine mySQL Datenbank schreibt welche UTF8 kodiert ist. Soweit so gut. Dort steht dann auch das richtige drin.

Nun muss ich aber das ja auch wieder auslesen und in eine javascript variable laden um damit weiter zu arbeiten. Also hole ich mir die Daten aus meiner PHP Klasse und habe dann den String in einer php variablen. Wenn ich jetzt aber versuche diesen Inhalt an eine javascript variable zu übergeben, dann habe ich verloren. Weil da ja dummerweise das " oder ' drin ist.

Quellcode

echo "var a="".$meineVar.'";";

geht also nicht.

Generell kann javascript aber ja strings mit Hochkomma. Wenn ich schreibe (Der einfacheit halber mal mit jquery syntax ansonsten oldscool dann mit document.getElementById)

Quellcode

var name=$('#meinTextFeld').val();

dann kann ich das wunderbar mit alert() ausgeben und es stehen die ' und " drin.

Wo ist denn nun mein Denkfehler?

Gruß

Claus

zerm · 13. August 2013, 16:52

Warum nicht als JSON bereitstellen und mit JavaScript abholen, wie man das normalerweise macht? Oh, warte, PHP, da ergibt ja nichts sinn

OK, dann so (brrr, entschuldige, wenn ich PHP fehler mache)

PHP-Quellcode

<script type="text/x-data" id="meineVar"> <?php echo $meineVar; ?> </script>

und dann einfach mit jQuery den inhalt von dem script holen.

EDIT: Durch den type "text/x-data" interpretiert niemand (sprich, nicht der Browser) das Script-Tag. Script-Tags werden auch nicht angezeigt. So macht man es üblicherweise, wenn man grössere Datenmengen an JS übergeben will, und aus irgendwelchen Gründen JSON ned will
EDIT2: Du solltest Sicherstellen, dass $meineVar kein "</script>" enthält. Ach halt, PHP. Da ignorier das, sonst kommst Du nicht auf genügend Schwachstellen in Deinem Code

little_pixel · 13. August 2013, 17:23

Hallo Claus,

Quellcode

echo "var a="".$meineVar.'";";

geht also nicht.

ich habe mit dem Zeugs nicht viel Ahnung, aber da ist sicherlich ein Fehler enthalten.
Ich sehe ein ' am Ende, aber keines am Anfang, d.h. die Syntax ist sicherlich falsch.

echo ( "var a=".$meineVar );

Das funktioniert nicht?

Viele Grüße

BlueM · 13. August 2013, 17:23

Also langer Rede kurzer Sinn: Du hast einfach ein Problem mit Hochkomma-Escaping in JavaScript. Wie in so ziemlich allen Sprachen, die syntaktisch C-artig sind, kannst Du ein Hochkomma mit Backslash escapen. Das kannst Du entweder zu Fuß machen oder in PHP mit addslashes() – wobei dann auch Zollzeichen escaped würden, und das ist vermutlich nicht das, was Du willst, sofern Du Hochkomma als String-Delimiter verwendest.

Carsten

Thallius · 13. August 2013, 17:29

Hi Zerm,

danke für den Tip. Ich probiere gerade schon die ganze Zeit mit json rum aber irgendwie klappt das auch nicht wie ich das will. Echt zum kotzen ist das.

Gruß

Claus

Thallius · 13. August 2013, 17:36

little_pixel schrieb:

Hallo Claus,

echo ( "var a=".$meineVar );

Das funktioniert nicht?

Viele Grüße

Hi Little,

nein das kann es ja nicht. Das würde ja dann ein

var a=test;

ergeben wenn in $meineVar test drin steht. Es muss aber var a="test"; sein. Steht nun aber ein " in $meineVar, dann ergibt

var a=\"".$meineVar."\";

natuerlich dann

var a=""";

was für den Interpreter ein fehler ist. Das Ganze kannst du jetzt mit " und ' beliebig kombinieren und kommst trotzdem auf keinen grünen Zweig.

Gruß

Claus

Günther · 13. August 2013, 17:45

Warum musst du soetwas eigentlich machen? Das ist eigentlich etwas unsauber, die Daten in so ein "dynamisch generiertes Javascript" reinzuschreiben, irgendwann verwirrt das, wenn es das in großen Projekten mehrmals gibt, weil man nicht mehr weiß, was statisch aus dem Script kommt, und was dynamisch geladen wurde.

Ich würde die Daten lieber per data- Attribut übertragen, im passenden DOM-Element.

gritsch · 13. August 2013, 17:49

warum nicht einfach var x = unescape(<?=escape($phpvar)?>);

Günther · 13. August 2013, 17:50

Ich wäre sonst für var x = <?php echo json_encode($x); ?>;

Das geht aber nur mit UTF-8, aber klappt dann immer...

Thallius · 13. August 2013, 17:55

Günther schrieb:

Warum musst du soetwas eigentlich machen? Das ist eigentlich etwas unsauber, die Daten in so ein "dynamisch generiertes Javascript" reinzuschreiben, irgendwann verwirrt das, wenn es das in großen Projekten mehrmals gibt, weil man nicht mehr weiß, was statisch aus dem Script kommt, und was dynamisch geladen wurde.

Ich würde die Daten lieber per data- Attribut übertragen, im passenden DOM-Element.

Kannst du mit dazu mal einen link geben das ich weiß was du damit meinst?

Oder irgendwie sonst kurz erklären?

Danke

Claus

zerm · 13. August 2013, 17:56

Er meint das, was ich fünf Posts weiter oben schon vorgeschlagen hatte. Nur schlechter.

Thallius · 13. August 2013, 18:02

zerm schrieb:

Er meint das, was ich fünf Posts weiter oben schon vorgeschlagen hatte. Nur schlechter.

Hm ich kapier das aber nicht

PHP-Quellcode

<script type="text/x-data" id="meineVar"> <?php echo $meineVar; ?> </script>

nehmen wir mal an in $meineVar wäre jetzt nur ein " drin. Dann würde aus Deiner Zeile

Quellcode

<script type="text/x-data" id="meineVar"> <"> </script>

und das macht keine Probleme?

Gruß

Claus

BlueM · 13. August 2013, 18:05

Ich bin zunehmend konsterniert. Die Antwort auf die Frage ist trivial, aber es entstehen Diskussionen, als ginge es um wasweißichwas.

Da meine abstrakte Beschreibung weiter oben offenbar nicht ankam, hier also nochmal zum Mitschreiben:

PHP-Quellcode

// Falls man " als JavaScript-String-Delimiter haben möchte
printf(
'var jsVariable = "%s";' . "\n",
str_replace('"', '\"', $myVariable)
);
// oder falls man ' als JavaScript-String-Delimiter haben möchte
printf(
"var jsVariable = '%s';\n",
str_replace("'", "\\'", $myVariable)
);

Alles anzeigen

Fertig. Inwiefern andere Ansätze stilistisch eleganter oder wartbarer sind, steht auf einem anderen Blatt (und ich würde das ohne Kontext nicht beurteilen wollen), aber zumindest ist das die Antwort auf das ursprüngliche Problem.

BlueM · 13. August 2013, 18:09

gritsch schrieb:

warum nicht einfach var x = unescape(<?=escape($phpvar)?>);

Weil a) es keine native PHP-Funktion escape() gibt und b) es nichts zu unescapen gibt, da es lediglich um ein banales JavaScript-String-Literal geht.

Thallius · 13. August 2013, 18:16

Ich habe es jetzt so gelöst, dass ich mir ein Objekt erzeuge und alle Werte dort hin schreibe. Ich denke das dürfte die suaberste Lösung sen.

Danke an alle

Gruß

Claus

zerm · 13. August 2013, 18:40

Thallius schrieb:

und das macht keine Probleme?

Nein. Das ist der Sinn. Macht man etwa häufig so für JavaScript Template, also etwa wenn man moderne Webentwicklung macht (sprich, nicht PHP).

Ich glaub, ich hab genug PHP-bashing in diesem Thread betrieben, ich verspreche, damit aufzuhören ;)

zerm · 13. August 2013, 18:44

BlueM schrieb:

Fertig

Cool. Was ist, wenn $myVariable einen Backslash enthält?

Sorry, fällt mir schwer mich in diese PHP-ich-hack-alles-irgendwie-zurecht reinzufinden. Stimmt. Keine grosse Diskussion um potentiell fragwürdige Daten, die auf der Webseite als JavaScript, also ausführbar, eingebettet werden. Dann würde man ja sonst auf die Idee kommen, andere Werkzeuge als PHP lieben zu lernen.

Thallius · 13. August 2013, 18:54

zerm schrieb:

BlueM schrieb:

Fertig

Cool. Was ist, wenn $myVariable einen Backslash enthält?

Sorry, fällt mir schwer mich in diese PHP-ich-hack-alles-irgendwie-zurecht reinzufinden. Stimmt. Keine grosse Diskussion um potentiell fragwürdige Daten, die auf der Webseite als JavaScript, also ausführbar, eingebettet werden. Dann würde man ja sonst auf die Idee kommen, andere Werkzeuge als PHP lieben zu lernen.

Backslash geht übrigens auch mit json nicht.

Gruß

Claus

BlueM · 13. August 2013, 19:04

zerm schrieb:

BlueM schrieb:

Fertig

Cool. Was ist, wenn $myVariable einen Backslash enthält?

Korrekter Einwand. Kann man mal sehen, wie mich der Thread bereits verwirrt hat

Wenngleich es am prinzipiellen Sachverhalt nichts ändert, sprich: einfach das vergessene zusätzliche Escapen des Backslashes ergänzen.

BTW: Von mir aus kannst Du PHP soviel bashen wie Du willst, ebenso wie jede andere Technologie. Aber da eigentlich nichts an diesem Thread PHP-spezifisch ist (die Frage hätte man sinngemäß genauso mit Java, Python, Groovy, Node.js o.ä. stellen können), wirkt Dein Herumreiten darauf etwas zwanghaft.

Carsten

gritsch · 13. August 2013, 19:14

BlueM schrieb:

gritsch schrieb:

warum nicht einfach var x = unescape(<?=escape($phpvar)?>);

Weil a) es keine native PHP-Funktion escape() gibt und b) es nichts zu unescapen gibt, da es lediglich um ein banales JavaScript-String-Literal geht.

ja ist klar, hat nur an der tür geklingelt. korrekt wäre die urlencode() gewesen.
dann hat man den vorteil dass man kein problem mit irgendwelchen sonderzeichen hat (oder kennst du die encoding des files und wie es dann vom browser gehandhabt wird etc) und eben auch um das problem mit quotes, single-quotes, backslashes, newlines, tabs etc zu umgehen.