Oppure

Loading
22/02/10 15:38
giocala88
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 ...:)

<?php 

// dichiaro una funzione di lettura di un campionato specifico query campionato_naz
function read_camp() {

	// recupero lo script responsabile della connessione al database
	include ("connect.php");

	// includo lo script responsabile del passaggio di parametri 
	include ("config.get.php");

	// seleziono la query
	$read_camp = mysql_query("SELECT * FROM campionato_naz WHERE id_campionato = '$id_camp'");

	// controllo l'esito
	if(!$read_camp) {

		// visualizzo un mex
		echo "<b>!</b>Errore lettura function read_camp.";

	}

	// ciclo gli elementi delle due funzioni di lettura
	while ($result    = mysql_fetch_array($read_camp)) {

		// associo una  più variabili a mysql_fetch_array 
		$id_camp = $result['id_campionato'];
		$anno    = $result['anno'];
		$nazione = $result['nazione'];
		$attivo  = $result['attivo'];
		
		// ritorno un valore vero
		echo "nome campionato : <b>$id_camp</b>" . "<br/>" . "anno : <b>$anno</b>" . "<br/>" . "nazione : <b>$nazione</b>" . "<br/>";
		
		// controllo l'esito: il campo attivo è uguale a zero
		if($attivo == 0) {

			// visualizzo un mex
                        echo "concluso : <b>no</b>";
		}
		//altrimenti, controllo l'esito: il campo attivo è diverso da zero
		else {

			// visualizzo un mex
			echo "concluso : <b>si [data]</b>";
		}		
	}

	// libero la memoria di PHP occupata dai record estratti da SELECT
	mysql_free_result($read_camp);
  
	// chiudo la connessione a mysql
	mysql_close();
	
}

?>


<?php 

// dichiaro una funzione di lettura dlla lista id_campionati query campionato_naz
function read_id_camp() {

	// seleziono la query
	$read_id_camp = mysql_query("SELECT * FROM campionato_naz");

	// controllo l'esito
	if(!$read_id_camp) {

		// visualizzo un mex
		echo "<b>!</b>Errore lettura function read_id_camp.";

	}

	// ciclo gli elementi delle due funzioni di lettura
	while ($result = mysql_fetch_array($read_id_camp)) {

		// associo una  più variabili a mysql_fetch_array 
		$id_camp = $result['id_campionato'];
		$naz     = $result['nazione'];
		$aaaa    = $result['anno'];

		// ritorno un valore vero
		echo "<a href='index.php?lc=lc&id_camp=$id_camp&naz=$naz&aaaa=$aaaa'><b>$id_camp</b></a>" . "<br/><br/>";

		
		
	}

	// libero la memoria di PHP occupata dai record estratti da SELECT
	mysql_free_result($read_id_camp);
  
	// chiudo la connessione a mysql
	mysql_close();
}

?>
aaa
22/02/10 19:53
Mte90
per me è pulito e ordinato anche se variabili come $aaaa le puoi evitare XD
aaa
23/02/10 8:11
cassin.simone
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:
aaa