Problema con SWITCH CASE

[angelomorto]

ho creato uno script che al suo interno contiene il seguete codice:

switch (true){
  case ($colname =='v_products_price'):
       if ($ricarico == '' || $ricarico == NULL){ $ricarico = 0; }
       $price = (($split_uploaded_filename[$col] * $ricarico) / 100) + $split_uploaded_filename[$col];
       $data_string .= $price.$tab;
       break;
  case ($search=(strpos($colname,'name')) !== FALSE):
       for ($z=1; $z<=$num_lang ; $z++){
       $data_string .= $split_uploaded_filename[$col].$z.$tab;
       }//eof for
       break;
   default: $data_string .= $split_uploaded_filename[$col].$tab;
}// eof switch 

ma lo script mi esegue sempre il secondo "case" anche quando nella stringa $colname non compare la parola name...secondo voi è giusto lo script?? Sicuramente ho sbagliato qualocosa che non arrivo a vedere!! :master:
Cmq voglio che il secondo "case" si attivi solo quando all'interno della stringa $colname compare la parola "name", è corretta la sintassi???

[Al_katraz984]

cos'è true????

switch (true){

[bonaprogram]

hai sbagliato la struttura dello switch

switch ($colname){
  case 'v_products_price':
       if ($ricarico == '' || $ricarico == NULL){ $ricarico = 0; }
       $price = (($split_uploaded_filename[$col] * $ricarico) / 100) + $split_uploaded_filename[$col];
       $data_string .= $price.$tab;
       break;
  case strpos($colname,'name'):
       for ($z=1; $z<=$num_lang ; $z++){
       $data_string .= $split_uploaded_filename[$col].$z.$tab;
       }//eof for
       break;
   default: $data_string .= $split_uploaded_filename[$col].$tab;
}// eof switch 

[angelomorto]

Originariamente inviato da bonaprogram
hai sbagliato la struttura dello switch

ho modificato lo script come mi hai detto, ma ora mi va sempre in default....ho fatto per sicurezza un echo di tutto e mi restituisce questo:

------------------------
v_products_model default v_products_image default v_products_name_ default v_products_description_ default v_products_url_ default v_specials_price default v_specials_date_avail default v_specials_expires_date default v_products_price case1 v_products_weight default v_date_avail default v_date_added default v_products_quantity default v_manufacturers_name default
------------------------

Per comodità visiva ho messo in grassetto il conetnuto di $colname e in rosso i punti dove ha sbagliato.
Dove sta l'errore?

[angelomorto]

Originariamente inviato da Al_katraz984
cos'è true????

switch (true){

avevo seguito questa idea http://it.php.net/manual/it/control-...itch.php#54251

ma a quanto pare nn funziona nessuno dei due metodi

[luca200]

Originariamente inviato da bonaprogram
hai sbagliato la struttura dello switch

Ma anche no. Sei tu che l'hai sbagliata.

@angelomorto: il tuo codice per me va bene. Facendo la stessa prova che hai fatto tu mi esce questo

v_products_model: default
v_products_image: default
v_products_name_: case2
v_products_description_: default
v_products_url_: default
v_specials_price: default
v_specials_date_avail: default
v_specials_expires_date: default
v_products_price: case1
v_products_weight: default
v_date_avail: default
v_date_added: default
v_products_quantity: default
v_manufacturers_name: case2

Questo è il codice che ho usato

function sw($colname) {
echo "$colname: ";
switch (true){
  case ($colname =='v_products_price'):
       echo"case1
";
       break;
  case ($search=(strpos($colname,'name')) !== FALSE):
       echo "case2
";
       break;
   default: echo "default
";
}
}

$string[] = 'v_products_model';
$string[] = 'v_products_image';
$string[] = 'v_products_name_';
$string[] = 'v_products_description_';
$string[] = 'v_products_url_';
$string[] = 'v_specials_price';
$string[] = 'v_specials_date_avail';
$string[] = 'v_specials_expires_date';
$string[] = 'v_products_price';
$string[] = 'v_products_weight';
$string[] = 'v_date_avail';
$string[] = 'v_date_added';
$string[] = 'v_products_quantity';
$string[] = 'v_manufacturers_name';
foreach ($string as $str) sw($str); 

[angelomorto]

Originariamente inviato da luca200
Ma anche no. Sei tu che l'hai sbagliata.

@angelomorto: il tuo codice per me va bene. Facendo la stessa prova che hai fatto tu mi esce questo

Bho! non so che dirti, per come l'avevo fatto nel primo post nn mi funzionava, invece modificandolo così funziona

switch (true){ 
  case ($colname =='v_products_price'): 
       if ($ricarico == '' || $ricarico == NULL){ $ricarico = 0; } 
       $price = (($split_uploaded_filename[$col] * $ricarico) / 100) + $split_uploaded_filename[$col]; 
       $data_string .= $price.$tab; 
       break; 
  case (strpos($colname,'name') == TRUE): 
       for ($z=1; $z<=$num_lang ; $z++){ 
       $data_string .= $split_uploaded_filename[$col].$z.$tab; 
       }//eof for 
       break; 
   default: $data_string .= $split_uploaded_filename[$col].$tab; 
}// eof switch 

:master:

[luca200]

Originariamente inviato da angelomorto
Bho! non so che dirti, per come l'avevo fatto nel primo post nn mi funzionava, invece modificandolo così funziona

Come hai visto io l'ho provato nella forma originale e funzionava.
Visto che le istruzioni che esegui nel secondo case e nel default si assomigliano, probabilmente avevi sbagliato a interpretare i risultati (oppure avevi un errore nel codice ma l'hai corretto postando qui )

[angelomorto]

Originariamente inviato da luca200
Come hai visto io l'ho provato nella forma originale e funzionava.
Visto che le istruzioni che esegui nel secondo case e nel default si assomigliano, probabilmente avevi sbagliato a interpretare i risultati (oppure avevi un errore nel codice ma l'hai corretto postando qui )

Molto probabile! Purtroppo non sembra ma il PHP è molto sensibile, basta una virgola che non funziona più niente!

[bonaprogram]

è cosi per tutti i linguaggi di programmazione