Witam stworzyłem sonde w metodzie proceduralnej i postanowiłem ją przenieść do OOP. Prosze was o ocenę wykonania tej sondy, bo szczerze mówiąc to nie potrafię zbytnio dostrzec zalet rozwiązania obiektowego, pewnie wszystko jest źle.

[PHP] pobierz, plaintext 
<?php 
 
class Sonda {
 
public $tablica = array();
 
public function __construct($odp,$cookie,$tablica) {
 
	$this->tablica = $tablica;
	$this->odp = $odp;
	$this->cookie = $cookie;
 
	$this->setCookie();
	$this->explodeArray();
	$this->addPointToBase();
	$this->showResults();
}
 
public function setCookie() {
	if (isset($this->odp)) 
	setcookie("sonda","1",time()+7*24*3600);
}
 
public function explodeArray() {
 
if ($this->cookie != "1" && !isset($this->odp)) {
	echo '<form action="" method="post"><ul id="sonda">';
	$array = $this->tablica;
    $explode_array = explode(",", $array);
 
	for ($i=0;$i<count($explode_array);$i++)
	echo '<li><label><input type="radio" name="odp" value="1" id="ankieta_0">'.$explode_array[$i].'</label></li>';
	echo '</ul><div><input name="submit" value="Głosuj" type="submit"></div></form>';
}}
public function addPointToBase() {
 
	if (isset($this->odp)) {
	include("config.php");
	$query = mysqli_query($con, 'SELECT odp'.$this->odp.' from sonda');
 
	@$result = mysqli_fetch_array($query);
	$wynik = $result["odp".$this->odp] + 1;
 
	$sql = mysqli_query($con, 'UPDATE sonda SET odp'.$this->odp.'='.$wynik.'');
}}
public function showResults() {
 
	if ($this->cookie == "1" || (isset($this->odp))) {
	include("config.php");
	$query = mysqli_query($con, 'select * from sonda');
	$result = mysqli_fetch_array($query);
 
	for($this->odp = 1; $this->odp<=6; $this->odp++) {
	$sum_votes = $sum_votes + $result['odp'.$this->odp];
}
	for ($this->odp = 1; $this->odp<=6; $this->odp++) {
	$procent[$this->odp] = round(($result['odp'.$this->odp]/$sum_votes)*100)."%";
}
	echo '<ul id="sonda">';
	$array = $this->tablica;
    $explode_array = explode(",", $array);
	for ($this->odp = 1; $this->odp<=6; $this->odp++) {
	echo '<li id="wyniksondy">'.$explode_array[$this->odp-1].": <b>".$procent[$this->odp]."</b> (".$result['odp'.$this->odp].")</li>";
}
	echo'</ul>';
 
	echo '<br>Ilość głosów: <b>'.$sum_votes.'</b>';
}}
}
 
?>
[PHP] pobierz, plaintext 

Nie generuj htmla w klasie .Nie wygłuszaj błędów, tym bardziej w fetch_array, staraj się nadawać nazwy metodom inne niż domyślnych funkcji czy instrukcji PHP, patrz przykład setCookie. Zastanawiam się po co masz 3 razy tę samą pętlę for.

Poczytaj np. to:
http://wortal.php.pl/phppl/Wortal/Artykuly...poczatkujacych2

@Turson - dzięki wielkie, zrealizowałem twoje uwagi, ale rada aby nie generować HTMLa jest dla mnie kompletnie niezrozumiała. To w takim razie jak mam stworzyć sondę w OOP gdy nie mogę stosować HTMLu?

@Pyton - dzięki, zabieram się za czytanie.

HTML możesz stosować, ale poza klasą. Poprzeglądaj jakieś przykładowe klasy to zobaczysz jak to wygląda.

Ocena: 0/5. Dlaczego? Ponieważ to nie jest OOP tylko kod proceduralny zapakowany w klasę i nie ma to nic wspólnego z obiektowością

Też tak uważam . A mógłbyś mi powiedzieć w takim ogólnym zarysie jak to mniej więcej powinno wyglądać. I jest w ogóle sens tworzyć sondę w OOP?

Po co pisać w OOP?
http://pl.wikipedia.org/wiki/Programowanie_obiektowe

Przykład systemu sondy w OOP? Nie podaję szczegołów kodu, ponieważ nie będę go pisał za Ciebie:

[PHP] pobierz, plaintext 
$survey = new Survey('Jak oceniasz stronę');
$survey->addAnswer(new Answer(1, 'Kiepsko'));
$survey->addAnswer(new Answer(2, 'Tak sobie'));
$survey->addAnswer(new Answer(3, 'Super'));
 
if (isset($_POST['answers'])) {
    $survey->setUserAnswer($_POST['answers']);
}
 
$printer = new HtmlPrinter();
echo $survey->display($printer);
[PHP] pobierz, plaintext