PHP code blok: variabele binnen string met dubbele quote's

[Alain]

PHP:

<?php

$substring = "substring";
$string = "Deze string heeft een substring: $substring";

?>

De laatste $substring zou ook blauw moeten zijn.

[ Voor 4% gewijzigd door Alain op 26-05-2008 21:51 ]

[Noork]

Is dat wel zo? Is deze syntax niet beter?

PHP:

$string = "Deze string heeft een substring: " . $substring;

[Alain]

Wat beter is, is een discussie voor in Programming

Een variabele binnen dubbele quote's wordt gezien als variabele en niet als string en dus zou ie blauw moeten zijn.

[Xander]

Noork schreef op maandag 26 mei 2008 @ 21:53:
Is dat wel zo? Is deze syntax niet beter?

Net wat je wilt, ik vind het beter.

AlainS' voorbeeld is ook gewoon correct PHP. Maar of dat betekent dat de highlighter het anders zou moeten doen? Op de PHP site is het niet anders...

[AW_Bos]

Ik vind eerder van niet. Want variabelen horen niet in een string.
En ook de higlighter in het PHP-progje wat ik gebruik thuis houdt voet bij stuk .

[BCC]

Qua leesbaarheid maakt het niets uit, dus welke is het snelst? Ik weet toevallig dat in Ruby de eerste variant vele malen sneller is... ff checken

2 Miljoen keer:
$substring = 'substring';
$string = "Deze string heeft een substring: $substring";

868 ms

2 Miljoen keer:
$substring = 'substring';
$string = 'Deze string heeft een substring: ' . $substring;

555 ms

Gebruik de tweede Dus de syntax checker heeft het voordeel van de twijfel

[ Voor 70% gewijzigd door BCC op 26-05-2008 22:14 ]

[McKaamos]

BCC schreef op maandag 26 mei 2008 @ 22:03:
Qua leesbaarheid maakt het niets uit, dus welke is het snelst? Ik weet toevallig dat in Ruby de eerste variant vele malen sneller is... ff checken

2 Miljoen keer:
$substring = 'substring';
$string = "Deze string heeft een substring: $substring";

868 ms

2 Miljoen keer:
$substring = 'substring';
$string = 'Deze string heeft een substring: ' . $substring;

555 ms

Gebruik de tweede Dus de syntax checker heeft het voordeel van de twijfel

Dat zal in PHP waarschijnlijk niet anders zijn. Je geeft er ook duidelijk mee aan dat er een string ge-concatenate wordt. De eerste variant pleurt de variabele in de string en dan moet de interpreter eerst gaan bekijken of het toevallig een variabele is, en dat kost extra tijd.

[BCC]

Ik zal mezelf even wat verduidelijken:

In Ruby is "iets #{blaat}" sneller dan "iets" + blaat
In PHP is "iets" . $blaat sneller dan "iets $blaat"

Maar verder is dit volledig offtopic En single of double quotes heeft geen invloed op de resultaten

[ Voor 15% gewijzigd door BCC op 26-05-2008 22:24 ]

[Alain]

De reden waarom ik dit meld, is omdat mijn eigen programmeertool dit wel blauw maakt. Op het forum wordt het gebruik van variabelen binnen dubbele quote's regelmatig gebruikt en dan lees je dus makkelijk over een variabele heen. Dat andere highlighters het niet doen is jammer, en voor korte stukjes code wel te doen. Bij langere stukken waarbij je soms op variabelen let, ga je niet elke string af voor eventuele variabelen. Daar is die highlighting voor bedoeld toch?

En zover ik weet staan de devvers van T.net altijd open voor verbeteringen.

[moto-moi]

Mjah, het gaat er niet om of het beter, het gaat erom dat het hoort, maar ik vraag me af of het wel zo simpel is om aan te passen dat hij bij double quotes de variablen moet verven. Het is tenslotte een highlighter, geen complete parser.

[crisp]

Punt is dat je dan ook de complexe syntax (met curly braces) moet gaan ondersteunen en dus, zoals moto-moi al zegt, de string-content al bijna aan het parsen bent in plaats van enkel simpel highlighten. Je zal dan eigenlijk ook moeten detecteren dat dit ongeldig is:

PHP:

$a = "tralala $foo['bar']";

[BCC]

Kun je ook niet gelijk de code uitvoeren en de output weergeven ?

[Alain]

Bij de highlighter die ik gebruik wordt: $<variabele> en $<variabele>[<integer>] als variabele gezien. Ik gebruik btw kate om te programmeren en daar zit geen parser in.

[Voutloos]

Het is gewoon (iig voor PHP) bij iedereen bekend als slechte code style. Als we dan toch code style zaken gaan meenemen, lijkt het corrigeren van indenting mij een stuk nuttiger.

Dat je nu die variabele niet ziet is gewoon het gedrag van >90% vd highlighters, en zelfs met jouw voorgestelde highlighting ziet het er stom uit. Dat zou gewoon een incentive moeten zijn om je aan het denken te zetten of de var niet buiten de string moet. Het niet highlighten is een feature voor betere code style.

[moto-moi]

Voutloos schreef op woensdag 28 mei 2008 @ 07:56:
Het is gewoon (iig voor PHP) bij iedereen bekend als slechte code style. Als we dan toch code style zaken gaan meenemen, lijkt het corrigeren van indenting mij een stuk nuttiger.

Het gaat er helemaal niet om of het netjes is, de taal akn eht, dus hoort het in theorie in de highlighter thuis probleem is alleen zoals ik al eerder zei, dat het nogal wat werk is om dat erin te bouwen.

[Voutloos]

Dat het bepaald niet triviaal is snap ik ook, maar ik zet gewoon graag het 'je code kan uberhaupt netter' argument in.

[Arjan90]

Het zit trouwens wel in phpDesigner, deze geeft hem geen aparte kleur maar maakt hem gewoon dikgedrukt volgens mij

Geshi heeft het er ook niet inzitten zie ik, toch wel jammer! Zeker als je het wel een keer gebruikt

[ Voor 10% gewijzigd door Arjan90 op 28-05-2008 11:58 ]