help with vote 1 of 2 random pictures

AlexCuria

First of all hello to everybody (this is my first post).

I'm starting (for the 2nd time in my life) to learn programming.
Last time was 15 years ago and those days i learned assembler, basic and C+. I actually was quite good in assembler, didn't like basic and never got enough practice with C+.

Well enough with the presentation, now to the problem:

I have 2 random numbers generated.
These 2 random numbers are the image_id of the picture ni the DB.

Now i need to know how to make the pictures 'clickable' and update the counters of both pictures as follows:

Clicked picture gets $totalhits + 1 AND $votes + 1
Non-clicked picture only $totalhits + 1

After this another set of 2 random pictures should display.

I'm totally confused on how to approach this.

Any ideas?

matt_4013

Well, I assume you have a field in your database called totalhits, and votes?

If so, this is how I would do it.

	// Check to see if the user has voted for a picture
	// $_GET is the variables from the URL, so by adding them onto the link, we can see what the user voted for
	if(isset($_GET['vote_id'])) {

	// If they have, update the database. 

	// This one is for the voted picture
	$sql = "UPDATE tablename SET totalhits=totalhits + 1, vote=vote + 1 WHERE id='".$_GET['vote_id']."'";
	// Run the query and error check to see if it was updated
	if(($result = mysql_query($sql)) === false) {

		// If the query failed, output this
		die("<h1>ERROR!</h1>".mysql_error());
	}

	// This query is just to update the loser
	$sql = "UPDATE tablename SET totalhits=totalhits + 1 WHERE id='".$_GET['loser_id']."'";
	if(($result = mysql_query($sql)) === false) {

		// If the query failed, output this
		die("<h1>ERROR!</h1>".mysql_error());
	}

	// Now the 2 have been updated, go back to normal and display your pictures
}

// Assuming $pic1 and $pic2 set elsewhere, you said you already got the random part

echo "<a href=\"".$_SERVER['PHP_SELF']."?vote_id=".$pic1."&amp;loser_id=".$pic2."\"><img src=\"".$path_to_pic1."\"></a>";
echo "<a href=\"".$_SERVER['PHP_SELF']."?vote_id=".$pic2."&amp;loser_id=".$pic1."\"><img src=\"".$path_to_pic2."\"></a>";

Hope tha helps!
Matt

AlexCuria

Thanks for your time in replying.

Yes i do have the table fields totalhits and votes (actually image_hits and image_votes).

I understand most of the code you posted but really cant see how to implement the isset($_GET) thing.

I think it's probably better you look at what i've done till now and tell me if my logic was totally wrong or if there is a better way to do what i intend.

<?php

include 'header.php';

// Content 
include 'admin/config.inc';

$db=mysql_connect("localhost",$uname,$pwd);
mysql_select_db($dbname,$db);
$result=mysql_query("select * from 4images_images");
$row=mysql_num_rows($result);

srand((double)microtime()*1234567);
$random1=rand(1,$row);
$random2=rand(1,$row);
if ($random2 = $random1)
	{
	$random2=rand(1,$row);
	}

$sqlresult = mysql_query("SELECT * FROM 4images_images WHERE (image_id = $random1) OR (image_id = $random2)");
$maxwidth = "380";

$imgbattle = $row["image_battle"];
$imgbattle = $imgbattle + 1;

$HTML="\n<tr>\n";

while ($row = mysql_fetch_array($sqlresult))
	{
	$imageid=$row["image_id"];
	$category=$row["cat_id"];
	$imgrating=$row["image_rating"];
	$imghits=$row["image_hits"];
	echo "Image_ID:",$imageid;
	$query="UPDATE 4images_images SET image_battle='$imgbattle' WHERE image_id=$imageid";
	mysql_query($query) or die(mysql_error());

$imagename=$row["image_name"];
$mediafile=$row["image_media_file"];
$mediaURL="../media/data/media/$categoria/$mediafile";
$imagehw = GetImageSize($mediaURL);
$imagewidth = $imagehw[0];
$imageheight = $imagehw[1];
$imgorig = $imagewidth;
if ($imagewidth > $maxwidth) {
	$imageprop=($maxwidth*100)/$imagewidth;
	$imagevsize= ($imageheight*$imageprop)/100 ;
	$imagewidth=$maxwidth; 
	$imageheight=ceil($imagevsize);
}
$HTML.="<td align=\"center\">";
$HTML.="<img src=\"../media/data/media/$category/$mediafile\" border=\"0\" width=$imagewidth height=$imageheight><br/>$imagename<br/>Number of battles: $imagebattle<br/>Actual Rating: $imgrating</td>\n";
}

