• User Attivo

    Si il libro fa pena, ma le versione italiana, mancano pezzi di codice e alcuni argomenti sono incompleti 😞 la versione inglese e' completa

    i dati puntano al file commit.php, li ci sono le query

    [PHP]<?php
    $db = mysql_connect('localhost', 'bp6am', 'bp6ampass') or
    die ('Unable to connect. Check your connection parameters.');
    mysql_select_db('moviesite', $db) or die(mysql_error($db));
    ?>
    <html>
    <head>
    <title>Commit</title>
    </head>
    <body>
    <?php
    switch ($_GET['action']) {
    case 'add':
    switch ($_GET['type']) {
    case 'movie':
    $query = 'INSERT INTO
    movie
    (movie_name, movie_year, movie_type, movie_leadactor,
    movie_director)
    VALUES
    ("' . $_POST['movie_name'] . '",
    ' . $_POST['movie_year'] . ',
    ' . $_POST['movie_type'] . ',
    ' . $_POST['movie_leadactor'] . ',
    ' . $_POST['movie_director'] . ')';
    break;
    }
    break;
    case 'edit':
    switch ($_GET['type']) {
    case 'movie':
    $query = 'UPDATE movie SET
    movie_name = "' . $_POST['movie_name'] . '",
    movie_year = ' . $_POST['movie_year'] . ',
    movie_type = ' . $_POST['movie_type'] . ',
    movie_leadactor = ' . $_POST['movie_leadactor'] . ',
    movie_director = ' . $_POST['movie_director'] . '
    WHERE
    movie_id = ' . $_POST['movie_id'];
    break;
    }
    break;
    }

    if (isset($query)) {
    $result = mysql_query($query, $db) or die(mysql_error($db));
    }
    ?>
    <p>Done!</p>
    </body>
    </html>[/PHP]

    accetto consigli :smile5:


  • User Attivo

    capito il problema, se riempio tutti i campi l'errore non viene visualizzato

    questo e' strano, nel database ho impostato tutti a Null, tranne movie_name

    image


  • User Attivo

    il codice funziona solo se compilo tutti i dati nella form, se lascio un campo/i vuoti genera errore :arrabbiato:


  • User

    L'errore a cui ti riferisci viene dal file commit.php nella parte in cui lanci la query di INSERT.
    Dovresti controllare cosa ricevi dal form in caso il campo non venga compilato. L'errore sta lì!


  • User Attivo

    @FedeMasiero said:

    L'errore a cui ti riferisci viene dal file commit.php nella parte in cui lanci la query di INSERT.
    Dovresti controllare cosa ricevi dal form in caso il campo non venga compilato. L'errore sta lì!

    questo

    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 ' , )' at line 8
    

  • User

    Esattamente, proprio questo! Deriva dal pezzo di codice in commit.php dove hai la query di inserimento: INSERT INTO ecc...


  • User Attivo

    in alto c'e' il codice di **commit.php
    ho controllato, sembra tutto a tosto :bho:
    **


  • Moderatore

    Ciao Alex, prova a riscrivere la query, come te la ho scritta io.

    [PHP]
    <?php
    $db = mysql_connect('localhost', 'bp6am', 'bp6ampass') or
    die ('Unable to connect. Check your connection parameters.');
    mysql_select_db('moviesite', $db) or die(mysql_error($db));
    ?>
    <html>
    <head>
    <title>Commit</title>
    </head>
    <body>
    <?php
    switch ($_GET['action']) {
    case 'add':
    switch ($_GET['type']) {
    case 'movie':
    $query = "INSERT INTO
    movie
    (movie_name, movie_year, movie_type, movie_leadactor,
    movie_director)
    VALUES
    ('" . $_POST['movie_name'] . "',
    " . $_POST['movie_year'] . ",
    " . $_POST['movie_type'] . ",
    " . $_POST['movie_leadactor'] . ",
    " . $_POST['movie_director'] . ")";
    break;
    }
    break;
    case 'edit':
    switch ($_GET['type']) {
    case 'movie':
    $query = "UPDATE movie SET
    movie_name = '" . $_POST['movie_name'] . "',
    movie_year = " . $_POST['movie_year'] . ",
    movie_type = " . $_POST['movie_type'] . ",
    movie_leadactor = " . $_POST['movie_leadactor'] . ",
    movie_director = " . $_POST['movie_director'] . "
    WHERE
    movie_id = " . $_POST['movie_id'];
    break;
    }
    break;
    }

    if (isset($query)) {
    $result = mysql_query($query, $db) or die(mysql_error($db));
    }
    ?>
    <p>Done!</p>
    </body>
    </html>
    [/PHP]


  • User Attivo

    Grazie Stefano 🙂
    provo

    ps: ricordo che questo e' un codice di test per testare le funzionalita', il database originale ha altri campi e altre relazioni, da riscrivere le query


  • User Attivo

    non funziona se si lasciano campi vuoti, restituisce questo errore (il solito)

    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 ' , )' at line 8

    line 8
    $query = 'SELECT

    se si inseriscono tutti i dati nei campi tutto procede bene senza errori

    :rollo:


  • User Attivo

    ok, lasciamo perdere il codice sopra, e' sbagliata la struttura

    sto sperimentando un'altro codice, questo e per UPDATE (poi cambio il parametro in INSERT, DELETE)

    qualcosa e' sbagliato, mi restituisce la query nella pagina, ma nel database effettua l'update correttamente

    ps: ho aggiunto un'altra query per role, ma non sono sicuro se e' strutturata giusta

    [PHP]<div>
    <label for="credited">Credited</label>
    <input type="text" class="textBox" placeholder="Text input" name="credited">
    </div><br>

        <div>
            <label for="role">Role</label>
            <input type="text" class="textBox" placeholder="Text input" name="role">
        </div><br>
        
        <div>
        
        
        <button type="submit" class="button">Add!</button></div><br>
    
        </form>
        <?php
            if (isset($_GET['movieid'])) {
                $movieid = $_GET['movieid'];
                $actorid = $_GET['actorid'];
                $credited = $_GET['credited'];
                $role = $_GET['role'];
                
                $sql = "UPDATE 
                film_actor 
                SET credited = '" . $credited . "' 
                , role = '" . $role . "' 
                WHERE film_actor.actor_id = '" . $actorid ."' 
                AND 
                film_actor.film_id = '" . $movieid ."'";
                
                echo $sql;
    
                $connection = new mysqli("localhost", "exadmin", "expass", "xxx");
                $result = $connection->query($sql);
                
            }
        ?>
    
    </div>[/PHP]
    

    image


  • Moderatore

    C'è un "echo $sql" nel codice, quindi ti stampa la query a video.
    Ti basta rimuoverla o commentarla


  • User Attivo

    ho risolto cosi 🙂

    [PHP]// esecuzione della query
    if (!$connection->query($sql)) {
    echo "Errore della query: " . $connection->error . ".";
    }else{
    echo "Dati <b> " . $credited . " " . $role . " </b> inseriti correttamente nel database."; $sql;[/PHP]

    grazie Stefano :smile5:

    mi aiuti a risolvere il mistero dell'altro thread

    http://www.giorgiotave.it/forum/php/260153-upload-foto-multiple-errore-nel-codice.html


  • User Attivo

    la query precedente funziona benissimo

    adesso mi serve una per inserire il titolo e collegarlo all'attore, ho fatto molti tentativi ma non riesco a risolvere :?:bho::rollo:

    Database struttura: 3 tabelle

    actor: <------Table
    actor_id
    nome
    foto

    film: <------Table
    film_id
    movie_title
    year

    film_actor: <------Table
    actor_id
    film_id

    **questo e' il codice completo
    **
    [PHP]<!DOCTYPE html>
    <html>
    <head> <title> I4 - Add Movie to Actor Relation </title> </head>
    <link rel="stylesheet" href="S1.css">
    <body>

    <div align="center" style="margin-left: 18%; padding: 1px 15px;">
        <h1> Add Movie to Actor Relation</h1>
        
        <form method = 'GET' action='#'> <div class="form-group"><label for="movieid">Movie Title   </label><option value=NULL> </option>
        <div>
            <label for="movie_title">Movie Title</label>
            <input type="text" class="textBox" placeholder="Text input" name="movie_title">
        </div><br>
        
        
        <div>
            <label for="credited">Credited</label>
            <input type="text" class="textBox" placeholder="Text input" name="credited">
        </div><br>
        
        <div>
        
        
        <div>
            <label for="year">Year</label>
            <input type="text" class="textBox" placeholder="Text input" name="year">
        </div><br>
        
        <label for="actorid">Actor   </label><select class="form-control" name='actorid'><option value=NULL> </option> 
    
        <?php 
            $servername = "localhost";
            $username = "xxx";
            $password = "xxx";
            $dbname = "xxx";
            
            $connection = new mysqli($servername, $username, $password, $dbname);
            
            $dataCells = array();
    
            $sql = "select concat(actor_id,' ', nome, ' ', credited, ' ') , actor_id from actor;";
            
            $result = $connection->query($sql);
            while ($resultProperties = mysqli_fetch_field($result)) {
                $name = $resultProperties->name;
                array_push($dataCells, $name); 
            }
            
            $colIndex = 0;
            $actorName;
            while ($row = mysqli_fetch_array($result)) {
                foreach ($dataCells as $cell) {
                    if ($colIndex == 0){
                        $actorName = $row[$cell];
    
                    }
                    else if ($colIndex == 1) {
                        echo "<option value=" . $row[$cell] . ">" . $actorName;
                        echo "</option>";
                    }
    
                    $colIndex++;
                }
                $colIndex = 0;
            }
        ?>
        </select><br><br>
        
        <div>
        
        
        <button type="submit" class="button">Add!</button></div><br>
    
        </form>
    
        <?php
            if (isset($_GET['movieid'])) {
                $movieid = $_GET['movieid'];
                $movie_title = $_GET['movie_title'];
                $credited = $_GET['credited'];
                $year = $_GET['year'];
                $actorid = $_GET['actorid'];
                
                
                
                $sql = "insert into film values 
                movie_title = '". $movie_title . "'
                , credited = '" . credited . "'
                , year = '" . $year ."'
                WHERE actor.actor_id = '" . $actorid ."' 
                AND 
                film.film_id = '" . $movieid ."'";
                    //echo $sql;
                    $result = $connection->query($sql);
                
                
                $connection = new mysqli("localhost", "exadmin", "expass", "xxx");
                $result = $connection->query($sql);
                
                // esecuzione della query
    

    if (!$connection->query($sql)) {
    echo "Errore della query: " . $connection->error . ".";
    }else{
    echo "Dati <b> " . $movie_title . " " . $also_known . " </b> inseriti correttamente nel database."; $sql;
    }

            }
    

    ?>

    </div>
    

    </body>
    </html>

    [/PHP]


  • User

    Ciao Alex, scusa la domanda che mi viene spontanea, hai studiato un po' di database relazionali prima di creare le tre tabelle sopra?
    Hai ben capito cos'è una "relazione molti a molti" e come viene implementata con le tabelle in un DB?
    In caso, ti consiglio i tutorial di W3C, sono molto chiari su questo aspetto e potrebbero risolverti molti problemi.

    Anyway, la tabella film_actor si chiama "tabella di congiunzione" e serve appunto a tener traccia dei collegamenti dei vari attori all'interno di un film e dei vari film recitati da un attore (per l'appunto la relazione molti a molti)!
    Quindi, per riuscire a collegare un attore ad un film devi andare ad eseguire una insert nella tabella film_actor con i due ID, in questo modo:

    INSERT INTO film_actor VALUES (id_film_ricevuto, id_attore_ricevuto);

    Ovviamente devi ricevere gli ID del film e dei vari attori da collegare.
    Potresti ricevere gli ID degli attori al momento dell'inserimento di un nuovo film, oppure anche in una pagina successiva che andrai ad implementare dopo aver fatto inserire le informazioni essenziali del film.
    In caso ricevessi tutte le informazioni in un solo momento, prima dovresti lanciare una query per inserire un nuovo film, poi, attraverso l'apposita funzione php mysql_insert_id recuperi l'ID dell'ultimo inserimento (in questo caso il film) e procedi all'inserimento dei vari ID degli attori ricevuti nella tabella di congiunzione film_actor!

    Spero di essere stato chiaro!


  • User Attivo

    Ciao Federico 🙂
    il concetto sulle query realazionali lo conosco,in teoria, ma nella pratica poi quando devo andare a creare le query sono un disastro,
    ci sono cose che non riesco ancora a fare,

    chiedo lumi a chi e' piu' esperto di me in materia

    spero di migliorare in futuro 😞

    il problema qui sta collegare il titolo, che io voglio inserire manualmente, con gli attori,
    ovviamente tutta la compilazione dei dati film deve stare nella stessa pagina, per l'ID attore che e' presente non si pone il problema,
    il titolo film pero' deve essere ancora inserito, quindi non esite ancora l'ID,


  • User

    OK Alex, allora devi utilizzare (come ti ho scritto sopra) la funzione di php (tu usi il 5.6) che si chiama:

    mysql_insert_id($conn);

    Grazie a questa funzione riesci a recuperare l'ultimo ID che è stato inserito nel DB.

    Allora, presupponendo di avere un form dove puoi compilare tutti i campi del film ed anche di selezionare gli attori, riceverai tutti i dati nel tuo script di conferma.
    A quel punto dovrai fare più query: una per inserire il film e altre query per collegare ogni attore al film.

    Quindi i passaggi sono:
    QUERY INSERT FILM -> nella tabella film;
    Recupero l'ultimo ID inserito con la funzione mysql_insert_id($conn) -> Sarà l'ID del tuo film appena inserito;
    UNA QUERY PER OGNI ATTORE DA COLLEGARE AL FILM -> nella tabella film_actor;

    Implementalo e vedi che funzionerà! 😉


  • User Attivo

    Federico grazie per le tue spiegazioni, mah.. non potresti inserirmi il pezzo di codice che manca nel post, prima che ci arrivo passeranno giorni :arrabbiato:
    come ho specificato nei primi thread ho cominciato a interessarmi al php da circa 2 mesi, ancora ho molto da imparare 😞


  • User Attivo

    Risolto :yuppi::fumato:

    una curiosita, quale sono gli input type corretti delle form per questi campi?
    io ho inserito questi, sono giusti?

    **Tipo

    varchan = <input type="text"

    year (4) = <input type="year"

    date = <input type="date"

    text = <input type="text"
    **[LEFT]

    enum=** <input type=" " ******questo non capisco ****
    [/LEFT]
    [LEFT]

    [/LEFT]


  • User

    Ciao Alex, se ti mettevo il codice avresti fatto semplicemente copia/incolla. Così invece hai analizzato la mia spiegazione e la hai implementata. Ora hai veramente imparato qualcosa di nuovo! 😉

    Per quanto riguarda gli input sono corretti.
    Per la questione ENUM, invece, sono dei valori nel DB che sono impostati inizialmente da te. Quindi nel momento in cui li inserisci dovranno rispettare le tue specifiche di creazione.
    Puoi utilizzare il type text nel campo di input, però devi controllare quando lo ricevi che sia formattato in modo corretto. Oppure, cosa più semplice, fai una tendina di selezione con gli ENUM disponibili e l'utente seleziona quello desiderato!