Problemi if else

[Zlatan94]

Salve. Vi spiego cosa vorrei fare.
Io nel database ho una tabella utenti con i campi id, numero, sponsor1 e sponsor2. Quando io sono nella pagina per modificare un utente ho in mio possesso il suo id.
Vorrei che se sponsor1 e sponsor2 sono maggiori di 0 compaia l'immagine si.png altrimenti compaia l'immagine no.png.

Io ho usato questo codice:

$id=$_GET['id'];
//asteclick
function sponsor1() {$query = "SELECT sponsor1 FROM utenti WHERE id = '$id'";

if($sponsor1>0) {
echo"[img]si.png[/img]";
}
else {
echo"[img]no.png[/img]";
}}

function sponsor2() {$query = "SELECT sponsor2 FROM utenti WHERE id = '$id'";

if($sponsor2>0) {
echo"[img]si.png[/img]";
}
else {
echo"[img]no.png[/img]";
}}

Escludendo poi la connessione al db che è nel codice più sopra. Ma purtroppo anche se $sponsor1 è maggiore di 0 me lo da negativo (no.png). Perchè?

[brodik]

spero che la esegui la query dopo averla scritta

$query = "SELECT sponsor1 FROM utenti WHERE id = '$id'";
mysql_query($query);

[Justino]

Scrivi la query sql ma non la esegui. E' normale che i valori ti risultino false e quindi non maggiori di 0.

Devi fare così:

$id=$_GET['id'];

//asteclick
function sponsor()
{
$query = "SELECT sponsor1, sponsor2 FROM utenti WHERE id = '$id'";
$result = mysql_query($query);
$row = mysql_fetch_row($result);

if($row['sponsor1'] > 0 && $row['sponsor2'] > 0) {
return "[img]si.png[/img]";
}
else {
return "[img]no.png[/img]";
}

} 

Così facendo l'immagine si.png apparirà solamente se sponsor1 e sponsor2 sono entrambi maggiori di 0, se vuoi che sia sufficiente che solo uno dei due valori sia positivo devi sostituire il && nell'if con un ||

Ricorda di fare attenzione con le funzioni, che devono avere un return e non un echo.

[Zlatan94]

Niente da fare. Ora il codice sta cosi ma da sempre i valori a caso.

function sponsor1() {$query = "SELECT sponsor1 FROM utenti WHERE id = '$id'"; 
$result = mysql_query($query); 
$row = mysql_fetch_row($result); 

if($row['sponsor1'] > 0) { 
return "[img]si.png[/img]"; 
} 
else { 
return "[img]no.png[/img]"; 
} }

function sponsor2() {$query = "SELECT sponsor2 FROM utenti WHERE id = '$id'"; 
$result = mysql_query($query); 
$row = mysql_fetch_row($result); 

if($row['sponsor2'] > 0) { 
return "[img]si.png[/img]"; 
} 
else { 
return "[img]no.png[/img]"; 
} }

.


Dove sponsor 1 nel db per esempio nell'id dell'utente "prova" ha "1" mentre sponsor 2 ha "2" ma compaiono tutti e due no.png.

[Justino]

Ma perchè eseguire due query su una stessa tabella con inutile spreco di risorse quando puoi utilizzare una sola query?

[Zlatan94]

Codice?

[brodik]

function sponsor() {

$query = "SELECT sponsor1, sponsor2 FROM utenti WHERE id = '$id'"; 
$result = mysql_query($query); 
$row = mysql_fetch_row($result); 

if(!empty($row['sponsor1'])){ 
return "[img]si.png[/img]"; 
}else{ 
return "[img]no.png[/img]"; 
}

if(!empty($row['sponsor2'])){ 
return "[img]si.png[/img]"; 
}else{ 
return "[img]no.png[/img]"; 
}

} 

[Justino]

Originariamente inviato da brodik

function sponsor() {

$query = "SELECT sponsor1, sponsor2 FROM utenti WHERE id = '$id'"; 
$result = mysql_query($query); 
$row = mysql_fetch_row($result); 

if(!empty($row['sponsor1'])){ 
return "[img]si.png[/img]"; 
}else{ 
return "[img]no.png[/img]"; 
}

if(!empty($row['sponsor2'])){ 
return "[img]si.png[/img]"; 
}else{ 
return "[img]no.png[/img]"; 
}

} 

In realtà questo codice non è del tutto corretto perchè, per come è scritto, php non analizzerà affatto la variabile $row['sponsor2'], dal momento che prenderà per buono il primo return che trova, relativo a sponsor1. Si può risolvere con un array come questo:

function sponsor() {

$query = "SELECT sponsor1, sponsor2 FROM utenti WHERE id = '$id'"; 
$result = mysql_query($query); 
$row = mysql_fetch_row($result); 

if(!empty($row['sponsor1'])){ 
$sponsor1 = "[img]si.png[/img]"; 
}else{ 
$sponsor1 = "[img]no.png[/img]"; 
}

if(!empty($row['sponsor2'])){ 
$sponsor2 = "[img]si.png[/img]"; 
}else{ 
$sponsor2 = "[img]no.png[/img]"; 
}

return array($sponsor1, $sponsor2);

} 

[brodik]

si è vero grazie per la correzione..

[Justino]

Siamo qui per offrirci supporto a vicenda, figurati