Is dit omslachtig?
Ik ben mezelf 'functies' aan het leren en ik snap het al redelijk. Toch vraag ik me af of ik wel een goede start maak door dit zo te programmeren.
Oftewel: is deze manier omslachtig of niet?
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
<?PHP
function emoticons($bericht, $par)
{
$map = "smilies/";
$smilies = array(
":P"=>"tong",
":)"=>"glimlach");
$opmaak = array(
"[b]"=>"<b>",
"[/b]"=>"</b>");
if(preg_match("/\ba\b/i", $par))
{
foreach($smilies as $smile=>$plaatje)
{
$bericht = str_replace($smile, '<img src="'.$map.$plaatje.'.gif">', $bericht);
}
}
if(preg_match("/\bb\b/i", $par))
{
foreach($opmaak as $code=>$code2)
{
$bericht = str_replace($code, $code2, $bericht);
}
}
return $bericht;
}
?>
function emoticons($bericht, $par)
{
$map = "smilies/";
$smilies = array(
":P"=>"tong",
":)"=>"glimlach");
$opmaak = array(
"[b]"=>"<b>",
"[/b]"=>"</b>");
if(preg_match("/\ba\b/i", $par))
{
foreach($smilies as $smile=>$plaatje)
{
$bericht = str_replace($smile, '<img src="'.$map.$plaatje.'.gif">', $bericht);
}
}
if(preg_match("/\bb\b/i", $par))
{
foreach($opmaak as $code=>$code2)
{
$bericht = str_replace($code, $code2, $bericht);
}
}
return $bericht;
}
?>
<form action="" method="post">
<p>
<textarea name="bericht" cols="40" rows="5"></textarea>
</p>
<p>
<input name="submit" type="submit" value="Verstuur" />
</p>
</form>
Het is toch niet te veel code?
M.v.g,
Jeffrey
EDIT: ik zie nu dat het script niet werkt, a pakt hij wel, maar ab niet.
Gewijzigd op 01/01/1970 01:00:00 door Vdleije .
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Andere tip die ik ooit van een van m'n leraren heb gekregen, die ik stiekem al heb toegepast, is uitstellen. Iedere functie hoeft maar een heel basale berekening te doen, en gebruikt weer andere functies om dan weer de kleine details onder handen te nemen. Op die manier blijft je code heel begrijpelijk (het is redelijk duidelijk wat deze functie nu doet) en je kan gemakkelijk dingen aanpassen en hergebruiken. Een functie hoeft eigenlijk nooit groter te zijn dan ongeveer 20 regels. Als hij groter is, kan hij waarschijnlijk beter onderverdeeld worden in kleinere functies :)
Gewijzigd op 01/01/1970 01:00:00 door Jelmer -
Quote:
Als hij groter is, kan hij waarschijnlijk beter onderverdeeld worden in kleinere functies :)
En dat heeft niet echt effect op de laadtijd of berekentijd van het script?
Ik ga het proberen toe te passen. Ik snap je code wel, denk ik.
Het is niet merkbaar negatief op de laadtijd. Voordeel is wel dat je vrij gemakkelijk later trucjes als memorization (bewaren van de antwoorden van een functie, zodat je die niet opnieuw hoeft uit te rekenen) herimplementatie van sommige stukjes code kan doen. En het leest veel gemakkelijker, en dat scheelt jou ook weer tijd :)
Ik vind het er zelf redelijk netjes uitzien :)