Visualizzazione dei risultati da 1 a 10 su 21

Discussione: Consiglio su PDO

Hybrid View

  1. #1
    Utente di HTML.it
    Registrato dal
    Oct 2014
    Messaggi
    539
    nella classe che ho postato, ci sono 3 funzioni che nella versione "produttiva" sono,
    Codice PHP:
      public function MyLog($text)
      {
        return;
        
    error_log($this->SWN." ".$text0);
      }  
      public function 
    MyErr($text)
      {
        
    error_log($this->SWN." ".$text0);
        require_once 
    NS('Imailsoliderr');
        
    logoff();
        print 
    "ERRORE INATTESO, contatta l'amministratore del sistema";
        die;
      }
      private function 
    handle_sql_errors($sql$e)
      {
        if (
    $this->inTransaction$this->pdo->rollBack();
        
    error_log("SQL cmd    : ".$sql0);
        
    error_log("error code : ".$e->getCode(), 0);
        
    error_log("error info : ".$e->getMessage(), 0);
        require_once 
    NS('Imailsqlerr');
        
    logoff();
        print 
    "ERRORE INATTESO, contatta l'amministratore del sistema";
        die;
      } 
    come vedi myLog è di fatto disabilitata, la utilizzo quando devo fare debug degli script che la richiamano,

    ho cercato di proteggere (spero) i vari script con accorgimenti "anti intrusione",quando mi accorgo che qualcosa non va, viene attivata la funzione myErr che invia mail e interrompe, quindi l'aspettativa è che non sia mai richiamata
    la terza gestisce le segnalazioni di errore delle query, anche in questo caso l'aspettativa è che non sia mai chiamata perché il debug è stato fatto in fase di sviluppo, quindi se dovesse essere richamata si tratta di un errore "grave", non certo imputabile ad errore di sintassi o simili

    il codice non è sicuramente OOP ma è contenuto al minimo (facile e con meno possibilità di errore) e dovrebbe essere scritto in maniera da essere leggibile dai più, non solo dai "guru" cosa importante quando si lavora in un team di persone con livelli di conoscenza diversificati
    Ultima modifica di marino51; 17-04-2015 a 13:29

  2. #2
    Utente di HTML.it L'avatar di Fractals87
    Registrato dal
    Apr 2008
    Messaggi
    1,202
    Quote Originariamente inviata da marino51 Visualizza il messaggio
    il codice non è sicuramente OOP ma è contenuto al minimo (facile e con meno possibilità di errore) e dovrebbe essere scritto in maniera da essere leggibile dai più, non solo dai "guru" cosa importante quando si lavora in un team di persone con livelli di conoscenza diversificati
    Anche se molti non condividono io quoto in pieno.
    D' altro canto alcune cose pur complicate che siano semplificano e velocizzano determinate operazioni.
    Che mestiere difficile.....essere da soli ancora di più

Permessi di invio

  • Non puoi inserire discussioni
  • Non puoi inserire repliche
  • Non puoi inserire allegati
  • Non puoi modificare i tuoi messaggi
  •  
Powered by vBulletin® Version 4.2.1
Copyright © 2026 vBulletin Solutions, Inc. All rights reserved.