query eseguite 2 volte

[camp91]

ciao a tutti, vi metto di seguito il codice e vi espongo alla fine il problema^^

#####CODICE####

<?php
session_start();
require "funzioni.php";

if($_SESSION['username'] == ""){
echo "Non sei autorizzato ad entrare!";
exit();
}


$utente=$_SESSION['username'];
$nome=$_POST['oggetto'];
$prezzo=$_POST['prezzo'];
$monete="monete";
$query="SELECT * FROM utente WHERE nome='$utente'";
$db->esegui($query);

for($i=0; $i<$db->righe(); $i++)
{
$soldi=$db->leggi($i, $monete);
}



if($soldi<$prezzo)
{
echo "monete insufficienti";
}
else
{
$soldi=$soldi-$prezzo;
$query="INSERT INTO inventario VALUES('', '$utente', '$nome', '0')";
$db->esegui($query);
$query="UPDATE utente SET monete='$soldi' WHERE nome='$utente'";
$db->esegui($query);
echo "acquisto avvenuto con successo";
}


$db->disconnetti(); ?>

so che è un po' disordinato, cmq il problema è uno sostanzialmente... tutto è progettato perchè venga inserito in inventario un solo oggetto.. NON capisco perchè tutto il codice venga eseguito 2 volte.. infatti nel database ottengo 2 record.. 1 corretto e 1 che ha $nome=NULL... riuscite ad aiutarmi? spero di essere stato chiaro

[eiyen]

se parli dell'INSERT nella tabella inventario:

1) aggiungi subito dopo tale riga un print $query; per vedere la query generata
2) stampa eventuali altre variabili

p.s.: riposta il codice nei tag PHP del forum per maggior leggibilità!

[camp91]

queste piccole precauzioni le ho prese senza trovare l'errore.. appare tutto corretto tranne pe ril fatto che (tutto il codice credo) venga letto 2 volte... ambiguo.. cmq grazie per ora

[eiyen]

quali sono i campi di "inventario" nell'ordine?
qual è il campo che diventa "null" (mi pare il terzo di "inventario", ma qual è)?

riposto il codice per chiarezza:

<?php
    session_start();
    require "funzioni.php";
    if($_SESSION['username'] == "") {
        echo "Non sei autorizzato ad entrare!";
        exit();
    };
    $utente=$_SESSION['username'];
    $nome=$_POST['oggetto'];
    $prezzo=$_POST['prezzo'];
    $monete="monete";
    $query="SELECT * FROM utente WHERE nome='$utente'";
    $db->esegui($query);
    for ($i=0; $i<$db->righe(); $i++) {
        $soldi=$db->leggi($i, $monete);
    };
    if($soldi<$prezzo) {
        echo "monete insufficienti";
    } else {
        $soldi=$soldi-$prezzo;
        $query="INSERT INTO inventario VALUES('', '$utente', '$nome', '0')";
        $db->esegui($query);
        $query="UPDATE utente SET monete='$soldi' WHERE nome='$utente'";
        $db->esegui($query);
        echo "acquisto avvenuto con successo";
    };
    $db->disconnetti();
?>

[camp91]

in inventario i campi sono
id (autoincrementante)
utente (l'utente che acquista)
nome ($_POST['oggetto'] nonchè il nome dell'oggetto)
equipaggiato (un boolean che sia 1 o 0 a seconda che lo sia o meno)

con questo codice il mio risultato finale è il seguente

1° record: 1 michele spada 0 //che è corretto
2° record: 2 michele 0 //che non dovrebbe proprio esistere questo record

[eiyen]

ma aggiungendo un print $query dopo l'insert che query stampa a video?

[camp91]

stampa la query esatta.. guarda come ho risolto (anche se cmq ha poco senso che lo faccia)

<?php
    session_start();
    require "funzioni.php";
    if($_SESSION['username'] == "") {
        echo "Non sei autorizzato ad entrare!";
        exit();
    };
    $utente=$_SESSION['username'];
    $nome=$_POST['oggetto'];
    $prezzo=$_POST['prezzo'];
    $monete="monete";
    $query="SELECT * FROM utente WHERE nome='$utente'";
    $db->esegui($query);
    for ($i=0; $i<$db->righe(); $i++) {
        $soldi=$db->leggi($i, $monete);
    };
    if($soldi<$prezzo) {
        echo "monete insufficienti";
    } else {
        if(empty($nome)) //qui
        {;} //qui
        else{ // e qui
        $soldi=$soldi-$prezzo;
        $query="INSERT INTO inventario VALUES('', '$utente', '$nome', '0')";
        $db->esegui($query);
        $query="UPDATE utente SET monete='$soldi' WHERE nome='$utente'";
        $db->esegui($query);
        echo "acquisto avvenuto con successo";
        }
    };
    $db->disconnetti();
?>

cosa dici?

[eiyen]

mah... cmq pare che lo script sia richiamato due volte: la prima dal form e la seconda... non so... dovresti scoprire questo... prova per esempio a stampare in cima allo script il referer per vedere da dove arriva l'altra chiamata (un refresh, per esempio)... cmq io metterei un if (!empty(...)) in cima a tutto

[camp91]

non sono molto esperto in realtà.. cioè cosa intendi dire?? se per favore puoi illuminarimi^^ non ho cpt

[camp91]

ho messo il refer... la chiamata viene dal form e basta