[RESOLVED] "If" statement - help needed

[Drabin]

Here's my current code and a URL to the php page I created:

<?php
if ($post == "yes") {

//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/
	$sql = "INSERT INTO pksignup (
name,
class,
level,
attendall
 $addl_insert_crit ) VALUES ( '$name',
'$class',
'$level',
'$attendall'
 $addl_insert_values )";
	if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
	$result = mysql_query($sql,db());
	if ($result == 1) {
	echo "You are now signed up!";
	echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
	} else {
	echo "Error inserting record (9994SQL)";
	}
//***** END INSERT SQL *****


} //END IF POST = YES FOR ADDING NEW RECORDS

if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (3) per class type<br>
will be allowed to attend the event,<br> 
with exception to Paladins, Priests,<br>
and Warriors which can be no more<br>
than (4).</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

//***** END ADD NEW ENTRY FORM*****

function db()
{
	$dbuser		= "testuser"; 
	$dbserver	= "testserver"; 
	$dbpass		= "testpass"; 
	$dbname		= "testdbname";

	//CONNECTION STRING
	$db_conn = mysql_connect($dbserver, $dbuser, $dbpass)
	or die ("UNABLE TO CONNECT TO DATABASE");
	mysql_select_db($dbname)
	or die ("UNABLE TO SELECT DATABASE");
	return $db_conn;
}//end function db
?>

Here's what I'm trying to do.

When a class type (let's say mage in this instance) reaches a count of 3 signed up, no one else can sign up using that class. So, if the following holds true:

joe - mage
john - mage
jack - mage

.. are signed up, and jane - mage tries to signup, they would not be allowed to. However, I'm not sure how to implement this. Any help would be appreciated.

Thanks,

[bpat1434]

Well, you'd have to do a quick query to count the current number of whatever type is currently signed up:

$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'";
$result = mysql_query($query);
$curr = mysql_result($result, 0, 'registered');

if($curr==3)
{
  die('Maximum '.$class.'s registration limit reached.');
}
else
{
  // Now do the insert into the database....
}

[Drabin]

Brett:

This worked out great. Now, I'm trying to tailor it a bit more. Here are the current vars that I am using and what I'm trying to accomplish.

<?php
if ($post == "yes") {

//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/

$druid = 3;
$hunter = 3;
$mage = 3;
$paladin = 4;
$priest = 4;
$rogue = 3;
$warlock = 3;
$warrior = 3;
$caplock = 20;

$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered'); 

if($curr >= 3) 
{ 
  die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 
} 
else 
{ 
	$sql = "INSERT INTO pksignup (
	name,
	class,
	level,
	attendall
	$addl_insert_crit ) VALUES ( '$name',
	'$class',
	'$level',
	'$attendall'
	$addl_insert_values )";
	if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
		$result = mysql_query($sql,db());
		if ($result == 1) {
		echo "You are now signed up!";
		echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
		} else {
	echo "Error inserting record (9994SQL)";
	}
}
//***** END INSERT SQL *****


} //END IF POST = YES FOR ADDING NEW RECORDS

if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (3) per class type<br>
will be allowed to attend the event,<br> 
with exception to Paladins, Priests,<br>
and Warriors which can be no more<br>
than (4).</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

//***** END ADD NEW ENTRY FORM*****

function db()
{
	$dbuser		= "leaving blank"; 
	$dbserver	= "leaving blank"; 
	$dbpass		= "leaving blank"; 
	$dbname		= "leaving blank";

	//CONNECTION STRING
	$db_conn = mysql_connect($dbserver, $dbuser, $dbpass)
	or die ("UNABLE TO CONNECT TO DATABASE");
	mysql_select_db($dbname)
	or die ("UNABLE TO SELECT DATABASE");
	return $db_conn;
}//end function db
?>

I want to use a switch case scenario so that if a class equals one of the numbers plugged into the top variables, it understands that instead. Right now, it's just looking for if the class being used on the form and if that is greater than or equal to 3, it stops the person from being able to insert the record. I need it to check per case of all the different types of classes.

