Reacties op het script Faculteit
|
Gepost op: 20 mei 2006 - 11:57 |
|
|
|

PHP expert
|
Leuk script! Er zit wel 1 kleine redeneringsfout in (in de zin van programmeren, niet op wiskundig niveau). Die breaks hoeven er namelijk helemaal niet te staan. Het is wss macht der gewoonte dat je ze er zet. Probeer het maar eens zonder die breaks, dan werkt het evengoed. |
|
|
|
Gepost op: 20 mei 2006 - 12:06 |
|
|
|
HTML interesse
|
Ik zie het bedankt voor de tip  |
|
|
|
Gepost op: 20 mei 2006 - 16:41 |
|
|
|
HTML interesse
|
Idd leuk script en handig.
1. Foutje.
Citaat: stel je wil de faculteit van 5, dan doen je: 1 x 2 x 3 x 4 x 5 = 120
Goede manier.
Citaat: stel je wil de faculteit van 5, dan doe je: 1 x 2 x 3 x 4 x 5 = 120 |
|
|
|
Gepost op: 21 mei 2006 - 15:05 |
|
|
|
PHP ver gevorderde
|
|
|
|
Gepost op: 21 mei 2006 - 15:54 |
|
|
|
Crew algemeen
|
Het ziet er wel handig uit;-)
maar... met een for lus gaat het sneller:
<?php
function faculty ($n)
{
$r = $n--;
for (; $n >= 1; --$n)
{
$r *= $n;
}
return $r;
}
?>
<?php function faculty ($n) { $r = $n--; for (; $n >= 1; --$n) { $r *= $n; } return $r; } ?>
Recursieve functies zijn namelijk niet de snelste;-)
verschi:
met recursieve functie: float(0.000810861587524)
met for: float(0.000120878219604)
Valt dus best wel mee;-) |
|
|
|
Gepost op: 21 mei 2006 - 16:27 |
|
|
|
HTML interesse
|
Ik had het al in gedachten, en toen zocht ik op google en vond ik deze, en aangezien deze manier mij iets beter leek dan mijn eigen heb ik deze gebruikt in mijn script |
|
|
|
Gepost op: 21 mei 2006 - 17:50 |
|
|
|
PHP ver gevorderde
|
ik zei het je, ik beschuldig je van niets, maar miss had je toch een kleine vermelding moeten maken voor het eerste stuk van je script |
|
|
|
Gepost op: 21 mei 2006 - 18:07 |
|
|
|
HTML interesse
|
|
|
|
Gepost op: 21 mei 2006 - 22:45 |
|
|
|
PHP ver gevorderde
|
Citaat: echo "Te groot getal, hoger dan 170 kan ik helaas niet aan";
En waarom niet?  |
|
|
|
Gepost op: 21 mei 2006 - 22:48 |
|
|
|
HTML interesse
|
dan geeft ie als antwoord: INF |
|
|
|
Gepost op: 22 mei 2006 - 17:49 |
|
|
|
PHP ver gevorderde
|
Je checkt nergens of het wel degelijk een numeriek iets is, zou je dat niet doen? |
|
|
|
Gepost op: 22 mei 2006 - 17:55 |
|
|
|
HTML interesse
|
dat zou inderdaad kunnen en handig zijn, maar ik vertrouw er op dit moment op dat niemand letters of leestekens invoerd. Misschien dat ik dat nog wel toevoeg
EDIT
Speciaal voor de liefhebbers controleert hij nu of je wel een getal invoerd |
|
|
|
Gepost op: 23 mei 2006 - 07:59 |
|
|
|
Crew algemeen
|
|
|
|
Gepost op: 23 mei 2006 - 08:11 |
|
|
|
HTML interesse
|
Bedankt voor de tip... maar dat wat ik deed werkte toch ook gewoon? |
|
|
|
Gepost op: 23 mei 2006 - 18:31 |
|
|
|
Crew algemeen
|
ja het werkt wel gewoon, maar het is veel omslachtiger dan een (veel snellere) functie die ervoor bedoeld is;-) |
|
|
|
Gepost op: 23 mei 2006 - 19:26 |
|
|
|
HTML interesse
|
achja, bedankt, ik heb dr weer wat door geleerd en daar gaat het vgm om  |
|
|
|
Gepost op: 24 mei 2006 - 07:52 |
|
|
|

PHP expert
|
@JeXuS: volgens mij zie jij het bos door de bomen niet meer.
De for loop kent 3 argumenten. Ja er is een 2-argumenten, 1-argument en 0-argument variant. Maar waarom zou je niet gebruik maken van het eerste argument als je dit _toch_ nodig hebt.
Dit dient om een 'beginsituatie' aan te geven. Wat is je eerste stap in je script: een beginsituatie.
Preincrement is ook nergens voor nodig en het derde argument dient om een actie uit te voeren. Aangezien je maar 1 actie uitvoert in je for-loop kan je deze als derde argument meegeven.
function faculty ($n) {
for ($r = $n--; $n >= 1; $r *= $n--);
return $r;
}
function faculty ($n) { for ($r = $n--; $n >= 1; $r *= $n--); return $r; }
|
|
|
|
Gepost op: 24 mei 2006 - 08:14 |
|
|
|
Crew algemeen
|
denk je dat ik meteen een perfect script schreef? ik maakte het snel om een voorbeeld te geven zoals hoe t met een for loop kan.
maarja, aangezien jij toch altijd alles beter denkt te weten, waarom zou ik dan ooit nog commentaar ergens op geven? |
|
|
|
Gepost op: 24 mei 2006 - 10:33 |
|
|
|

PHP expert
|
Je moet niet direct alles persoonlijk opnemen wat ik zeg. Ik versta gewoon niet waarom je de standaard for-loop vervangt door een variant als dit nergens voor nodig is. Als jij zaken wil verbeteren heb ik hier toch net zo goed recht op?
Er is een verschil tussen kritiek en opbouwende commentaar. |
|
|
Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|