[php] manier van proggen

[koli-man]

Ik heb gepoogd OO te programmeren in php. Een simpel gastenboek. En wel ongeveer op deze manier.

PHP:

<?php
//begin class Guestbook
class Guestbook
{
 
  function Guestbook(){
  }
  //begin function
  function GuestbookPrint()  //afdrukken van inhoud class persoon
  {
     include("connect.php");
     $sql_select = "SELECT * FROM gastenboek ORDER BY id DESC";       
     $result = mysql_query($sql_select,$db);
     if($result){
          while ($message = mysql_fetch_object($result)) {
            echo "<center><p><table>";
            echo "<tr><td 

width=125><b>Naam:</b></td><td>$message->naam</td></tr>";
            echo "<tr><td 
width=125><b>WWW:</b></td><td><a href='$message->homepage'>$message->homepage</a></td></tr>";

            echo "<tr><td valign=top width=125><b>E - mail:</b></td><td>
<a href='mailto:$message->email'>$message-
>email</td></tr>";
            echo "<tr><td valign=top 
width=125><b>berichtje:</b></td><td>$message->text</td></tr>";
            echo "<tr><td width=125><b>Toegevoegd 
op:</b></td><td><font color=#FF0000>$message->datumtijd</font></td></tr>";

            echo "</table></p><br><hr size=1 color=#000000 
width=60%></center>";
          }
     }  
  }//end function
  
  
}
//end class guestbook

//begin class message
class Message
{
  var $name;
  var $homepage;
  var $email;
  var $ip;
  var $datetime;
  var $text;
  
  //begin function
  function Message($_name,$_homepage,$_email,$_text,$_ip,$_datetime) 
  {
    $this->name = $_name; 
    $this->homepage = $_homepage ;
    $this->email = $_email;
    $this->text = $_text;
    $this->ip = $_ip;
    $this->datetime =  $_datetime;//current date and time
    $this->browser = $_browser;
  }//end function
  

  
  //begin function
  function AddtoDatabase()
  {
     include("connect.php");//connect met de database
     //$this->datetime = "";
     $sql_insert = "INSERT INTO gastenboek VALUES
('','$this->name','$this->homepage','$this->email','$this->datetime','$this->text','$this->ip')"; 
     $resultaat = mysql_query($sql_insert, $db);
      if($resultaat){
        echo "Uw reactie is toegevoegd "; 
     }
     else
 {echo "Excuses, er is een fout opgetreden! Druk op de back - knop en <br> 
probeer het nog een keer!";
           echo mysql_error(); 
          }
    mysql_close($db);     
  } 
  //end function
} //end class message 

Nou heb ik een html - pagina waarbij ik bij het posten de variabelen meegeef naar het volgende php - bestand

PHP:

<?php
include("classmessage.php");
$_message = new Message($name,$homepage,$email,$text,$ip,$datetime);
$_message->AddtoDatabase();
?>

Nu mijn vraag: "Is dit ongeveer de manier waarop men OO programmeert in PHP?"

Of is een andere manier gebruikelijk?

[ Voor 52% gewijzigd door koli-man op 28-06-2003 16:16 . Reden: layout ]

[kvdveer]

Het kan nog netter: Een gastenboek bestaat toch uit messages? Laat dan een message zichzelf wegschrijven.

Je krijgt dan in guestbook zoiets:

PHP:

while($row = fetch_object()) {
  $a[] &= $x = new Message();
  $x->readFromDb($row);
  echo $x->getHtml();
}

[koli-man]

hmmz ja, maar hier bouw je toch elke keer het hele gastenboek op, terwijl je eigenlijk maar één bericht wilt wegschrijven, want in de lus leest hij telkens van de DB?

[ Voor 12% gewijzigd door koli-man op 28-06-2003 16:33 ]

[Verwijderd]

Zoek eerst wat op over templates & PHP, en begin dan opnieuw!

HTML & PHP door elkaar (en dat zelfs in 1 class) is echt geen werk.

Succes

[kvdveer]

Verwijderd schreef op 28 juni 2003 @ 16:57:
Zoek eerst wat op over templates & PHP, en begin dan opnieuw!

HTML & PHP door elkaar (en dat zelfs in 1 class) is echt geen werk.

Succes

