Parere su query

[cris.calleo]

Ciao a tutti.
Volevo prima di tutto ringraziarvi per l'aiuto, che è stato veramente importante a far si che iniziassi con il Php. E sinceramente é molto piacevole programmare.

Sto iniziando a scrivere qualcosa di più complesso e avrei bisogno di un vostro parere per assicurarmi che sono sulla strada giusta.
Ovviamente sono ancora al livello procedurale.

Ho scritto questa query. Che ne pensate?

// Mostra
$result = mysql_query("SELECT * FROM container WHERE container_id='$container_id'",$con);
$row = mysql_fetch_array($result, MYSQL_BOTH);
$container_id = $row['container_id'];
$container_cliente_id = $row['container_interno'];
$container_interno = $row['container_interno'];

$result = mysql_query("SELECT * FROM clienti WHERE cliente_id='$container_cliente_id'",$con);
$row = mysql_fetch_array($result, MYSQL_BOTH);

$cliente_nome = $row['cliente_nome'];

// Recupera Contenuto
$query="SELECT cc.contenuto_titolo FROM container_contenuto cc WHERE contenuto_container_id='$container_id'";

$result = mysql_query($query) or die(mysql_error());
$num = mysql_numrows($result);

mysql_close();

// Consegna
$data_atual  = date("Y-m-d"); 

if ($_REQUEST['azione'] == "consegna")
        {
            $data = ($_POST);
            $movimento_container_id = $container_id;
            $movimento_data = $data_attuale;
            $movimento_tipo = $data['movimento_tipo'];
                        
            $sqlQuery = "INSERT INTO movimento_conteiner   (
                                                                movimento_container_id,
                                                                movimento_data,
                                                                movimento_tipo
                                                                )
                                                                VALUES (
                                                                '$movimento_container_id',
                                                                '$movimento_data',
                                                                '$movimento_tipo'
                                                                )";
$result = MYSQL_QUERY($sqlQuery, $con);
mysql_query("UPDATE container SET container_situzione='In Consegna' WHERE container_id=".$container_id, $con); 

[cris.calleo]

Nessuna critica?

[fontanalorenzo]

Nell'ultima linea ti sei dimenticato di chiudere le virgolette che avevi aperto "".
Tu hai scritto:

mysql_query("UPDATE container SET container_situzione='In Consegna' WHERE container_id=".$container_id, $con); 

Corretto:

mysql_query("UPDATE container SET container_situzione='In Consegna' WHERE container_id=".$container_id", $con); 

Per il resto, ci sono sintassi molto più semplici per fare la stessa cosa, dovresti lavorare sulla semplificazione del codice

[cris.calleo]

Grazie Lorenzo,
Si, in effetti lavorerò per la semplificazione del codice dopo che lo script sarà pronto.
Diciamo che volevo assicurarmi che le query si montavano su per giù come sto facendo.
Chiama il dato da una tabella, scrivi nell'altra e altera una terza, tutto nella stessa query.

Grazie ancora Lorenzo.

[satifal]

Originariamente inviato da cris.calleo
...tutto nella stessa query.

Non so cosa sia per te una query (forse lo script che la contiene?) ma io di query ne leggo 5 e non una sola

[fontanalorenzo]

Originariamente inviato da cris.calleo
tutto nella stessa query.

Quando l'hai detto


Comunque ha ragione satifal, la query è solo la richiesta al database.

Per intenderci: Quelle stringhe rosse che vedi con scritto SELECT * FROM container etc...
UPDATE container SET etc...

[cris.calleo]

Si, mi sono espresso male.
É che ho iniziato a programmare da pochissimo.
Come già detto, mi serviva la critica a sincerarmi che sono sulla buona strada.