In addition, the form registration limit should be set to a maximum of 20 persons. So, once an overall limit of 20 is reached (counting all classes), it should block access.

I'm presuming that this would be a 9 case scenario or an 8 case scenario with an exception using the 20 rule at the bottom here.

$druid = 3;
$hunter = 3;
$mage = 3;
$paladin = 4;
$priest = 4;
$rogue = 3;
$warlock = 3;
$warrior = 3;
$caplock = 20;

My only question is how to implement the case? I tried a few times but it's looking ugly. I was using something along the lines of:

<?php

if ($post == "yes") {

//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/

$druid = 3;
$hunter = 3;
$mage = 3;
$paladin = 4;
$priest = 4;
$rogue = 3;
$warlock = 3;
$warrior = 3;
$caplock = 20;

$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered'); 

$diemessage = die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 



switch ($class) {
	case 'Druid':
		if($curr >= $druid) 
		{ 
		$diemessage
		}
		break;
	case 'Hunter':
		if($curr >= $hunter) 
		{ 
		$diemessage
		}
		break;
	case 'Mage':
		if($curr >= $mage) 
		{ 
		$diemessage
		}
		break;
	case 'Paladin':
		if($curr >= $paladin) 
		{ 
		$diemessage
		}
		break;
	case 'Priest':
		if($curr >= $priest) 
		{
$diemessage		
		}
		break;
	case 'Rogue':
		if($curr >= $rogue) 
		{ 
		$diemessage
		}
		break;
	case 'Warlock':
		if($curr >= $warlock) 
		{ 
		$diemessage
		}
		break;
	case 'Warrior':
		if($curr >= $warrior) 
		{ 
		$diemessage
		}
		break;
	default:
		$sql = "INSERT INTO pksignup (
		name,
		class,
		level,
		attendall
		$addl_insert_crit ) VALUES ( '$name',
		'$class',
		'$level',
		'$attendall'
		$addl_insert_values )";
		if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
			$result = mysql_query($sql,db());
				if ($result == 1) {
					echo "You are now signed up!";
					echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
				} else {
				echo "Error inserting record (9994SQL)";
		}
}

?>

But, I haven't worked in the 20 limit I wanted above and I'm not sure if this is going to work at all.

[Drabin]

Okay I tested my script and changed a few things. Here is what I have now and what it is doing:

<?php



if ($post == "yes") {

//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/

$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered'); 

$diemessage = die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 



switch ($class) {
	case 'Druid':
		if($curr >= $druid) 
		{ 
		$diemessage;
		}
		break;
	case 'Hunter':
		if($curr >= $hunter) 
		{ 
		$diemessage;
		}
		break;
	case 'Mage':
		if($curr >= $mage) 
		{ 
		$diemessage;
		}
		break;
	case 'Paladin':
		if($curr >= $paladin) 
		{ 
		$diemessage;
		}
		break;
	case 'Priest':
		if($curr >= $priest) 
		{
		$diemessage;	
		}
		break;
	case 'Rogue':
		if($curr >= $rogue) 
		{ 
		$diemessage;
		}
		break;
	case 'Warlock':
		if($curr >= $warlock) 
		{ 
		$diemessage;
		}
		break;
	case 'Warrior':
		if($curr >= $warrior) 
		{ 
		$diemessage;
		}
		break;
	default:
		$sql = "INSERT INTO pksignup (
		name,
		class,
		level,
		attendall
		$addl_insert_crit ) VALUES ( '$name',
		'$class',
		'$level',
		'$attendall'
		$addl_insert_values )";
		if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
			$result = mysql_query($sql,db());
				if ($result == 1) {
					echo "You are now signed up!";
					echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
				} else {
				echo "Error inserting record (9994SQL)";
		}
}

//***** END INSERT SQL *****


} //END IF POST = YES FOR ADDING NEW RECORDS

