script php non funziona su firefox e explorer

[ddc90]

salve a tutti! è la prima volta che scrivo su questo forum, da cui però spesso attingo per utilissime informazioni! quindi vi ringrazio! vi espongo subito il mio problema

ho scritto un form che poi invia una mail. questo form prende informazioni da una pagina html nella quale il cliente sceglie il prodotto

il problema è che funziona perfettamente su chrome e safari, ma su firefox e explorer non ne vuole sapere!


potete provarlo online su http://www.terraditaliaproduction.it/APPOGGIO/file.html

il codice che credo non funziona riguarda una lavorazione sulle stringhe

spiego quel che viene fatto

prendo una stringa 'quantprez' che è del tipo "47/29.3" (ovvero due numeri separati da '/') e la divido inserendo rispettivamente il primo numero in 'quant' e il secondo in 'prezzo'

invece da

 if(isset($_POST['selezione'])){ 

fino alla fine controllo quali checkbox (della pagina html precedente) sono state spuntate e nel caso modifico il prezzo

<?php
$quantprez = $_POST[quantitaprezzo];
$prezzo = strstr($quantprez, '/');
$quant = str_replace($prezzo, ' ', $quantprez); 
$prezzo = substr($prezzo, 1);
$quant = trim($quant);
$prezzo = trim($prezzo);

if(isset($_POST['selezione'])){
foreach ($_POST['selezione'] as $index => $value)
    {
        if(isset($value))
        {
            switch ($index)
            {
                case 0: if(strcmp("$value","aggiungi")==0) {$aggiungi=true;$urgente=false;} else {$aggiungi=false; $urgente=true;} break;
                case 1: $urgente=true; break;
                default: break;
            }
        }
    }}
if($aggiungi)
$prezzo+=10;
if($urgente)
$prezzo+=$prezzo/5;
?>

chiedo umilmente il vostro aiuto!
grazie anticipatamente

[poyel]

Invece di usare strstr e fare tutti quei passaggi, usa la funzione explode del php:

$stringa = explode("/",$_POST[selezione]);

[8matt5]

Non ho idea del perchè... ma sono sicuro che ti sei complicato la vita per una cosa molto semplice.

Tutto quel papiro che hai scritto si può riassumere in poche righe.

Innanzitutto chiamare gli input selezione[] non ha alcuna utilità in questo modo. Poi fai dei controlli inutili e concettualmente sbagliati (ad esempio isset($value) è sempre vero; se $index è 0 $value è sempre uguale, quindi al condizione strcmp non ha senso...) e c'è un errore in $_POST[quantitaprezzo], ma probablimente avrai display_errors settato a Off e non te ne sei accorto.

Veniamo al dunque.

Io chiamerei i due checkbox urgente e aggiungi, quindi il php lo farei così:

$quantitaprezzo= explode('/',$_POST['quantitaprezzo']);
$prezzo = $quantitaprezzo[1];
$quant = $quantitaprezzo[0];

if(isset($_POST['aggiungi']))
$prezzo+=10;
if(isset($_POST['urgente']))
$prezzo+=$prezzo/5; 

[ddc90]

grazie per le riposte celerissime e utili! ho già ottimizzato l'obsoleto codice da me scritto
però il problema su firefox e explorer persiste