Poprawność według OOP Opcje

[Bora]

z OOP jestem raczkujący napisałem klase okórej cześć zamieszczam poniżej. Znajdują sie tam 2 podobne funkcje, która z nich jest poprawna (chodzi o zapis zgodny z ideą OOP);
ps obie działąją poprawnie przy czym 2ga jest niewiele szybsza (testowane w pętli 1000 razy)

[php:1:2b899e2e72]<?php
class Db
{
function num_rows1($tabela,$pole,$warunek="") {
if ($warunek=="") {
$this->where = "";
} else {
$this->where = " WHERE ".$warunek;
}
$this->zapytanie = "SELECT COUNT(".$pole.") FROM ".$tabela.$this->where;
$this->rezultat = mysql_query($this->zapytanie);
return mysql_num_rows($this->rezultat);
}

function num_rows2($tabela,$pole,$warunek="") {
if ($warunek=="") {
$where = "";
} else {
$where = " WHERE ".$warunek;
}
$zapytanie = "SELECT COUNT(".$pole.") FROM ".$tabela.$where;
$rezultat = mysql_query($zapytanie);
return mysql_num_rows($rezultat);
}
}
?>[/php:1:2b899e2e72]

[Nalfein][WR]

Nie wiem czy śmiać się czy płakać (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) A cóż to za idea OOP? Po prostu chodzi o to, aby każdy obiekt był za COŚ odpowiedzialny, a nie za wszystko na raz. Nie można powiedzieć, że coś zaprojektowałeś "dobrze" lub "źle", gdy pokazujesz jedną klasę. Ważny jest po prostu podział obowiązków, a nie co jak zapiszesz. Tak więc, nie jest ważne czy w środku metody napiszesz $where czy $this->where. To jest Twoja sprawa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ważne są metody i właściwości, ich nazwy i listy argumentów. Powinny być jak najbardziej intuicyjne. Tutaj tak nie ma, bo jak można zliczać wiersze w bazie danych? "Num_rows" znaczy mniej więcej "liczba wierszy", a wiadomo, że wiersze są w tabeli, a nie w bazie danych. A twoje zapytanie zlicza ilość wierszy z niepustymi polami (takie które nie są NULL) spełniającymi jakiś warunek. Jako potencjalny użytkownik Twojej klasy mi odrzuciłbym ją od razu, bo piszemy w OOP po to, aby później ułatwić sobie używanie tego kodu, a nie przeglądać kod klasy, aby dojść do tego co twórca klasy mógł mieć na myśli tak nazywając swoją metodę.