if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (3) per class type<br>
will be allowed to attend the event,<br> 
with exception to Paladins, Priests,<br>
and Warriors which can be no more<br>
than (4).</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

//***** END ADD NEW ENTRY FORM*****

function db()
{
	//CONNECTION STRING
	$db_conn = mysql_connect($db_host, $db_user, $db_passwd)
	or die ("UNABLE TO CONNECT TO DATABASE");
	mysql_select_db($db_name)
	or die ("UNABLE TO SELECT DATABASE");
	return $db_conn;
}//end function db
?>

Now what happens is no matter what class I try to signup with, it says maximum limit reached etc. So, it's not separating each case but finding that one of the cases is true and die'ing.

I need to figure out how to match the case up to the class type and separate them.

[bpat1434]

Do this:

<?php

// Your query for class stuff, and declaration of max registrants

## Get total number of registrants (globally)
$gq = "SELECT COUNT(*) AS max FROM pksignup";
$rslt = mysql_query($gq);
$curr_global = mysql_result($rslt, 0, 'max');
##

$globalmax = 20;

switch($class)
{
  case 'druid':
    $max = 3;
    break;
  case 'hunter':
    $max = 3;
    break;
  case 'mage':
    $max = 3;
    break;
  case 'paladin':
    $max = 4;
    break;
  case 'priest':
    $max = 4;
    break;
  case 'rogue':
    $max = 3;
    break;
  case 'warlock':
    $max = 3;
    break;
  case 'warrior':
    $max = 3;
    break;
}

if($max == $curr)
{
  // Max class users registered
}
else if($globalmax == $curr_global)
{
  // Global maximum users registered
}
else
{
  // They can register!!
}

[Drabin]

Not working mate. Here's the code I used:

if ($post == "yes") 
{
	//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
	//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/

## Get total number of registrants (globally) 
$gq = "SELECT COUNT(*) AS max FROM pksignup"; 
$rslt = mysql_query($gq); 
$curr_global = mysql_result($rslt, 0, 'max'); 
## 

$globalmax = 20; 

switch($class) 
{ 
case 'druid': 
	$max = 3; 
	break; 
case 'hunter': 
	$max = 3; 
	break; 
case 'mage': 
	$max = 3; 
	break; 
case 'paladin': 
	$max = 4; 
	break; 
case 'priest': 
	$max = 4; 
	break; 
case 'rogue': 
	$max = 3; 
	break; 
case 'warlock': 
	$max = 3; 
	break; 
case 'warrior': 
	$max = 3; 
	break; 
} 

if($max >= $curr) 
	{ 
		echo "My curr is $curr";
		echo "My max is $max";
		//die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 
	} 
else if($globalmax >= $curr_global) 
	{ 
		die('Maximum RAID LIMIT of 20 registrations has been reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>');
	} 
else 
	{ 
$sql = "INSERT INTO pksignup (
name,
class,
level,
attendall
$addl_insert_crit ) VALUES ( '$name',
'$class',
'$level',
'$attendall'
$addl_insert_values )";
if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
	$result = mysql_query($sql,db());
	if ($result == 1) 
	{
	echo "You are now signed up!";
	echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
	} 
	else 
	{
	echo "Error inserting record (9994SQL)";
	}
}
} 

When I put the echos in the first die() it basically gives me back no values. So, it's not counting up properly for some reason. Without the echos, it hits the first die, regardless of whether or not there are no classes signed up.

The table in question has 4 fields: name, class, level, and attendall. I appreciate all of the help mate. This is what what was happening to me on my case as well. It probably has something to do with the SQL or where the cases are hitting?

Not sure.. hrm.

[Drabin]

bpat1434 wrote:

Do this:

<?php

// Your query for class stuff, and declaration of max registrants

## Get total number of registrants (globally)
$gq = "SELECT COUNT(*) AS max FROM pksignup";
$rslt = mysql_query($gq);
$curr_global = mysql_result($rslt, 0, 'max');
##

$globalmax = 20;

