Avatar billede kviller Novice
12. oktober 2013 - 22:51 Der er 7 kommentarer og
1 løsning

Strukturering med PDO / MySql / php

Jeg er lidt ny inden for PDO og prøver desperat at finde en fornuftig måde at strukturere det på, inden min side bliver for stor.

Jeg har altid kørt med funktioner i de tidligere måder - Jeg synes bare det bliver meget forvirrende.

Jeg er startet med at lave en klasse:
class Database
{
  protected $dbh;
  public function __construct() {
      $dsn = 'mysql:dbname=XXX;host=XXXX';
      $user = 'XXXX';
      $pass = 'XXXX';

      try {
        $this->dbh = new PDO($dsn, $user, $pass);
      } catch (PDOException $e) {
        echo 'Connection failed: ' . $e->getMessage();
      }
  }

  public function PostBy($PostNr) {
      $stmt = $this->dbh->prepare("
      SELECT PostNr, Bynavn
        FROM `PostNr`
        WHERE `PostNr` =$PostNr");
      if( $stmt->execute() ) {
        $lokation = $stmt->fetch(PDO::FETCH_ASSOC);
        return $lokation['PostNr'] . " " . $lokation['Bynavn'];
      }
      // Query failed
      return false;
  }
}

Nu nåede jeg så lidt længere og begyndte at få problemer med at lave insert og updates.

Er det smart at lave det i en stor klasse, laver i funktioner eller hvordan strukturere i jeres PDO? Jeg er nemlig meget forvirret...
Avatar billede arne_v Ekspert
13. oktober 2013 - 03:38 #1
} catch (PDOException $e) {
        echo 'Connection failed: ' . $e->getMessage();
      }

er et stort problem:
- du outputter noget i dit data access layer fremfor i dit presentation layer
- du koerer videre i tilfaelde af en fejl som goer at resten vil fejle
Avatar billede arne_v Ekspert
13. oktober 2013 - 03:39 #2
public function PostBy($PostNr) {

at slaa by op udfra postnr skal med stor sandsynlighed ikke laves i seperat query men som dem del af en JOIN
Avatar billede arne_v Ekspert
13. oktober 2013 - 03:40 #3
hverken "database" eller "postnr konkataneret med by" er oplagte business objekter
Avatar billede arne_v Ekspert
13. oktober 2013 - 03:41 #4
overvej f.eks. en Person klasse med oplysninger om person og en PersonManager klasse som kan hente/gemme Person instanser fra/i en database.
Avatar billede kviller Novice
13. oktober 2013 - 23:04 #5
Tak for feedback.
Det jeg egentlig eftersøgte var, om det var smartest at lave diverse funktioner, eller om det var en fordel at skrive selve quries og pdo i siderne.

Jeg synes hurtigt det bliver meget uoverskueligt uanset hvilken måde man vælger at gøre det på.

Lave en fil med classes og funktioner, eller bare holde det samlet i en stor pærevældling?
Avatar billede arne_v Ekspert
14. oktober 2013 - 04:16 #6
du skal selvfoelge bruge en lag opdeling
Avatar billede kviller Novice
04. august 2014 - 20:58 #7
Arne smider du ikke lige et svar? Så jeg kan lukke denne gamle tråd ;)
Avatar billede arne_v Ekspert
04. august 2014 - 21:02 #8
svar
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Vi tilbyder markedets bedste kurser inden for webudvikling

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester