Reacties op het script [class] Grafiekjes
|
Gepost op: 15 februari 2009 - 10:33 |
|
|
|
Beheerder
|
Misschien wil je in de volgende versie rekening houden met PHP 5? PHP 4 wordt namelijk niet meer ondersteund door PHP. |
|
|
|
Gepost op: 15 februari 2009 - 11:26 |
|
|
|
Nieuw lid
|
Ow, oke, hoe dan, var vervangen door public en zo? |
|
|
|
Gepost op: 15 februari 2009 - 15:36 |
|
|
|
PHP gevorderde
|
Misschien beter om je class point te vervangen door een 2-dimensionele array.
offtopic: ik vind dit wel een leuke script, misschien een uitbreiding met meerdere soorten grafieken, meerdere soorten gegevens in 1 tabel, ...
dan zou dit een echte grafiekclass worden voor een framework misschien
(oei, ik overdrijf nu waarschijnlijk )
@post hieronder: - typen en performance hebben totaal niets met elkaar te maken, dus ik zou eigenlijk best kijken wat het snelst werkt
- ik denk wel dat je best bij de uitbreiding toch meerdere classes kan gebruiken (maar dit wordt dan te ingewikkeld voor beginners, want je komt dan uit op frameworktoestanden - dus goed documenteren is de boodschap!)
maar als je toch kiest voor de oo-methode, laat de class tenminste het werk doen:
bv: $gr->addPoint(5, 'januari')->addPoint(4, 'februari')
dit lijkt mij mooier maar ik zou dan nog de oude manier behouden.
(lees dit eens) |
|
|
|
Gepost op: 15 februari 2009 - 16:38 |
|
|
|
Nieuw lid
|
Overdrijven, welnee, daar wilde ik net mee beginnen! ;) Die class point vond ik gemakkelijker. Geef toe, wat is korter:
new point(3005,"Janvier");
of
array("point"=>3005,"name"=>"Janvier");
? Toch leuk he, die oop?
Trouwens, dit is versie 1! Dat wil zeggen dat er wel uitbreidingen mogelijk zijn! |
|
|
|
Gepost op: 15 februari 2009 - 19:37 |
|
|
|
Nieuw lid
|
Oké, bezig met versie 2, bugs in 1 nog altijd te zeggen.
Op m'n server zie je de vooruitgang: http://scriptsofrubdos.byethost13.com
Verbeteringen:
-PHP 5 wordt gerespecteerd
-1e stap naar verschillende soorten grafieken (tot nu toe nog alleen verticalBarGraph)
-verschillende Pointsets |
|
|
|
Gepost op: 17 februari 2009 - 18:31 |
|
|
|
Crew algemeen
|
for($i=0;$i<count($value);$i++) $this->values[] = $value[$i];
for($i=0;$i<count($value);$i++) $this->values[] = $value[$i];
2 opmerkingen: Die count($value) wordt elke keer als de for-loop doorlopen wordt opnieuw uitgevoerd, onnodig verbruik van resources dus. Je kan dit oplossen door de variabele (tijdelijk) in een variabele te steken:
$iValueCount = count($value);
for($i=0;$i<$iValueCount;$i++) $this->values[] = $value[$i];
$iValueCount = count($value); for($i=0;$i<$iValueCount;$i++) $this->values[] = $value[$i];
Verder kan je je for loop ook verkort schrijven dmv de laatste parameter aan te passen:
$iValueCount = count($value);
for($i=0;$i<$iValueCount;$this->values[] = $value[$i++]);
$iValueCount = count($value); for($i=0;$i<$iValueCount;$this->values[] = $value[$i++]);
Dit laatste MOET niet! Persoonlijk vind ik het overzichtelijker, maar kan me best voorstellen dat jij vindt van niet...
----
Waarom niet -=?
----
Class: point
Waarom gebruik je hier een classe voor? is het niet handiger om met 1 gegevens array te werken? =>
array(
'Gegeven 1' => 'waarde 1',
'Gegeven 2' => 'waarde 2',
'Gegeven 3' => 'waarde 3',
'Gegeven 4' => 'waarde 4'
)
----
Voor de rest ziet je script er wel leuk uit! Een goede aanvulling aan de library!
Deze puntjes zijn trouwens maar details hoor! (onderschat echter de laadtijd van count() binnen een for-loop niet!) |
|
|
|
Gepost op: 18 februari 2009 - 20:47 |
|
|
|
Nieuw lid
|
Dat van die -= is een goede tip! Ook voor de volgende versie, ik ben gewoon gewoon om += te gebruiken.
Op m'n server heb ik het fontbestandje geüpload, dan zie je de ondertitels ook |
|
|
|
Gepost op: 20 februari 2009 - 20:40 |
|
|
|
Crew algemeen
|
die count buiten de for-loop is een betere tip |
|
|
|
Gepost op: 21 februari 2009 - 08:14 |
|
|
|
Nieuw lid
|
|
|
Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|