switch($class)
{
  case 'druid':
    $max = 3;
    break;
  case 'hunter':
    $max = 3;
    break;
  case 'mage':
    $max = 3;
    break;
  case 'paladin':
    $max = 4;
    break;
  case 'priest':
    $max = 4;
    break;
  case 'rogue':
    $max = 3;
    break;
  case 'warlock':
    $max = 3;
    break;
  case 'warrior':
    $max = 3;
    break;
}

if($max == $curr)
{
  // Max class users registered
}
else if($globalmax == $curr_global)
{
  // Global maximum users registered
}
else
{
  // They can register!!
}

I see two potential issues but am not sure how to fix them. Here's what I see that may be causing problems looking at your example:

$curr in the statement if($max == $curr) isn't defined so I'm not sure what $curr is referring to.

Also, "SELECT COUNT(*) AS max FROM pksignup"; is going to count all items in the table which means it would pick up names, class, level, etc. so I thought about changing it to "SELECT COUNT(class) AS max FROM pksignup";

However, it's not working well. After altering a few scripts, it now shows everything maxed out to 20.

Here's the new adjusted script. Let me know if I'm doing something wrong here. Thanks again for all of your help. I'm learning from your examples 🙂

<?php
if ($post == "yes") 
{
	//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
	//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/
## Get total number of class (locally)
$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered');

## Get total number of registrants (globally) 
$gq = "SELECT COUNT(class) AS max FROM pksignup"; 
$rslt = mysql_query($gq); 
$curr_global = mysql_result($rslt, 0, 'max'); 
## 

$globalmax = 20; 

switch($class) 
{ 
case 'druid': 
	$max = 3; 
	break; 
case 'hunter': 
	$max = 3; 
	break; 
case 'mage': 
	$max = 3; 
	break; 
case 'paladin': 
	$max = 4; 
	break; 
case 'priest': 
	$max = 4; 
	break; 
case 'rogue': 
	$max = 3; 
	break; 
case 'warlock': 
	$max = 3; 
	break; 
case 'warrior': 
	$max = 3; 
	break; 
} 

if($max >= $curr) 
	{ 
		die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 
	} 
else if($globalmax >= $curr_global) 
	{ 
		die('Maximum RAID LIMIT of 20 registrations has been reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>');
	} 
else 
	{ 
$sql = "INSERT INTO pksignup (
name,
class,
level,
attendall
$addl_insert_crit ) VALUES ( '$name',
'$class',
'$level',
'$attendall'
$addl_insert_values )";
if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
	$result = mysql_query($sql,db());
	if ($result == 1) 
	{
	echo "You are now signed up!";
	echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
	} 
	else 
	{
	echo "Error inserting record (9994SQL)";
	}
}
} 
//***** END INSERT SQL *****


if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (4) per class type<br>
will be allowed to attend the event.</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

//***** END ADD NEW ENTRY FORM*****

function db()
{
	$dbuser		= "blank"; 
	$dbserver		= "blank"; 
	$dbpass		= "blank"; 
	$dbname		= "blank";

	//CONNECTION STRING
	$db_conn = mysql_connect($dbserver, $dbuser, $dbpass)
	or die ("UNABLE TO CONNECT TO DATABASE");
	mysql_select_db($dbname)
	or die ("UNABLE TO SELECT DATABASE");
	return $db_conn;
}//end function db

?>

[bpat1434]

COUNT(*) will count the total number of returned rows from the query... alternatively you can use mysql_num_rows() via PHP, but why make PHP do what mySQL can?

The $curr variable referred to your previous code where you said that there were $curr number of * class registered.... that's where that came from...

Take a few minutes and rethink your logic:

