require some advice

Evan_Morgan

i think im doing this the long way round but i need to be able to put a get variable in a post variable, im having problems. So far ive done it like this.

include ('dbconnect.php');

$addmatch = $_GET['addmatch'];

if(!isset($addmatch))
{

$sql = mysql_query("SELECT * FROM matches WHERE winner='tba'");

print "<table>";
while($ab = mysql_fetch_array($sql))
{
print "<tr><div align='center'><font face='tahoma' size='2'>\n<form action='matches.php?addmatch=".$ab['id']."' method='POST'>";
print "<td><input type='radio' name='".$ab['id']."' value='h'>\n<a href='matches.php?game=".$ab['id']."'>".$ab['team1']."</a></td>";
print " <td><input type='radio' name='".$ab['id']."' value='d'>draw </td>\n";
print "<td><input type='radio' name='".$ab['id']."' value='a'>\n<a href='matches.php?game=".$ab['id']."'>".$ab['team2']."</a><br></td>";
print "<td><input type='submit' value='bet'></td>\n";
print "</div></font></form>\n";
}
print "</table>";
}
else
{
print "$_POST["$addmatch"]";

eventually im going to use sessions to make a sportsbook betslip

Thanks

Evan

tekky

to me... this would seem like you are setting addmatch EVERYtime you load the page even via the submit on the form....

form action='matches.php?addmatch=".$ab['id']."' method='POST'>";

so your if isset($addmatch) would ALWAYS be true... I would remove the var assignment out of the ACTION param.... atleast judging by what you have coded... it doesnt need to be there... and could cause issues...

Evan_Morgan

but i thought i needed the if(isset) there because it will then show the form if nothing is set. It wont be set unless a form has been submitted because its a get var and wont be there if they just access matches.php, or thats what i thought. Do you have any ideas how to put the get variable inside the post variable. I think i have done it a hard way. Basically what i need to do is to register whatever they bet on, via the radio buttons, as a session variable, then i can list them on the betslip. Is there an easier way of doing it. Heres the current process. The while loop churns out every match from the db, it outputs each one in its own form. The get variable in the form action sends the id of the match, which is stored in the database and the radio button sends either h, d or a (standing for home draw or away). IS there an easier way of doing this?

Evan

tekky

post variables are done by submitting a form with METHOD=post....

the way you said it, yes your idea should work for now...

i thought you were only outputting 1 form, and using the addmatch to tell if were submitted or not...

add this for your else....

else 
{ 
  echo '<hr><br><pre>';
  print_r($_POST);
  echo '</pre>';
} // you also forgot to close your else...
?>

this will output an HR, then dump ALL the data from your POST variable to the browser so you can see exactly what your form is putting into the post var....

Evan_Morgan

Array
(
[9] => h
)

it outputs this, sorry about this but im new to php and i really dont have a clue what it means. Other than it being an array which i assume contains the 9 and h. This means it is sending the match id and the team, which is great but how to i access the array, what is it called?

evan

tekky

my guess would be

foreach $_POST as $key => $value
{
  $id = $key;
  $type = $value;
}

that would set $id to 9 $type to h

Evan_Morgan

you are a legend 😃, so just to clarify, i dont really need to be using get because post is sending everything.

tekky

that would be correct... however I would maybe give your ID's a name... instead of just the #... that way you can get like $POST['id'] and $POST['type'] or whatever you want to call them... just to make things more clear to read/handle

reason being, if you add ANY more POST vars into the forms... it will break the loop i gave you to work with the array you posted earlier