Wiam,

Czy pisząc klasę bardziej zaawansowaną, złożona z konstruktorów, funkcji w dużej ilości (nie jakieś 5 linijek kodu), powinienem w funkcjach ograniczyć się tylko do zwracania określonych wartości, pomijając np echo (przeglądałem sporo róznych kodów zaawansowanych i tam programiści zwracają jedynie jakieś wartości, wgl nie używają w klasach "pisania" tekstu metodą echo

To już zależy od tego, do czego dana klasa ma służyć, ale dla własnej wygody, lepiej jak metoda zwróci string, który następnie wyświetlisz. Np.

[PHP] pobierz, plaintext 
class View {
  public function body() {
    return '<div>body</div>';
  }
}
 
echo View::body();
[PHP] pobierz, plaintext 

Raczej powinno sie unikac echo bo tak ograniczasz swoja klase. Klasy w duzej mierze poto sa aby ulatwiac Tobie prace. A budowanie czegos na echo sprawia ze w 95% staje sie ta klasa bezuzyteczna bo rownie dobrze wtedy moglbys jej wgl nie tworzyc. Dopiero zwracanie danych daje Ci mozlowosc operacji na nich. Wiec echo jak najmniej

Tak właśnie myślałem i sie sprawa komplikuje bo już takie łatwe nie będzie: echo i po sprawie

Myślałem jak to zrobić ale bez echa to nie da rady chyba,

mam klasę query która zapytanie do bazy wykonuje, i tak kod muszę stworzyć żeby w pliku index.php jedynie napisać np:

[PHP] pobierz, plaintext 
$q = new query;
$q->query($zapytanie);
[PHP] pobierz, plaintext 

I w zależności czy jakiś błąd wystąpi w funkcji query błąd się na ekran wypisać ma ale skoro w klasie nie mam echo i tutaj nie mam echo to nie da się tego rozwiązać przecież, a za każdym razem sprawdzać czy zapytanie się powiodło w index.php to wg mnie co najmniej głupota, to ma robić za mnie klasa i wyświetlać błędy

Przechytywaniem błędów (czyli wyjątków) ogólnych powinna zając się aplikacja. Akurat query zwraca zazwyczaj zbiór wyników lub nic więc jest najprostszym przykładem. Później w widoku iterujesz tylko po otrzymanych wynikach.

pseudokod:

[PHP] pobierz, plaintext 
try {
  $db->beginTransaction();
  result = $db->query();
  $db->commit();
  //jakieś dodatkowe warunki jak trzeba if (result) costam
  View::render(['wyniki' => $result]);
} catch (EException $e) {
  $db->rollback();
  Logger::error('błąd zapytania');
  View::render('jakiś błąd');
}
[PHP] pobierz, plaintext 

Troche ciezkie to dla mnie nie ukrywam, wyjątki znam i je stosuje ale do czego w twoim kodzie mialy by sluzyc poszczególne funkcje?

@EDIT

Tutaj korzystałeś z PDO?

To jest pseudokod, ale PDO się nada do takiego użycia.

Widać, że to tylko przykładowa klasa do bazy danych, nie konkretnie PDO ani nic innego.

Ludzie powinni nauczyć się słowa pseudokod bo potem dostają funkcję do_something_or_nothing (arg) i kopiują zywcem

@up a gdzie autor napisał że coś przekopiował.

Fred1485

No ale nikt nie powiedział że w miejscu gdzie operujesz na zwróconych danych nie może być echo.
W tym twoim przykładzie zwracasz sobie dane z zapytania do bazy a potem już na nich operujesz czyli robisz jakiegoś foreach czy cokolwiek.
Przykład:
http://forum.php.pl/index.php?s=&showt...t&p=1151267

Zabiorę się lepiej za PDO, na pewno na darmo nie pójdzie a zobaczę różnice

Owszem warto poznać i mysqli i pdo ale wiele rzeczy jest tam podobnych do siebie

com, a widzisz w poście wikinga, informację, że to pseudokod? więc nie wiem po co pytanie czy to pdo czy cokolwiek innego...

Ale korzystając z PDO chcący stworzyć własną klasę obsługującą to wszystko, a potem w pliku dawać przykładowe

[PHP] pobierz, plaintext 
$a = new klasa_pdo;
 
$a->query_database($jakies_zapytanie)
[PHP] pobierz, plaintext 

chcąć korzystać z preparowania danych ciężko to będzie zrobić?

Po pierwsze obudowanie PDO to trochę jest bez sensu bo co to niby daje, ale jak przekażesz sobie array ze zmiennymi to co za problem to zbindować z niego

I za każdym razem jak zapytanie chcę zrobić to powtarzać to?

[PHP] pobierz, plaintext 
$stmt = $database->prepare('SELECT * FROM settings WHERE id = :id');
$stmt->bindValue(':id', 1, PDO::PARAM_INT);
$stmt->execute();
$row = $stmt->fetch();
[PHP] pobierz, plaintext 

dla mnie bez sensu

to sobie to zamykasz w funkcji i wywołujesz tylko funkcje

Ale i tak będę to musial przystosować do roznych zapytan z preparowaniem. Dobra czas do roboty w praktyce wyjdzie może

Próbowałem to jakoś zrobić, żeby sobie stworzyć funkcję i nie pisać za każdym razem:

[PHP] pobierz, plaintext 
    $stmt = $database->prepare('SELECT * FROM settings WHERE id = :id');
    $stmt->bindValue(':id', 1, PDO::PARAM_INT);
    $stmt->execute();
    $row = $stmt->fetch();
[PHP] pobierz, plaintext 

W postaci tablic odpowiednie wartości przesyłałem, np:

[PHP] pobierz, plaintext 
$connection->query(array('query' => 'SELECT * FROM settings WHERE id = :id', 'values' => array(':id' => $id)));
[PHP] pobierz, plaintext 

Chcąc dodać jeszcze typ zmiennej musiałbym 3-wymiarową tablicę zrobić
To się stanie jeszcze bardziej skomplikowane niż "normalne" wywoływanie.

Czy pisząc w PDO za każdym razem programiści taki

[PHP] pobierz, plaintext 
    $stmt = $database->prepare('SELECT * FROM settings WHERE id = :id');
    $stmt->bindValue(':id', 1, PDO::PARAM_INT);
    $stmt->execute();
    $row = $stmt->fetch();
[PHP] pobierz, plaintext 

kod wykonują z każdym zapytaniem? chyba nie..

Jeżeli piszesz czystym PDO to tak się pisze. Co innego jeżeli korzystasz z ORM

Są w necie dostępne jakieś CMS'y w oparciu o PDO?

[PHP] pobierz, plaintext 
    $stmt = $database->prepare('SELECT * FROM settings WHERE id = :id');
    $stmt->bindValue(':id', 1, PDO::PARAM_INT);
    $stmt->execute();
    $row = $stmt->fetch();
[PHP] pobierz, plaintext 

Jeżeli wstawiasz wartośc w zapytaniu na sztywno, która nie pochodzi z $_GET/$_POST itd. to możesz uzyć skroconej wersji bez bindowania.

Czyli nie ma żadnego sensu opakowywać tego w klasę albo jakąś funkcje jak wyżej koledzy napisali?

Zależy... Co w ogóle widzisz złego w czystym PDO? Jeżeli chcesz, możesz napisać do tego nakładkę

No nie nie. Nie nam nic do czystego pdo alr chciałem ten problem z długością zapytania rozwiązać ale na jedno by mi wyszło

Oj nie kolego bindowanie ma chronić przed błedem programistów jak i celowym działaniem osób 3

Czyli bindowanie w KAŻDYM zapytaniu?

Niekoniecznie. Zależy od człowieka i sytuacji

no ale nie ma ludzi nieomylnych, można pominąć tylko wtedy nie gwarantujesz że skrypt jest bezpieczny

Dlatego własnie mysql stał się deprecated a w 7 ostatecznie zostanie usunięty, bo nie było bindowania

Przykładowo chcesz pobrać główne kategorie
'SELECT * FROM categories where parent_id = 1'
co tu bindować? Chyba tylko idiota zrobiłby tu błąd i nie zauważył w testach

ok ale nie mówię już o takim niski poziomie, bo zapytania jak powyżej zdarzają się raz na parę tysięcy a może nawet rzadziej tu masz rację tego można nie zbindować i nic się nie stanie, ale kiedy przekazujemy coś ze zmiennej a nie tak, choc ta zmienna wcale nie musi pochodzić z GET/POST to już wtedy jednak zaleca sie bindować

Aa no widzisz, mówię tylko o tego typu zapytaniach, nie innych

Przerywając wasze ciekawe dyskusje

[PHP] pobierz, plaintext 
public function query($query = FALSE) {
 
        try {
 
            $this->query = $query;
            if($this->connection == FALSE || $this->query == FALSE) {
 
                throw new Exception('DO DOPRR', 1);  //do zrobienia !!!
            }
            $this->stmt = $this->connection->prepare($this->query);  
        }
        catch(PDOException $e) {
 
            $this->error(array('text' => $e->getMessage(), 'code' => $e->getCode()));
        }
        catch(Exception $e) {
 
            echo $e->getMessage();
        }
    }
[PHP] pobierz, plaintext 

Tworzę nakładkę na PDO bo chcę trochę OOP zasysnąć
W konstruktorze tworzę połączenie z bazą, czy jest jaki sens sprawdzać czy jest połączenie skoro tworząc nową instancję tej klasy w konstruktorze, wyjątek zostaje przechwycony i zostaje wyświetlony stosown6y komunikat + die();