if($max >= $curr)
        {
        }
    else if($globalmax >= $curr_global)
        { 

That says:
if $max (3) is greater than or equal to $current (1):
[indent]Kill the script[/indent]
if $global_max (20) is greater than or equal to $current_global (10):
[indent]Kill the script[/indent]

Might want to siwtch those around... (the numeric representations are for help, they mean nothing).

Other than that, should work (oh, and change COUNT(class) to COUNT(*))....

[Drabin]

Thanks Brett, I appreciate the very informative feedback once again.

Here's my current script (tweaked from your suggestions) and what it is doing:

<?php
if ($post == "yes") 
{
	//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
	//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/
## Get total number of class (locally)
$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered');

## Get total number of registrants (globally) 
$gq = "SELECT COUNT(*) AS max FROM pksignup"; 
$rslt = mysql_query($gq); 
$curr_global = mysql_result($rslt, 0, 'max'); 
## 

$globalmax = 20; 

switch($class) 
{ 
case 'druid': 
	$max = 3; 
	break; 
case 'hunter': 
	$max = 3; 
	break; 
case 'mage': 
	$max = 3; 
	break; 
case 'paladin': 
	$max = 4; 
	break; 
case 'priest': 
	$max = 4; 
	break; 
case 'rogue': 
	$max = 3; 
	break; 
case 'warlock': 
	$max = 3; 
	break; 
case 'warrior': 
	$max = 3; 
	break; 
} 

if($globalmax >= $curr_global) 
	{ 
		die('Maximum RAID LIMIT of 20 registrations has been reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>');
	} 
else if($max >= $curr) 
	{ 
		die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 
	}
else 
	{ 
$sql = "INSERT INTO pksignup (
name,
class,
level,
attendall
$addl_insert_crit ) VALUES ( '$name',
'$class',
'$level',
'$attendall'
$addl_insert_values )";
if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
	$result = mysql_query($sql,db());
	if ($result == 1) 
	{
	echo "You are now signed up!";
	echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
	} 
	else 
	{
	echo "Error inserting record (9994SQL)";
	}
}
} 
//***** END INSERT SQL *****


if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (4) per class type<br>
will be allowed to attend the event.</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

//***** END ADD NEW ENTRY FORM*****

function db()
{
	$dbuser		= "blank"; 
	$dbserver	= "blank"; 
	$dbpass		= "blank"; 
	$dbname		= "blank";

	//CONNECTION STRING
	$db_conn = mysql_connect($dbserver, $dbuser, $dbpass)
	or die ("UNABLE TO CONNECT TO DATABASE");
	mysql_select_db($dbname)
	or die ("UNABLE TO SELECT DATABASE");
	return $db_conn;
}//end function db

?>

The table in question only has 17 records or rows and yet when I try to add anyone, it says that the maximum of 20 has been reached. So, it's hitting the first die with only 17 rows in the table.

Anything I could be doing wrong here?

[Drabin]

Wow, you have to bonk me on the head in the morning. I misunderstood what you meant! You meant that I'm basically saying >= when those should be swapped! Going to try it now and see what happens. Wow, I'm brain saturated this week. 🙂

[Drabin]

Here's the final script and it works perfectly:

<?php

// Establish our connection and select our database
$link = mysql_connect($db_host, $db_user, $db_passwd) or die ("Could not connect to desired database.");
mysql_select_db($db_name) or die ("Could not select desired database.");

