cgm225 Posted February 21, 2008 Share Posted February 21, 2008 How would you "combine" the two sections of code below to run all at once? They are are basically the same except for minor differences in variable name. I want to do this the best way possible. /*//////Adding any new images to the MySQL gallery database//////////////////////////////////////////// */ $number_of_image_files = count(glob("$server_photo_album_location/$album/*.*")); $number_of_mysql_image_entries = mysql_num_rows(mysql_query("SELECT * FROM images WHERE album = '$album_id'")); if ($number_of_image_files != $number_of_mysql_image_entries) { include_once "gallery.add.images.inc.php";} /*//////Adding any new videos to the MySQL gallery database//////////////////////////////////////////// */ $number_of_video_files = count(glob("$server_video_album_location/$album/*.*")); $number_of_mysql_video_entries = mysql_num_rows(mysql_query("SELECT * FROM videos WHERE album = '$album_id'")); if ($number_of_video_files != $number_of_mysql_video_entries) { include_once "gallery.add.videos.inc.php";} Quote Link to comment Share on other sites More sharing options...
neylitalo Posted February 21, 2008 Share Posted February 21, 2008 That looks pretty efficient to me - you're dealing with two separate entities there (videos and images), so it's good that you have their logic separated. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.