PHP Developers Network

A community of PHP developers offering assistance, advice, discussion, and friendship.
It is currently Sat Jul 11, 2020 9:21 pm

All times are UTC - 5 hours

Post new topic Reply to topic  [ 2 posts ] 
Author Message
 Post subject: Code Improvement
PostPosted: Tue Jan 24, 2012 12:02 pm 
Forum Newbie

Joined: Tue Jan 24, 2012 11:30 am
Posts: 1
Hi All,

I'm new to PHP and I'm working on creating a personal movie database using PHP & MYSQL. As of now it is working fine as per my expectation but I I'm using lots of repeated code to insert or select data from database. Below are 3 codes which are being used to query multiple tables to either insert or select data. I just need to replace the "cast" word with "producer" or "director" or "writer" and I can use the same query.

What is the better way improve the code so that I can reduce the no. of lines for the code.

1st Code is for insert in the table.
Syntax: [ Download ] [ Hide ]

//******************************************* CAST UPDATE *******************************************//

$sql = array();
    foreach( $cast as $row ) {
        $imdb = mysql_real_escape_string($row['imdb']);
        $name = mysql_real_escape_string($row['name']);
       $sql[] = '("'.$imdb.'", "'.($name).'" )';
    $result = 'INSERT INTO cast (imdb, name) VALUES '.implode(',', $sql).' ON DUPLICATE KEY UPDATE lastupdate = NOW()';
        mysql_query($result) or die('log error with' .mysql_error());

//******************************************* TITLE_CAST UPDATE *******************************************//  

        $sql_query = "INSERT INTO title_cast (id_title, id_cast)
                                        SELECT $title_id, hi.cast_id
                                        FROM cast_inserts hi
                                        WHERE hi.conn_id = CONNECTION_ID()"
        if (!mysql_query($sql_query,$con))
          die('Error: ' . mysql_error());
                $sql_del = "DELETE FROM cast_inserts WHERE conn_id = CONNECTION_ID()";
        if (!mysql_query($sql_del,$con))
          die('Error: ' . mysql_error());


2nd code for select query

Syntax: [ Download ] [ Hide ]

$sql_cast = "SELECT *
                          INNER JOIN title ON (title_cast.id_title =
                          INNER JOIN `cast` ON (title_cast.id_cast = `cast`.id)
                          title_cast.id_title = '$movieid'"
$result_cast = mysql_query($sql_cast) or die('log error with' .mysql_error());

          <div class="desc">CAST</div>
            <?php while ($row = mysql_fetch_array($result_cast)) {?>
                                $id = $row['imdb'];
                                $name = $row['name'];
                                $img = $row['photo_localurl'];
                                $poster = str_replace("./", "lib/", $img);
//              $poster_cast = str_replace("../", "", $img);
            <div class="details1"><table><tr>
                <td class="personimg"><img src="<?php if(!empty($poster)){echo $poster;}else{echo $dft_img;}?>" width="32" height="42" alt="<?php echo $name;?>" ></td>
                <?php echo "<td class='personname'><a href='persons.php?det=cast&perid=".$id."'> ".$name." </a></td>";?>
                <td class="personrole">"personrole"</td></tr></table>        
            <?php } ?>

3rd query again for select

Syntax: [ Download ] [ Hide ]
        $qAsCast = "SELECT
                          INNER JOIN title_cast ON ( = title_cast.id_title)
                          INNER JOIN `cast` ON (title_cast.id_cast = `cast`.id)
                          `cast`.imdb = $imdb"
$rAsCast = mysql_query($qAsCast);
        echo "<div class='sidecont'>";
                                $total = mysql_num_rows($rAsCast);
                                        if($total == '0' ){ echo "$msg";}
                                        while ($row = mysql_fetch_array($rAsCast)) {
                                                $id = $row['id'];
                                                $title = $row['title'];
                                                $year = $row['year'];
                                                $poster = $row['poster'];
                                                $poster = str_replace("./", "lib/", $poster);
                                                $rating = $row['ratings'];
                                                echo "<table><tr><td>";
                                                echo "<div id='a'>";
                                                echo "<div id='b'>".$title.'</div>';
                                                echo "<div id='c'>".$year.'</div>';
                                                echo "<div id='f'>".rat($rating).$rating.'</div>';
                                                echo "<div id='d'><a href='movies.php?movieid=$id'><img src='$poster' alt='' border='1' align='center' width='107' height='157' /></a></div>";
                                                echo "</div>";
                                                echo "</td></tr></table>";
        echo "<div>"

 Post subject: Re: Code Improvement
PostPosted: Fri Jan 27, 2012 9:49 am 
DevNet Master
User avatar

Joined: Sun Feb 15, 2009 12:08 pm
Posts: 2794
Location: .za

“Don’t worry if it doesn’t work right. If everything did, you’d be out of a job.” - Mosher’s Law of Software Engineering

Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 2 posts ] 

All times are UTC - 5 hours

Who is online

Users browsing this forum: No registered users and 4 guests

You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Jump to:  
Powered by phpBB® Forum Software © phpBB Group