login  Naam:   Wachtwoord: 
Registreer je!
 Forum

query in database class (Opgelost)

Offline Ultimatum - 20/05/2007 22:37
Avatar van UltimatumPHP expert Ey, ik ben bezig met een database class die werkt met try en catch. Maar mijn hele mysql_query() word niet gepakt.

  1. <?php
  2. public function query($query) {
  3.  
  4. try {
  5. if(mysql_query($query) === false) {
  6.  
  7. throw new errorHandeling('Sorry, could not execute the query');
  8. return false;
  9.  
  10. } else {
  11. return true;
  12. }
  13. } catch (errorHandeling $e) {
  14.  
  15. echo $e->showError();
  16. return false;
  17. }
  18. ?>


Of moet ik bij de true nog een keer mysql_query neerzetten? Is een beetje overbodig tocht of niet..

6 antwoorden

Gesponsorde links
Offline Wave6 - 20/05/2007 22:40
Avatar van Wave6 PHP beginner Ik denk dat je het eens zo moet proberen

  1. <?php
  2. public function query($query) {
  3.  
  4. try {
  5. if(mysql_query($query) === false) {
  6.  
  7. throw new errorHandeling('Sorry, could not execute the query');
  8. return false;
  9.  
  10. } else {
  11. return mysql_query($query);
  12. }
  13. } catch (errorHandeling $e) {
  14.  
  15. echo $e->showError();
  16. return false;
  17. }
  18. ?>


Maar ik kan ook nog fout zitten weet het niet zo heel zeker eerlijk gezegd
Offline Ultimatum - 20/05/2007 22:44 (laatste wijziging 20/05/2007 22:45)
Avatar van Ultimatum PHP expert Het werkt, maar de enige manier is dus om eerst te testen met mysql_query() en dan returnnen met nogmaals mysql_query(); ?

edit: bovenstaande werkt wel dus 
Offline Rens - 20/05/2007 23:00
Avatar van Rens Gouden medaille

Crew algemeen
Probeer het zo eens:
  1. <?php
  2. public function query($query) {
  3.  
  4. try {
  5. if(!$rResult = mysql_query($query)) {
  6.  
  7. throw new errorHandeling('Sorry, could not execute the query');
  8. return false;
  9.  
  10. } else {
  11. return $rResult;
  12. }
  13. } catch (errorHandeling $e) {
  14.  
  15. echo $e->showError();
  16. return false;
  17. }
  18. ?>

Zo doe ik het altijd, maar dan zonder dat try/catch gebeuren...
Offline Ultimatum - 20/05/2007 23:05
Avatar van Ultimatum PHP expert Hmm, waarom dacht ik daar niet aan . Bedankt het werkt :]
Offline Rens - 20/05/2007 23:06
Avatar van Rens Gouden medaille

Crew algemeen
Stuk logischer aangezien je dan maar 1 query uitvoerd.
Stel je zou dat moeten doen voor 30 queries, dan zou je er opeens 60 uitvoeren...
Beetje doelloos hè 
Offline MrD - 21/05/2007 21:40
Avatar van MrD Nieuw lid dit is niet echt de juiste manier om try en catch te gebruiken. Die hou je buiten je class en in je class kan je dan throw doen
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.188s