[RESOLVED] Checking if a record exists

planethax

Ok, I have a few page that connect to db, get info display, user selects 7 drivers then input back into db in a different table.

Now I need to check to see the username (or even id? ) is already in db, if not proceed if it is, display a message.
The middle code sec tion is what I already have, and the first and last code sections are what I have been trying to get to work;

$sql = "SELECT * FROM `user_drivers`" ;

if ($row['userid'] = $userid ) ;   // here I am trying to have, if current userid is not in row then proceed
{

include_once('db_conn.php');
$sql = "SELECT * FROM `nascar_standings` ORDER BY driver ASC LIMIT 65";
$rs = mysql_query($sql);
$username = me('username');
$userid = me('id');
$i = 0;
$sel_text = "";
$column = "</tr><tr>";
while($row = mysql_fetch_assoc($rs)){
  $label = $row['driver'];
  $sel_text .= "<td><input type='checkbox' name='chks[]'> $label </td>";
    if (++$i % 4 == 0)
    {
        $sel_text .= $column;
    }
}
$sel_text .= "</tr><tr><td colspan =4 valign = center><input type='submit' value='Validate & Submit' name='validate' onclick='if(!checkCount()) return false;'></td></tr></table></form>";

$sel_text .= "<script type='text/javascript'>

function checkCount(){
    var mincheck = 7;
    var testform = document.getElementById('frmChk');
    var items = testform.getElementsByTagName('input');
    var count = 0;

    for (var i=0; i < items.length-1; i++)
    {
        if (items[i].type == 'checkbox' && items[i].checked)
        {
            count++;
        }
    }

    if(count != mincheck)
    {
        alert('You must select ' + mincheck + ' Drivers');
        return false;
    }
    return true;
}

</script>";
$tpl -> AssignArray(array('selteam' => $sel_text));

}
else
{ // already in db message here
}

Am I even close here?

Thanx

planethax

This is what I have tried, but page is empty,
Thought this would be close?

include_once('db_conn.php');
$sql = "SELECT * FROM `user_drivers`" ;

if ($row['userid'] = $userid ) ;   // here I am trying to have, if current userid is not in row then proceed
{ 
$sql = "SELECT * FROM `nascar_standings` ORDER BY driver ASC LIMIT 65";
$rs = mysql_query($sql);
$username = me('username');
$userid = me('id');
$i = 0;
$sel_text = "";
$column = "</tr><tr>";
while($row = mysql_fetch_assoc($rs)){
  $label = $row['driver'];
  $sel_text .= "<td><input type='checkbox' name='chks[]' value='$label'> $label </td>";
    if (++$i % 4 == 0)
    {
        $sel_text .= $column;
    }
}
$sel_text .= "</tr><tr><td colspan =4 valign = center><input type='submit' value='Validate & Submit' name='validate' onclick='if(!checkCount()) return false;'></td></tr></table></form>";

$sel_text .= "<script type='text/javascript'>

function checkCount(){
	var mincheck = 7;
	var testform = document.getElementById('frmChk');
	var items = testform.getElementsByTagName('input');
	var count = 0;

	for (var i=0; i < items.length-1; i++)
	{
		if (items[i].type == 'checkbox' && items[i].checked)
		{
			count++;
		}
	}

	if(count != mincheck)
	{
		alert('You must select ' + mincheck + ' Drivers');
		return false;
	}
	return true; 
}

</script>";
}
else
{ $sel_text = "Already picked Team" ;// already in db message here
} 
$tpl -> AssignArray(array('selteam' => $sel_text));

planethax

Well I am Half way there, I have been able to figure out how to check if userid exists, but I can not get the ELSE statement to work, the following is what is working

include_once('db_conn.php');
$username = me('username');
$userid = me('id');
$sql_check = mysql_query("SELECT * FROM user_drivers WHERE userid='$userid'");
$sql_check_num = mysql_num_rows($sql_check);
if($sql_check_num == 0){
$sql = "SELECT * FROM `nascar_standings` ORDER BY driver ASC LIMIT 65";
$rs = mysql_query($sql);
$i = 0;
$sel_text = "";
$column = "</tr><tr>";
while($row = mysql_fetch_assoc($rs)){
  $label = $row['driver'];
  $sel_text .= "<td><input type='checkbox' name='chks[]' value='$label'> $label </td>";
    if (++$i % 4 == 0)
    {
        $sel_text .= $column;
    }
}
$sel_text .= "</tr><tr><td colspan =4 valign = center><input type='submit' value='Validate & Submit' name='validate' onclick='if(!checkCount()) return false;'></td></tr></table></form>";

$sel_text .= "<script type='text/javascript'>

function checkCount(){
	var mincheck = 7;
	var testform = document.getElementById('frmChk');
	var items = testform.getElementsByTagName('input');
	var count = 0;

	for (var i=0; i < items.length-1; i++)
	{
		if (items[i].type == 'checkbox' && items[i].checked)
		{
			count++;
		}
	}

	if(count != mincheck)
	{
		alert('You must select ' + mincheck + ' Drivers');
		return false;
	}
	return true; 
}

</script>";
}

But page breaks when trying to add the ELSE statement,
What is the correct way of adding the else statement?
Thanx.

else
{
$sel_text = "Sorry, you have already selected a team, you can only select ONE and it can not be change. <br> Please go to your Team Page. <br> If you need assistance, please contact an administrator."
};

coldwerturkey

you wrote:

else 
{ 
$sel_text = "Sorry, you have already selected a team, 
you can only select ONE and it can not be change. <br> 
Please go to your Team Page. <br> If you need assistance, 
please contact an administrator."[U][COLOR="Red"]_[/COLOR][/U]
}[B][COLOR="Red"][U];[/U][/COLOR][/B]

You should turn errors on so php can point you in the right direction. google for how-to

planethax

Ahhh, a misplaced ; again lol.

I had put the
// Report all PHP errors (see changelog)
error_reporting(E_ALL);

lines in the code but it doesn't do anything, I am thinking it has something to do with my Webserver provider not turning it on 🙁

Thanx for your help!

coldwerturkey

ini_set('display_errors', 1);
error_reporting(E_ALL);

may do it for you

rincewind456

You don't want the ; at the end of the else clause. Other than that there appears no reason why it shouldn't work.

You may want to consider changing your MySQL select statements so that you don't retrieve all rows when you only need to know if any record exists so I would change this

$sql_check = mysql_query("SELECT * FROM user_drivers WHERE userid='$userid'");

to this

$sql_check = mysql_query("SELECT userid FROM user_drivers WHERE userid='$userid'");

You also should have some form of error handling if the SQL statement fails.

In general it is also a better idea to state all the columns you require in the SQL statement even if you need all of them, as it is supposed to be marginally quicker than using the * operator.