Lekkere flame... bovendien ongefundeerd.
Het is onmogelijk om een class te schrijven die output genereert zonder er HTML in te zetten. Oftewel: je praat onzin (oh, da's ook een flame)

[koli-man]

wat is daar verkeerd aan? html in je php, tja ik kan constant de <?php ?> enzo erin zetten, maar dan komt de leesbaarheid niet ten goede

[muis]

Iedere klasse zou alleen verantwoordelijk moeten zijn voor zich zelf.

De klasse Message zou alleen de message zelf moeten opmaken dmv een query oid.
Nu houdt de klasse Message zich ook nog bezig met de db connectie en verzorgt ie een stuk HTML

Als de klasse bv voor 2 verschillende guestbooks wilt gebruiken maar bij allebei een ander bericht wilt teruggeven wanneer een message is toegevoegd dan wordt dat nu een beetje moeilijk. Door bv alleen een "true" of een "false" terug te geven kan je vervolgens per guestbook bepalen watvoor melding je erbij geeft

[Verwijderd]

kvdveer schreef op 28 June 2003 @ 17:09:
[...]

Lekkere flame... bovendien ongefundeerd.
Het is onmogelijk om een class te schrijven die output genereert zonder er HTML in te zetten. Oftewel: je praat onzin (oh, da's ook een flame)

Ja natuurlijk moet je 'ergens' html hebben, best logisch, weet hij ook wel.

Maar het is wel mogelijk om een class te schrijven die je messages ophaalt/wegschrijft van/naar de database en een andere class die de output genereert.

En dat moet hij nu namelijk zelf ontdekken; en als je het principe van die scheiding van de eerste keer aanleert dmv. templates, zal het resultaat niet zo slecht zijn, ipv. hem nu verder te laten klooien met classes waar hij na een aantal maanden toch geen raad meer mee weet.

Ik heb hem een tip gegeven ('zoek naar templates'), dat is al een goede duw in de richting. Waarom jij dat aanziet als flame, is mij een raadsel.

edit: idd muis, zo kan je het stellen. Wat heeft het nu voor zin hem hier verder mee te laten vechten, als het toch al een verloren strijd is

[ Voor 21% gewijzigd door Verwijderd op 28-06-2003 17:21 ]

[koli-man]

muis schreef op 28 June 2003 @ 17:15:
Iedere klasse zou alleen verantwoordelijk moeten zijn voor zich zelf.

De klasse Message zou alleen de message zelf moeten opmaken dmv een query oid.
Nu houdt de klasse Message zich ook nog bezig met de db connectie en verzorgt ie een stuk HTML

Ja, inderdaad daar heb je een punt, die db connectie kan ik ergens anders zetten en de html kan ik ook wel ergens anders zetten. Daar hedde gelijk in.

Als de klasse bv voor 2 verschillende guestbooks wilt gebruiken maar bij allebei een ander bericht wilt teruggeven wanneer een message is toegevoegd dan wordt dat nu een beetje moeilijk. Door bv alleen een "true" of een "false" terug te geven kan je vervolgens per guestbook bepalen watvoor melding je erbij geeft

[koli-man]

Verwijderd schreef op 28 June 2003 @ 17:17:
[...]
Ja natuurlijk moet je 'ergens' html hebben, best logisch, weet hij ook wel.


Ik heb hem een tip gegeven ('zoek naar templates'), dat is al een goede duw in de richting. Waarom jij dat aanziet als flame, is mij een raadsel.

edit: idd muis, zo kan je het stellen. Wat heeft het nu voor zin hem hier verder mee te laten vechten, als het toch al een verloren strijd is

Ten eerste: Ik wil niet persé templates gebruiken. De functionaliteit van templates kan ik ook bereiken met het zelf schrijven van functies.

Ten tweede: "verloren strijd?!? " Er kan altijd iets aangepast worden en daar sta ik voor open!!!

[Verwijderd]

Je titel van je thread is 'manier van proggen'. Bij deze?

[JeRa]

Het combineren van PHP en HTML in een class maakt het er gewoon niet duidelijker op; wat ik gewoonlijk doe is een class maken die al het dataverkeer regelt, en een frontpage (met die class geincluded) die de HTML output.

[koli-man]

JeRa schreef op 28 June 2003 @ 17:32:
Het combineren van PHP en HTML in een class maakt het er gewoon niet duidelijker op; wat ik gewoonlijk doe is een class maken die al het dataverkeer regelt, en een frontpage (met die class geincluded) die de HTML output.

Ja, dat is inderdaad beter. Net zoals de connectie met de database e.d. apart gezet kan worden. Maar het gebruik van templates ben ik eigenlijk niet zo zeker van.

[JeRa]

koli-man schreef op 28 June 2003 @ 17:34:
[...]


Ja, dat is inderdaad beter. Net zoals de connectie met de database e.d. apart gezet kan worden. Maar het gebruik van templates ben ik eigenlijk niet zo zeker van.

Templates zijn in dit geval overbodig; dit is alleen handig als je je code wilt gaan publiceren en door anderen wilt laten gebruiken zodat zij makkelijk de templates kunnen aanpassen zonder in de PHP-code te zitten.

[MisterData]

JeRa schreef op 28 June 2003 @ 17:36:
[...]


Templates zijn in dit geval overbodig; dit is alleen handig als je je code wilt gaan publiceren en door anderen wilt laten gebruiken zodat zij makkelijk de templates kunnen aanpassen zonder in de PHP-code te zitten.

...en ze zijn handig om je code net te houden. Ik doe niets liever dan templates proggen: niet alleen omdat ik dan makkelijk de layout kan aanpassen, maar nog veel meer omdat de code net wordt. Geen HTML meer tussen PHP. Ik gebruik voor templates m'n eigen stackbased parser (http://dev.trag.nl/ccbv/include/tpltest.php en www.codebase.nl voor de download).

[djluc]

Ik heb een aantal classes. De template engine is natuurlijk ook een class. Het combineren van beide gaat in PHP echter lastig omdat je dan met bijvoorbeeld extends moet gaan werken? Je zult toch op een of andere manier die dingen in je class moeten krijgen.