[RESOLVED] User Functions

[xxdalexx]

I'm working on designing a large form, to which there are 4 different entries in the form that are a series of check boxes, and I'm basically trying to make the code more manageable for later on. Here is how I defined a function for this:

function ogame()
{
echo "<center><table><tr>";
$i = 0;
$sql = "SELECT * From lgame";

$result = mysql_query($sql, $dbcnx);



while($row = mysql_fetch_array($result))

  {
  	$i++;

$name = $row['name'];

echo "<td><input type='checkbox' name='ogame' value='$name' />$name</td>";

if ($i == 2)
	{
		$i = 0;
		echo "</tr><tr>";
	}


  }
echo "</tr></table></center>";
}

The problem I'm running into is when I call the function, it doesn't do anything. Any help would be appreciated, I'm thinking maybe it needs to be defined as something other then a function, but not sure. Everything works when not in a function, but not when I try doing it this way. Thanks.

[xxdalexx]

Well, I figured it out, and here's how if this helps anyone in the future. First I figured out it was only the mysql query inside of the function that was not working. After some playing around, figured out if I opened the connection at the beginning of the function and closed it at the end it worked, but for 5 different functions just for that page alone, that seemed like to much work.

So it turns out that if I removed the $dbcnx, it would use same connection from prior on the page and that worked, but if you wanted to leave it in, you have to define everything for that connection inside of the function. Doesn't necessarily make sense to me, but hey it works.

[dalecosp]

It's called "scope"; variables from outside a function aren't considered to be local to the function unless they've been declared as global.

At some point, consider marking your thread as resolved.

EDIT: see http://www.php.net/manual/en/language.variables.scope.php

[xxdalexx]

Thank you for that.