if ($post == "yes") 
{
	//THEN WE ARE ADDING A NEW RECORD SUBMITTED FROM THE FORM
	//REPLACE THE FIELD CONTENTS SO THEY DON'T MESS UP YOUR QUERY

/*-- SECTION: 9994SQL --*/
## Get total number of class (locally)
$query = "SELECT COUNT(class) AS registered FROM pksignup WHERE class='".$class."'"; 
$result = mysql_query($query); 
$curr = mysql_result($result, 0, 'registered');

## Get total number of registrants (globally) 
$gq = "SELECT COUNT(*) AS max FROM pksignup"; 
$rslt = mysql_query($gq); 
$curr_global = mysql_result($rslt, 0, 'max'); 
## 

$globalmax = $caplock; 

switch($class) 
{ 
case 'Druid': 
	$max = $druid; 
	break; 
case 'Hunter': 
	$max = $hunter; 
	break; 
case 'Mage': 
	$max = $mage; 
	break; 
case 'Paladin': 
	$max = $paladin; 
	break; 
case 'Priest': 
	$max = $priest; 
	break; 
case 'Rogue': 
	$max = $rogue; 
	break; 
case 'Warlock': 
	$max = $warlock; 
	break; 
case 'Warrior': 
	$max = $warrior; 
	break; 
} 

if($curr_global >= $globalmax) 
	{ 
		die('Maximum RAID LIMIT of 20 registrations has been reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>');
	} 
else if($curr >= $max) 
	{ 
		// echo "Current is '".$curr."' and Max is '".$max."'";
		die('Maximum '.$class.'s registration limit reached.<br><br><a href="http://praetorianknights.mmorpg-gaming.net/zgsignups/">Click Here to return to main listing</a>'); 
	}
else 
	{ 
$sql = "INSERT INTO pksignup (
name,
class,
level,
attendall
$addl_insert_crit ) VALUES ( '$name',
'$class',
'$level',
'$attendall'
$addl_insert_values )";
if ($sql_debug_mode==1) { echo "<BR>SQL: $sql<BR>"; }
	$result = mysql_query($sql);
	if ($result == 1) 
	{
	echo "You are now signed up!";
	echo "<br><br><a href=\"http://praetorianknights.mmorpg-gaming.net/zgsignups/\">Click Here to return to main listing</a>\n";
	} 
	else 
	{
	echo "Error inserting record (9994SQL)";
	}
}
} 
//***** END INSERT SQL *****


if (!$post) {
//THEN WE ARE ENTERING A NEW RECORD
//***** BEGIN ADD NEW FORM*****
	/*-- SECTION: 9994FORM --*/
	?>
	<form method="post" action="<?php echo $thispage; ?>?proc=New&post=yes&<?php echo $pagevars; ?>">
	<center>
	<TABLE>
	<TR>
		<TD>
			<B>NAME:</B>
		</TD>
		<TD>
					<INPUT TYPE='name' NAME='name' SIZE='20'>		</TD>
	</TR>
		<TR>
		<TD>
			<B>CLASS:</B>
		</TD>
		<TD>
		<SELECT NAME='class' SIZE='1'>
		<option>Druid</option>
		<option>Hunter</option>
		<option>Mage</option>
		<option>Paladin</option>
		<option>Priest</option>
		<option>Rogue</option>
		<option>Warlock</option>
		<option>Warrior</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>LEVEL:</B>
		</TD>
		<TD>
		<SELECT NAME='level' SIZE='1'>
		<option>58</option>
		<option>59</option>
		<option>60</option>
		</SELECT>		</TD>
	</TR>
		<TR>
		<TD>
			<B>ATTENDALL:</B>
		</TD>
		<TD>
				<INPUT TYPE='checkbox' NAME='attendall' VALUE='checked'>		</TD>
	</TR>
	 <TR>
	 <TD>
	 </TD>
	 <TD>
	  <INPUT TYPE="SUBMIT" VALUE="Signup" NAME="submit">
	  <INPUT TYPE="RESET" VALUE="Reset" NAME="reset">
	 </TD>
	 <TD>
	 Note: No more than (4) per class type<br>
will be allowed to attend the event.</td>
 </TR>
 <TD>
 	</TABLE>
	</CENTER>
	<BR>
	</FORM>
<?php
} //END if post=""

?>

[bpat1434]

Great!!! glad to hear you got it working.... figured it would be something small like that. Kudos to re-reading my post(s) to get a clearer picture...

[Drabin]

Hey Brett,

I really want to thank you for helping me out with this. It takes a very patient and understanding person like yourself to help us "newbies" get through all this code, heh. The nice thing is, people like myself really appreciate it because I'm the type of person that learns from how someone else puts in code.

In the past week I've learned from others, like yourself, on how to create arrays, case scenarios, and a few tips on how to optimize code so it's a little cleaner and manageable by using an include config file.

If I can learn this much in a week from you guys, I can only imagine how much I'll learn in a month! 🙂

Thanks so much again.