Pagina: 1 2 volgende »
15/08/2012 08:40:47
Beste,
Wat vinden jullie mooier / hoe doen jullie dit?
Stel je wil een waarde returnen. Als het goed gaat dan wordt de waarde gereturned en als het fout gaat dan wordt false gereturned. Nu vraag ik me af wat mooier is. Zowel in het "if" en het "else" gedeelte een return zetten? Of de waarde toekennen aan een variabele en eenmaal die variabele returnen buiten het if/else statement?
Om het wat duidelijker te maken... zou jij dit doen:
Code (php)
1
2
3
4
5
6
7
8
<?php
// hier hebben we dus 2x een return
if (true) {
return 'mijn_waarde';
} else {
return false;
}
?>
of dit:
Code (php)
1
2
3
4
5
6
7
8
9
<?php
// hier hebben we 1x een return
if (true) {
$waarde = 'mijn_waarde';
} else {
$waarde = false;
}
return $waarde;
?>
Gewijzigd op 15/08/2012 08:41:38 door
Ozzie PHP
27/11/2024 23:26:23
15/08/2012 09:25:11
Ik gebruik zelf je 2de optie.
En in sommige meer uitgebreidere statements gebruik ik een switch om zo aan de hand van ee returnwaarde de juiste melding/tekst weer te geven.
15/08/2012 09:27:16
Als de functie vrij simpel is direct een return. Maar ik heb regelmatig dat de return langzaam wordt gevuld als het goed gaat, dan gebruik ik een variabele.
15/08/2012 09:41:15
Maar is het een meer "gebruikelijk" dan het andere? Is het bijvoorbeeld gebruikelijker om per functie maar 1 return te hebben? Of maakt het niet uit als je meerdere returns hebt?
15/08/2012 09:47:14
Als je je bedenkt dat er altijd maar 1 return kan plaatsvinden kan je altijd een default return aan het einde geven. Immers, komt het script daar, dan is er blijkbaar geen return mogelijk geweest en dus is de enige mogelijke return op dat punt dus false:
Code (php)
1
2
3
4
5
6
7
8
9
<?php
function getSomething( $value ){
if ( $value > 0 ){
return 'iets';
}
return false;
}
?>
Op deze manier kan je vanalles doen in je script, zonder voor alle mogelijke uitzonderingen apart false te moeten returnen. Als er iets niet klopt, wat dan ook, wordt er altijd false geretourneerd.
15/08/2012 09:48:56
Dat is afhankelijk van wat er allemaal in de function wordt gedaan.
Zomaar een verzinsel:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
function doeiets($var)
{
if($var == '') return false;
$var .= '.....';
switch($var)
{
case ...: $var .= '....';
break;
default: return false;
}
$var .= '.....';
return strlen($var);
}
?>
15/08/2012 09:56:12
Santhe, tuurlijk zijn er op alle standaarden uitzonderingen te verzinnen waarvoor de regel niet meer op gaat. Daarom ook de uitspraak 'uitzonderingen bevestigen de regel' ;-)
15/08/2012 10:17:57
SanThe, maar jouw manier zou je ook zoiets kunnen doen (klopt niet helemaal maar gaat om t idee):
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<?php
function doeiets($var)
{
if($var == '') $return_value = false;
$var .= '.....';
switch($var)
{
case ...: $var .= '....';
break;
default: $return_value = false;
}
$var .= '.....';
$return_value = strlen($var);
return $return_value;
}
?>
Maar de vraag is dus nog steeds of het netjes / duidelijk / overzichtelijk / wenselijk is om meerdere returns in 1 functie te hebben. Ik meen ooit eens gehoord te hebben dat het het mooiste is om maar 1 return per functie te hebben, maar ik vraag me af of dat zijn persoonlijke mening was of dat dat een soort van "stelregel" is.
15/08/2012 10:24:22
Volgens mij komt dat dan van iemand met een achtergrond in een andere taal. In bijvoorbeeld Delphi heb je geen return zoals in php, maar geef je de return waarde aan een variabele (met de naam result als ik me niet vergis) en de waarde van die variabele wordt getourneerd. Nadat je daar een waarde aan hebt gegeven gaat de executie echter gewoon door. De methode die ik boven gaf zou daar dus niet werken, want dan krijg je altijd false.
In php ken ik echter niet zo'n stelregel en ik zou geen reden weten waarom je geen twee returns in een functie zou moeten willen hebben.
15/08/2012 10:36:19
Oké, thanks. Dat zou kunnen. Bedankt voor de reacties tot zover!
Als iemand nog iets wil toevoegen hoor ik het wel...
15/08/2012 10:38:00
@Ozzie: Ik ga voor mijn voorbeeld en niet voor die van jou. Ik ben van mening dat, indien iets niet voldoet, er direct een return mag worden gegeven. En ook vind ik mijn methode duidelijker/overzichtelijker.
Tutorial mod15/08/2012 11:14:43
@Erwin,
In Delphi doe je het precies andersom:
Code (php)
1
2
3
4
5
6
7
<?php //voor de kleurtjes
function TfrmMain.DoeIets(aName: string):boolean;
begin
result := false;
if aName <> '' then result := true;
end;
?>
Je moet daar dus ook vooraf bepalen wat je terug gaat geven, als je in het voorbeeld van result een string zou maken krijg je een heerlijke foutmelding.
15/08/2012 11:29:51
@SanThe, jij zou het dus zo doen?
en niet zo?
Code (php)
1
2
3
4
5
6
7
8
<?php
if (true) {
$waarde = 'mijn_waarde';
} else {
$waarde = false;
}
return $waarde;
?>
15/08/2012 11:38:50
@Ger,
Precies, je kan daar niet een default waarde aan het einde geven, maar inderdaad wel aan het begin. Op zich dus dezelfde methode. In elk geval scheelt het je voor elke uitzondering een aparte waarde ergens aan toe te moeten kennen. En daarbij kan je inderdaad niet zomaar iets geven, het type is al bepaald. Zoals overigens voor alle variabelen geldt in Delphi.
Tutorial mod15/08/2012 11:58:10
Inderdaad, en niet alleen in Delphi (beter gezegd Pascal).
Ozzie, mijn voorkeur gaat ook uit naar de methode van SanThe.
15/08/2012 12:28:24
Ozzie PHP op 15/08/2012 11:29:51:
In dit voorbeeld zou ik de else weghalen:
15/08/2012 12:39:45
SanThe, waarom zou je de else weghalen?
15/08/2012 12:42:17
Ozzie PHP op 15/08/2012 12:39:45:SanThe, waarom zou je de else weghalen?
Die is nutteloos.
15/08/2012 12:42:53
Ik persoonlijk probeer mijn return altijd uit te stellen naar het einde toe. Ik doe dit voornamelijk om de leesbaarheid te vergroten. Ik weet nu namelijk direct waar mijn return staat.
Sinds dit jaar volg ik ook hoger onderwijs, en daar verplicht men zelfs om de return op het einde te plaatsen. Wellicht om dezelfde reden: leesbaarheid.
15/08/2012 12:46:17
- SanThe - op 15/08/2012 12:42:17:Ozzie PHP op 15/08/2012 12:39:45:SanThe, waarom zou je de else weghalen?
Die is nutteloos.
Maar het is toch een if else? Als niet dit... dan dat?
(Ik zie het wel vaker overigens hoor, maar vraag me af wat het nut precies is.)
15/08/2012 12:56:15
- SanThe - op 15/08/2012 12:28:24:In dit voorbeeld zou ik de else weghalen:
En dat is precies wat ik dus eerder al had.....
Ozzie, je kan de else weglaten, omdat als je de if in gaat, de executie er nooit meer uit zal komen. De return verhindert dat. Dat is wat ik bedoelde met het punt dat je de 'default' return aan het einde kan zetten. Is er geen eerdere return geweest, dan heb je de default nodig.
Gewijzigd op 15/08/2012 12:56:44 door
Erwin H
