stringa non letta

ciao a tutti ho questo script che non mi funziona:

$query = " SELECT * FROM foto WHERE nick = '$nickname' ";
 $invio=mysql_query($query); $numrows = mysql_num_rows($invio);

while($row = mysql_fetch_array($invio)){
 $url = $row['url'];$visib = $row['visibile'];

 if(file_exists($url))
{ if($visib="pubblica"){ ...stampa a schermo la foto...} else { ...non visibile}
 }
} 

in pratica uso il database per memorizzare le url delle immagini + un altro campo per tenerle private o pubbliche... però il sistema non va perchè me le stampa tutte senza tener conto della visibilità assegnata!! qualcuno sa dirmi dove sbaglio?? il campo visibilità era varchar poi longtext ma non mi funziona lo stesso :(

[alemoppo]

if($visib="pubblica") 

Quello è l'operatore di assegnamento... Invece lì andrebbe l'operatore di confronto

Comunque, usare una stringa o addirittura un longtest per salvare chi lo può vedere, mi sembra eccessivo...
Potresti usare un bool (0 o 1). Ad esempio, puoi codificarli con 0=privato, 1=pubblico. Se hai altre combinazioni, puoi usare un numero, ma usare una stringa mi sembra uno spreco di memoria!

Ah, comunque è molto meglio selezionarli direttamente in mysql, ad esempio, nel tuo caso (esempio):

$query = " SELECT * FROM foto WHERE nick = '$nickname' AND visibile='pubblica'";

Ciao!

ciao alemoppo si lo so ma sta cosa del pubblico e privato mi è venuta in secondo momento e ora dovrei rifare tutto daccapo :( non cè qualche altra soluzione?? intendo qui ho sintetizzato il problema ma lo script è molto+lungo.... :(

EDIT:

Codice:
$query = " SELECT * FROM foto WHERE nick = '$nickname' AND visibile='pubblica'";

so che è meglio così ma a me serve visualizzarli in quel modo.. con questo dovrei fare un altra query per il visibile='privato' però ho già impaginato il tutto per visualizzare 5righe alla volta

[alemoppo]

Mi sembra ti ho dato più di una soluzione, no?

Ricapitolando:

$query = " SELECT * FROM foto WHERE nick = '$nickname' ";
 $invio=mysql_query($query); $numrows = mysql_num_rows($invio);

while($row = mysql_fetch_array($invio)){
 $url = $row['url'];$visib = $row['visibile'];

 if(file_exists($url))
{ if($visib=="pubblica"){ ...stampa a schermo la foto...} else { ...non visibile}
 }
} 

Oppure:

<?php
$query = " SELECT * FROM foto WHERE nick = '$nickname' AND visibile = 'pubblica' ";
 $invio=mysql_query($query); $numrows = mysql_num_rows($invio);

while($row = mysql_fetch_array($invio)){
stampa a schermo la foto$row['url']
} ?>

Ciao!

{ if($visib==="pubblica"){ ...stampa a schermo la foto...} else { ...non visibile}
 } 

uff ho provato anche così ma me le stampa tutte lo stesso!!!
il campo visibile sul database può essere pubblico e privato ma non capisco xkè mi esegue sta cosa in modo anomalo :(

[alemoppo]

Metti solo due "uguale", non tre (anche se in effetti dovrebbe andar uguale).

Prova a stampare $visib per vedere cosa contiene...

Ciao!

si fatto anche in questo modo!!! mi stampa il valore esatto dal database ma nnon riesco a capire cosa non và eppure è uno script semplice ::(

[alemoppo]

se $visib contiene "pubblica", l'if:

if($visib == "pubblica") 

non fallisce...

Ciao!