Questo sito utilizza cookies, anche di terze parti, per mostrare pubblicità e servizi in linea con il tuo account. Leggi l'informativa sui cookies.
Username: Password: oppure
PHP - consiglio su come rendere il codice ordinato e pulito ..
Forum - PHP - consiglio su come rendere il codice ordinato e pulito ..

Avatar
giocala88 (Normal User)
Expert


Messaggi: 243
Iscritto: 23/04/2008

Segnala al moderatore
Postato alle 16:38
Lunedì, 22/02/2010
Salve ragazzi, ho concluso finalmente il progetto per la gestione dei campionati di calcetto, grazie anche al vostro supporto, ora però vorrei un consiglio su come rendere il codice più pulito ed ordinato.
Queste due funzioni, prese come esempio, rispecchiano in linea generale le strutture delle altre mie funzioni di lettura.
C'è da fare qualche modifica ai fini di rendere il codice più pulito ed ordinato?
grazie anticipatamente ...:)

Codice sorgente - presumibilmente Php

  1. <?php
  2.  
  3. // dichiaro una funzione di lettura di un campionato specifico query campionato_naz
  4. function read_camp() {
  5.  
  6.         // recupero lo script responsabile della connessione al database
  7.         include ("connect.php");
  8.  
  9.         // includo lo script responsabile del passaggio di parametri
  10.         include ("config.get.php");
  11.  
  12.         // seleziono la query
  13.         $read_camp = mysql_query("SELECT * FROM campionato_naz WHERE id_campionato = '$id_camp'");
  14.  
  15.         // controllo l'esito
  16.         if(!$read_camp) {
  17.  
  18.                 // visualizzo un mex
  19.                 echo "<b>!</b>Errore lettura function read_camp.";
  20.  
  21.         }
  22.  
  23.         // ciclo gli elementi delle due funzioni di lettura
  24.         while ($result    = mysql_fetch_array($read_camp)) {
  25.  
  26.                 // associo una  più variabili a mysql_fetch_array
  27.                 $id_camp = $result['id_campionato'];
  28.                 $anno    = $result['anno'];
  29.                 $nazione = $result['nazione'];
  30.                 $attivo  = $result['attivo'];
  31.                
  32.                 // ritorno un valore vero
  33.                 echo "nome campionato : <b>$id_camp</b>" . "<br/>" . "anno : <b>$anno</b>" . "<br/>" . "nazione : <b>$nazione</b>" . "<br/>";
  34.                
  35.                 // controllo l'esito: il campo attivo è uguale a zero
  36.                 if($attivo == 0) {
  37.  
  38.                         // visualizzo un mex
  39.                         echo "concluso : <b>no</b>";
  40.                 }
  41.                 //altrimenti, controllo l'esito: il campo attivo è diverso da zero
  42.                 else {
  43.  
  44.                         // visualizzo un mex
  45.                         echo "concluso : <b>si [data]</b>";
  46.                 }              
  47.         }
  48.  
  49.         // libero la memoria di PHP occupata dai record estratti da SELECT
  50.         mysql_free_result($read_camp);
  51.  
  52.         // chiudo la connessione a mysql
  53.         mysql_close();
  54.        
  55. }
  56.  
  57. ?>



Codice sorgente - presumibilmente Php

  1. <?php
  2.  
  3. // dichiaro una funzione di lettura dlla lista id_campionati query campionato_naz
  4. function read_id_camp() {
  5.  
  6.         // seleziono la query
  7.         $read_id_camp = mysql_query("SELECT * FROM campionato_naz");
  8.  
  9.         // controllo l'esito
  10.         if(!$read_id_camp) {
  11.  
  12.                 // visualizzo un mex
  13.                 echo "<b>!</b>Errore lettura function read_id_camp.";
  14.  
  15.         }
  16.  
  17.         // ciclo gli elementi delle due funzioni di lettura
  18.         while ($result = mysql_fetch_array($read_id_camp)) {
  19.  
  20.                 // associo una  più variabili a mysql_fetch_array
  21.                 $id_camp = $result['id_campionato'];
  22.                 $naz     = $result['nazione'];
  23.                 $aaaa    = $result['anno'];
  24.  
  25.                 // ritorno un valore vero
  26.                 echo "<a href='index.php?lc=lc&id_camp=$id_camp&naz=$naz&aaaa=$aaaa'><b>$id_camp</b></a>" . "<br/><br/>";
  27.  
  28.                
  29.                
  30.         }
  31.  
  32.         // libero la memoria di PHP occupata dai record estratti da SELECT
  33.         mysql_free_result($read_id_camp);
  34.  
  35.         // chiudo la connessione a mysql
  36.         mysql_close();
  37. }
  38.  
  39. ?>


PM Quote
Avatar
Mte90 (Member)
Guru


Messaggi: 1144
Iscritto: 25/03/2008

Segnala al moderatore
Postato alle 20:53
Lunedì, 22/02/2010
per me è pulito e ordinato anche se variabili come $aaaa le puoi evitare XD


http://www.mte90.net/
Kde, Debian e Programmazione web/Desktop!


Linux Counter.li #482442
PM Quote
Avatar
cassin.simone (Member)
Guru


Messaggi: 1051
Iscritto: 03/06/2009

Segnala al moderatore
Postato alle 9:11
Martedì, 23/02/2010
Testo quotato

Postato originariamente da Mte90:

per me è pulito e ordinato anche se variabili come $aaaa le puoi evitare XD



Concordo. Comunque la variabile $aaaa ha una logica: s'ignifica anno e (per intuizione) verrà utilizzato a 4 cifre (ad es. 2040, 5432, 1203, etc.). :k:


Hey, sono anche su Twitter! www.twitter.com/simonecassin
PM Quote