• User

    Che sbadato, caspita! Non avevo pensato che essendo numeri potevo togliere gli apici, pur avendo compreso il motivo del NULL non funzionante!

    Stasera provo poi tornerò per un feedback, per ora grazie mille!!!


  • Consiglio Direttivo

    Attendiamo il responso con ansia. 😄


  • User

    Uff, niente, ancora errore.

    La query che passa php è

    [php]UPDATE qm_optometria SET ID_Cliente='0', ID_Occhiale='6', Data='2009-01-14', IntTabo='TABO', SferaLDx=, SferaLSx=0.25, CilindroLDx=-6.00, CilindroLSx=-2.75, AsseLDx=75, AsseLSx=80, SferaVDx=0.00, SferaVSx=0.00, CilindroVDx=0.00, CilindroVSx=0.00, AsseVDx=0, AsseVSx=0, DPL=75, HPup=0, NoteOcchiali='' WHERE ID_Occhiale='6'[/php]

    ma dà errore

    You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' SferaLSx=0.25, CilindroLDx=-6.00, CilindroLSx=-2.75, AsseLDx=75, ' at line 7Che cosa sbaglio ancora? :mmm:

    Grazie per l'aiuto enorme che mi stai dando!!!


  • Consiglio Direttivo

    Prova a togliere dalla query questo:

    SferaLDx=,
    

  • User

    Ma esiste un sistema per farlo in automatico secondo te, o devo proprio farlo a mano?

    Nel senso, nella funzione io gli passo i parametri, ma se il campo è vuoto gli passo NULL, che però in php non si può passare, ossia è vuoto.

    Così la funzione

    [php]function occhialiUpdate($ID_Cliente,$ID_Occhiale,$Data,$IntTabo,$SferaLDx,$SferaLSx,$CilindroLDx,$CilindroLSx,$AsseLDx,$AsseLSx,$PrismaLDx,$PrismaLSx,$BaseLDx,$BaseLSx,$SferaVDx,$SferaVSx,$CilindroVDx,$CilindroVSx,$AsseVDx,$AsseVSx,$PrismaVDx,$PrismaVSx,$BaseVDx,$BaseVSx,$DPL,$HPup,$Diametro,$NoteOcchiali)
    {
    global $link, $occhiali_table;
    $query = "UPDATE ".$occhiali_table."
    SET
    ID_Cliente='$ID_Cliente',
    ID_Occhiale='$ID_Occhiale',
    Data='$Data',
    IntTabo='$IntTabo',
    SferaLDx=$SferaLDx,
    SferaLSx=$SferaLSx,
    CilindroLDx=$CilindroLDx,
    CilindroLSx=$CilindroLSx,
    AsseLDx=$AsseLDx,
    AsseLSx=$AsseLSx,
    SferaVDx=$SferaVDx,
    SferaVSx=$SferaVSx,
    CilindroVDx=$CilindroVDx,
    CilindroVSx=$CilindroVSx,
    AsseVDx=$AsseVDx,
    AsseVSx=$AsseVSx,
    DPL=$DPL,
    HPup=$HPup,
    NoteOcchiali='$NoteOcchiali'
    ";

        $query=$query." WHERE 
        ID_Occhiale='$ID_Occhiale'";
        $result = mysql_query ($query, $link);
        if (! $result )
            die ("errore di aggiornamento in ".$query." - ".mysql_error() );
    }[/php]se $SferaLDx è vuoto, mette nella query **$SferaLDx=,** proprio perchè non c'è nulla nella variabile.
    

    Io dovrei per forza nella funzione esaminare per ogni campo, in qualche modo,

    [php]if ($SferaLDx == "")
    $SferaLDx = NULL;[/php]Come posso fare nella funzione a fare un controllo del genere?

    Dovrei fare una cosa tipo

    [PHP].........................
    IntTabo='$IntTabo',".
    if (SferaLDx=="") ."SferaLDx=NULL,".
    else."SferaLDx=$SferaLDx,[/PHP]

    per ogni campo?


  • Consiglio Direttivo

    Beh al recupero dati potresti fare

    [php]if ($_POST['sferadx']) { $sferadx=$_POST['sferadx']; } else { $sferadx=null;}[/php]


  • User

    Ma questo lo faccio, infatti ad inizio pagina vi è un controllo

    [php]
    if ($form["SferaLDx"] == "")
    $form["SferaLDx"] = NULL;
    if ($form["SferaLSx"] == "")
    $form["SferaLSx"] = NULL;
    if ($form["CilindroLDx"] == "")
    $form["CilindroLDx"] = NULL;
    if ($form["CilindroLSx"] == "")
    $form["CilindroLSx"] = NULL;
    .........................................
    [/php]quando io però poi chiamo la funzione con

    [php]
    if (isset($actionflag) && $actionflag=="Update") {
    if ( $message == "") {
    $result = occhialiUpdate($ID_Cliente,
    $ID_Occhiale,
    $form["Data"],
    $form["IntTabo"],
    $form["SferaLDx"],
    $form["SferaLSx"],
    $form["CilindroLDx"],
    $form["CilindroLSx"],
    ...........................
    $form["NoteOcchiali"]);
    $message = "Dati Aggiornati!";
    }
    }[/php]la funzione di cui sopra occhialiUpdate mette $SferaLDx=, e non $SferaLDx=NULL,.


  • User

    Forse ci sono, secondo me dovrei fare una cosa del tipo

    [PHP]
    $query = "UPDATE ".$occhiali_table."
    SET
    ID_Cliente='$ID_Cliente',
    ID_Occhiale='$ID_Occhiale',
    Data='$Data',
    IntTabo='$IntTabo',"

        if ($SferaLDx)
            {
                $query=$query."SferaLDx=$SferaLDx,";
            }
        $query=$query."
    
        SferaLSx=$SferaLSx,
        CilindroLDx=$CilindroLDx,
        CilindroLSx=$CilindroLSx,
        AsseLDx=$AsseLDx,
        AsseLSx=$AsseLSx,
        SferaVDx=$SferaVDx,
        SferaVSx=$SferaVSx,
        CilindroVDx=$CilindroVDx,
        CilindroVSx=$CilindroVSx,
        AsseVDx=$AsseVDx,
        AsseVSx=$AsseVSx,
        DPL=$DPL,
        HPup=$HPup,
        NoteOcchiali='$NoteOcchiali'
        ";
    
        $query=$query." WHERE 
        ID_Occhiale='$ID_Occhiale'";
        $result = mysql_query ($query, $link);
    

    [/PHP]

    con il controllo centrale (da fare per ogni variabile numerica)

    [PHP]
    if ($SferaLDx)
    {
    $query=$query."SferaLDx=$SferaLDx,";
    }
    [/PHP]

    che aggiunge alla query il campo se è valorizzato, se no passa oltre.

    Solo che così dà errore. Voi cosa ne pensate? Qualcuno di buona volontà è per caso in grado di correggere il mio codice?

    Grazie!


  • User

    Risolto, direi. Mancava un ;

    Il codice corretto dovrebbe essere (perdonatemi i tagli)

    [PHP]
    .....................

        IntTabo='$IntTabo',";
        if ($SferaLDx)
            {
                $query=$query."SferaLDx=$SferaLDx,";
            }
        else
            {
                $query=$query."SferaLDx=NULL,";
            }
        $query=$query."
        SferaLSx=$SferaLSx,
    
        .....................
    

    [/PHP]

    Se avete idee su come ottimizzare il tutto fatemi sapere, altrimenti direi che la mia richiesta si conclude qui!

    Grazie mille! :ciauz:


  • Consiglio Direttivo

    Eccomi, scusa per il ritardo, giornata pienotta. 🙂

    Beh, direi che va bene come soluzione, non incorri nei pericoli di campi null 😉