Chart API

[Piczer]

Piszę mini-bibliotekę do własnego użytku, ale przy okazji uczę się zachowywać maksymalną czytelność kodu, dlatego zwracam się do Was z pytaniem, czy jesteście w stanie "ogarnąć" poniższe API bez dokumentacji?

 

Biblioteka służy do wyświetlania wykresów korzystając z biblioteki Google do wizualizacji (oraz w tym przypadku Google Analytics, które służą jako źródło danych).

 

<?php

AreaChartVisualizator::initializeLibrary();

$Views = new AreaChartVisualizator( "Liczba wejść" );
$Views->setChartPlaceOnDOM( "views" );
$Views->setAttribute( "width", 800 );
$Views->setAttribute( "height", 240 );
$Views->setAttribute( "legend", "bottom" );

$VisitsModel = new AreaChartModel();
$VisitsGoogleModel = new GoogleAnalyticsModel();
$VisitsModel->retrieveSource( $VisitsGoogleModel->authorize( "login", "pass" )->setTimeInterval( "2010-05-10", "2010-05-17" )->retrieveParameter( "visits" ) );

$BouncesModel = new AreaChartModel();
$BouncessGoogleModel = new GoogleAnalyticsModel();
$BouncesModel->retrieveSource( $VisitsGoogleModel->authorize( "login", "pass" )->setTimeInterval( "2010-05-10", "2010-05-17" )->retrieveParameter( "bounces" ) );

$Views->addModel( $VisitsModel, "Liczba wejść" );
$Views->addModel( $BouncesModel, "Liczba odbić" );
$Views->generateCode();

?>

 

Przykładowe wyniki:

[Crackkk-GM]

Obiektowo... Ja bym dał radę ;)

[Piczer]

Tylko ty?

[Kofel]

Ja nie, bo u mnie kiepsko z rosyjskim.

[Piczer]

Więc jakie proponujesz zmiany?

[Kofel]

$VisitsModel = new AreaChartModel();

$VisitsGoogleModel = new GoogleAnalyticsModel();

 

Trochę mnie to deprymuje. Nie dało by się to sprowadzić do jednej instancji?

[Piczer]

Można by to ukryć w fabryce, ale po co? Te dwa obiekty spełniają kompletnie różne zadania.

[Kofel]

Ja bym jednak zrezygnował z AreaChartModel >:D

[PsichiX]

tak czy inaczej, dokumentacja by się przydała - nie ma znaczenia czy obiektowo czy nie, czy po rusku, czy suahili - skądś tę funkcjonalność trzeba poznać i dokumentacja jest wymagana, bo nie każdy będzie chciał zaglądać w kod by sprawdzić co z czym się je :P

[Piczer]

Nie chodzi mi o to, czy dokumentacja jest, czy nie ma, tylko, żeby kod był na tyle czytelny, by do niej nie zaglądać.

 

Co do tematu, to można w sumie zrobić, że GoogleAnalyticsChartModel będzie dziedziczył po AreaChartModel...

[Kofel]

AreaChartVisualizator::initializeLibrary();

 

Nie powinienem się o takie rzeczy troszczyć :)

[Piczer]

To wkleja kod js dołączający bibliotekę. Jakieś lepsze pomysły?

[gnysek]

a nie może być to w konstruktorze wtedy, skoro zaraz i tak tworzysz obiekt tej klasy ?

[Piczer]

Przewiduję możliwość utworzenia wielu wykresów.

[Kofel]

static $lolkorfalol;

 

if( !$lolkorfalol ) $lolkoralol = true;

 

Chyba cuś takiego

[Piczer]

Nikt się nie kapnął, że ten sam obiekt loguje się 2 razy?

[Kofel]

Nie wytykałem tego, skoro sam mi o tym na GG napisałeś :P

[Piczer]

Oprócz Ciebie.