login  Naam:   Wachtwoord: 
Registreer je!
Scripts > PHP > GD library > [class] Grafiekjes


Reacties op het script [class] Grafiekjes

Offline  marten
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.

Offline  Rubdos
Gepost op: 15 februari 2009 - 11:26
Nieuw lid



Ow, oke, hoe dan, var vervangen door public en zo?

Offline  avdg
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)

Offline  Rubdos
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!

Offline  Rubdos
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

Offline  Wim
Gepost op: 17 februari 2009 - 18:31
Crew algemeen



  1. 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:
  1. $iValueCount = count($value);
  2. 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:
  1. $iValueCount = count($value);
  2. 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...
----

  1. $lowest += -1;

Waarom niet -=? 
  1. $lowest -= 1;


----

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!)

Offline  Rubdos
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

Offline  Wim
Gepost op: 20 februari 2009 - 20:40
Crew algemeen



die count buiten de for-loop is een betere tip 

Offline  Rubdos
Gepost op: 21 februari 2009 - 08:14
Nieuw lid



Ja, oké, geestig hoor!


Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.
 
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.067s