problema con elseif

[Rossella_75]

Ciao a tutti,

non riesco a capire dovestò sbagliando

			$tipologia = strtoupper($row_compra['tipo']);
			if ($tipologia == "MONO"){
			$tipologia = "monolocale";
			}elseif($tipologia == "BILO"){
			$tipologia == "bilocale";
			}elseif($tipologia == "TRILO"){
			$tipologia == "trilocale";
			}elseif($tipologia == "VILLINI"){
			$tipologia == "villini";
			};

miviene fuori sempre MONO anche quando la variabile $row_compra['tipo']contiene VILLINI

:master: GRAZIE

[SoulHome]

sembrerebbe tutto giusto, hai controllato ke $row_compra['tipo'] contenga effettivamente la parola VILLINI?
forse sarebbe il caso ke tu postassi tutto il codice...

[thepooh]

le elseif non le utilizzo mai perchè mi incasino, prova a scrivere il codice così

if ($tipologia == "MONO"){
$tipologia = "monolocale";
}else{
if($tipologia == "BILO"){
$tipologia == "bilocale";
}else{
if($tipologia == "TRILO"){
$tipologia == "trilocale";
}else{
if($tipologia == "VILLINI"){
$tipologia == "villini";
};
}
}
}

[mcganass]

si ti conviene controllare cosai ci sia in $row_compra

ma se devi fare controlli di questo genere, al posto di usare cosi tanti if...else if (poi prova a scrivere else if e non elseif non penso sia questo il problema ma tentar non nuoce)

ti conviene usare la funzione switch

http://it.php.net/manual/it/control-...res.switch.php

nel tuo caso

$tipologia = strtoupper($row_compra['tipo']);
switch($tipologia)
{
case "MONO":
$tipologia="monolocale";
break;
case"VILLINI"
$tipologia="villini";
break;
default:
$tipologia="non classificato";
}

[thepooh]

mi ha formattato il codice malissimo
ti volevo dire che mi è scappato un ; che non ci va

};

comunque stai testando lo stesso campo, nulla vieta di eseguire una if e chiuderla subito immediatamente per 4 volte.

if ($tipologia == "MONO"){ 
$tipologia = "monolocale"; 
}
if($tipologia == "BILO"){ 
$tipologia == "bilocale"; 
}
if($tipologia == "TRILO"){ 
$tipologia == "trilocale"; 
}
if($tipologia == "VILLINI"){ 
$tipologia == "villini"; 
}

[mcganass]

if ($tipologia == "MONO"){
$tipologia = "monolocale";
}else{
if($tipologia == "BILO"){
$tipologia == "bilocale";
}else{
if($tipologia == "TRILO"){
$tipologia == "trilocale";
}else{
if($tipologia == "VILLINI"){
$tipologia == "villini";
};
}
}
}

brutto ancora peggio delle else if

[gianiaz]

ragazzi usate switch();
semplice e pulito.

ciao

[Rossella_75]

PERFETTO COSI!!

grazie mille a tutti!

Originariamente inviato da mcganass
si ti conviene controllare cosai ci sia in $row_compra

ma se devi fare controlli di questo genere, al posto di usare cosi tanti if...else if (poi prova a scrivere else if e non elseif non penso sia questo il problema ma tentar non nuoce)

ti conviene usare la funzione switch

http://it.php.net/manual/it/control-...res.switch.php

nel tuo caso

$tipologia = strtoupper($row_compra['tipo']);
switch($tipologia)
{
case "MONO":
$tipologia="monolocale";
break;
case"VILLINI"
$tipologia="villini";
break;
default:
$tipologia="non classificato";
}