$HTML.="	</tr><br/><br/>\n";
echo $HTML;

// End of Content
include 'footer.php';
?>

What i still dont understand is how to update the database depending on which picture i click. I display each picture in the while loop so i don't know what to put in the <a href> part.

There are probably lots of other errors in the code but
please bear with me because i'm still learning. :rolleyes:

pointsplat

There are a couple different ways of going about this, being as im working on a hot or not script now we're doing pretty much the same thing 🙂

What I need from you in order to answer your question is what variable you are looking to use in order to set the winner of the "battle".

Looking at your code it appears you would need $imageid to to set the score approriately. (I am guessing you have given this a trial run so should know what variable you need in order to select the winner)

As far as the isset($_GET[]) goes that just checks to see if the variable you are "getting" exists. The get command itself works like this....
http://www.yourdomainhere.com/blah.php?username=God

if you do
$username = $_GET['username'];
echo $username;

it will output God

This means all you would need to do for your script is create a hyperlink around your image like this
http://www.yourdomainhere.com/yourscript.php?imageid=yourimage
and add the $imageid=$_GET['imageid']; to the top of the script.

Wow I am really no good at this explaining business, if you would like a copy of my source for the hot or not voter I would be more then happy to share if it would help.

P.S.
I strongly advise against using GET if you are doing something that a) needs to be clean and b) you want to look professional this is because using get you are limited to putting variables into the URL....this can get very ugly and will be stored in the users recent URLs menu. I use POST because it a) makes it harder for "dumb" users to vote for themselves a million times and is cleaner cause variables are "hidden"

AlexCuria

Hello thx 4 your reply.

What I need from you in order to answer your question is what variable you are looking to use in order to set the winner of the "battle".

Well i still don't have any variable for this, just fields in the database table with counters for total battles, wins and losses. Everytime a picture is shown total battles increments 1. If the picture is chosen the wins increments 1 aswell. If not then the losses increments 1.
(I'm not very good explaining either) hahaha.

Thx also for offering the hot or not source but I prefer taking the challenge myself to get something done by myself.
It will take ages since i'm a newbie but it's the only way i can learn 🙁

I think i'm doing something wrong at the logical thinking because i thought i could do the picture selection and UPDATE in the same php file without passing any variables in the URL.

I mean is this possible?

-> Display 2 random pictures.
-> Click on picture A or B UPDATES all counters in the database and display a new set of random pictures.

Just something i'm thinking as i write: maybe if i put the pictures inside a FORM and let the picture act as if it would be the submit button ???????

As you might see i'm really novice and what i'm trying to do is probably too complicated for me at the beginning. :rolleyes:

pointsplat

im looking over the code now and im a little confused as to how the database is setup, what you are trying to do is a seemingly very simple task just need to figure out how the database is organized before I start messing with it 🙂

If you go the chance post a reply with what fields you have in the database and what each means. It seems from looking at the code that there are alot more then you have mentioned previously in the post. Once you get that up I can rewrite it and hopefully in a clearer way that you understand how the code works 🙂

Grant

AlexCuria

Hi again,

well the database fields i use in the 4images_images table are:

image_id = unique image identifier (autoincrement) (i.e. 459)
cat_id = category to which the image belongs (i.e. Basketball)
image_name = the name of the image (i.e. M.Jordan)
image_description = description of the image (i.e. Jordan in the air)
image_date = date image has been added to the database
image_active = if image is visible or hidden (1 true or 0 false)
image_media_file = the filename with extension (i.e. DSCF1233.jpg)
image_battle = nº of times image has been displayed
image_winner = nº of battles won
image_looser = nº of battles lost

There are some other irrelevant fields like comments and thumbnail but i think i've mentioned the most important ones.