[RESOLVED] Looped Radio Post

[GeoLoK8R]

Yes, I am a newbie, and I'm sure someone has a simple answer... My formatting is terrible, and I apologize.

Here is my problem. Can't seem to get the value of the radio buttons to post. Here is where I get frustrated... I know it must be something simple. If I echo the $theid, they are the correct numbers. I am assuming that the problem is in my SQL statement in the process.php but, thats why I'm here. Here is part of the code from the 'for' loop. I receive no errors. It just doesn't update the 'pick' to a 'A' or 'H' in the database.

<input name="pick[<? echo "$theid" ?>]" type="radio"  value="A">

<?	
	echo "$theawayteam";
	echo "  @  ";
	echo "$thehometeam";
?>
<input name="pick[<? echo "$theid" ?>]" type="radio"   value="H">
<?
echo " $thevalue  ";
?>
<br>
<br>
<?

}
// Find first and last games ID numbers for week.	
$thestart =($theid+1)-$num_rows1;
$theend = ($theid);
//echo "$thestart ";
//echo " $theend";

?>
<input type="hidden" name="subnflpicks" value="1"> 
<input type="submit" name="nflpicks" value="Submit picks" />
</form>
<?

Then in my 'process.php' I have:

      /* Pick submission attempt */
	  for ($d=$_POST[$thestart]; $d<$_POST[$theend]; $d++) {

    # setup SQL statement 

	$sql = "UPDATE nflpicks SET pick=$_POST[pick][$d] WHERE game =$d AND user = ('$session->username')"; 



    #execute SQL statement 
	mysql_query($sql);

	}

	header("Location: ".$session->referrer);
	}

Again, I apologize for the terrible formatting. Any help someone can give me would be greatly appreciated!

[bpat1434]

In your process.php script, where does $thestart amd $theend come from?

[GeoLoK8R]

They both come from the original script. You can see them at the bottom. I know that those numbers are correct. They could be a number between 1-267 depending on the query.

[bpat1434]

Yes... I see you define $thestart and $theend, but I don't see how they get to the process.php page 😉 You don't send them in a hidden field, you don't set them to a session or cookie. How exactly do you get them to go over? If they don't go over, your loop probably is running once with $d being "0" 🙁 Have you checked in process.php that $thestart and $theend are in fact what you expect?

[GeoLoK8R]

Okay, I see your point. I edited the followint to the original script. Does this look correct?

// Find first and last game ID's for the week.
$thestart1 =($theid+1)-$num_rows1;
$theend1 = ($theid);
//echo "$thestart ";
//echo " $theend";

?>
<input type="hidden" name="thestart" value="<? echo $thestart1 ?>">
<input type="hidden" name="thesend" value="<? echo $theend1 ?>">
<input type="hidden" name="subnflpicks" value="1">
<input type="submit" name="nflpicks" value="Submit picks" />
</form>
<?

And thank you for you help!

[GeoLoK8R]

I did notice the "thesend" error and corrected it.

[bpat1434]

Okay, that looks okay. Now the only thing left is to get the values in process.php by doing:

$thestart = $_POST['thestart'];
$thesend = $_POST['thesend'];

[GeoLoK8R]

So the process.php should look like this?

   function procNFLpicks(){
      global $session, $form;


$thestart = $_POST['thestart'];
$theend = $_POST['theend']; 


  /* Pick submission attempt */
  for ($d=$thestart; $d<$theend; $d++) {

    # setup SQL statement 

	$sql = "UPDATE nflpicks SET pick=$_POST[pick][$d] WHERE game =$d AND user = ('$session->username')"; 



    #execute SQL statement 
	mysql_query($sql);

	}

[bpat1434]

yup, pretty-much.

[GeoLoK8R]

I'll keep messing with it. Still no go. I think there is still something wrong with my SQL statement, and I only say this becasue I get no errors, but I does log me off the site which its what happened when I had other statement errors. (in other areas) instead of 'or die' Like I said, I'm new and I apologize for my ignorance.

[GeoLoK8R]

Got it! Had to do the same with the other variable and ad '$picked'

$thestart = $POST['thestart'];
$theend = $POST['theend']+1;

/ Pick submission attempt /
for ($d=$thestart; $d<$theend; $d++) {

$picked = $_POST['pick'][$d];

setup SQL statement

$sql = "UPDATE nflpicks SET pick = '$picked' WHERE game = $d AND user = ('$session->username')";

[bpat1434]

Fantastic. Don't forget to mark this thread as resolved.