1 opdracht per regel

Door Ozzie PHP op 20-03-2014 14:27 gewijzigd op 20-03-2014 14:40

3.955 views

Hey guys,

Bestaat er binnen het programmeren eigenlijk zoiets als een "1 opdracht per regel" principe?

Wat ik bedoel is dit. Je kan iets in 1 regel schrijven (wat ik zelf wel eens doe):

<?php
return !empty($_SERVER['HTTP_ACCEPT_ENCODING']) ? explode(',', str_replace([' ', ';'], ['', ','], $_SERVER['HTTP_ACCEPT_ENCODING'])) : [];
?>
Maar je kan ook 1 opdracht per regel doen:

<?php
if (empty($_SERVER['HTTP_ACCEPT_ENCODING'])) return [];
$_SERVER['HTTP_ACCEPT_ENCODING'] = str_replace([' ', ';'], ['', ','], $_SERVER['HTTP_ACCEPT_ENCODING']);
$_SERVER['HTTP_ACCEPT_ENCODING'] = explode(',', $_SERVER['HTTP_ACCEPT_ENCODING']);
return $_SERVER['HTTP_ACCEPT_ENCODING'];
?>
Als je 1 opdracht per regel doet, is het beter leesbaar... en nu vroeg ik me dus af of er een soort "regel/principe" bestaat waarbij je slechts 1 opdracht per regel uitvoert. Iemand die dat weet?

Ger van Steenderen

20-03-2014 19:22

Die query's schrijf je waarschijnlijk niet in PHP, nietwaar?
Nee in kladblok, en dan kopieer ik ze over naar Netbeans .... geloof je het?

Wat ik bedoelde met mijn vraag is:
Is het letterlijk één regel of is het één opdrachtregel (i.e tot aan ;)

Ward van der Put

20-03-2014 19:33

Wat is dan nog de definitie van één regel?

PSR-2 gaat uit van LF, maar je wilt zo'n ding niet in Kladblok openen.

Ozzie PHP

20-03-2014 19:37

@Ger... ik had het over 1 opdracht per regel. Op de eerste regel een str_replace, op de volgende regel een explode en op de laatste een return. Dus 1 "opdracht" per regel, en niet zowel de str-replace, explode en return allemaal op 1 regel (achter elkaar).

Wouter J

20-03-2014 22:37

maar wat ger bedoelt, is dit ook 1 regel?
<?php
return !empty($_SERVER['HTTP_ACCEPT_ENCODING'])
? explode(
',',
str_replace([' ', ';'], ['', ','], $_SERVER['HTTP_ACCEPT_ENCODING'])
)
: [];
?>

Ozzie PHP

20-03-2014 22:43

>> maar wat ger bedoelt, is dit ook 1 regel?

"Vroeger" zou ik zoiets op 1 regel knallen. Nu zou ik zeggen dat je dit beter kunt verspreiden over meerdere regels met slecht 1 "opdracht" per regel. Dus dit:

<?php
if (empty($_SERVER['HTTP_ACCEPT_ENCODING']) return [];
$encoding = str_replace([' ', ';'], ['', ','], $_SERVER['HTTP_ACCEPT_ENCODING']);
$encoding = explode(',', encoding);
return $encoding;
?>

Wouter J

20-03-2014 22:46

Je begrijpt dat je nog steeds geen antwoord geeft? Bij de vraag hoort een "ja" of "nee" antwoord, geen "Vroeger, toen ik nog in het hoekhuis in lutjebroek woonde, ..." :)

Ozzie PHP

20-03-2014 23:49

In dat geval "nee".

Reageren

Inloggen om te reageren