help with if ( ) statement

[msnhockey]

Below I have some code that I am trying to fix within the stuff I have colored red..

before I had the code like this if(($i == 4 || $i == 5 ) && !mysql_numrows($ssresult)){
and the $i == 4 and $i == 5 represent when a game goes into overtime and are only displayed if the game goes that far
$i == 4 is 1st Overtime and $i == 5 is 2nd overtime.

recently i had a game go into 3rd overtime but the code i originally had didn't allow for me to display that when the game went that far so i tried adding in $i == 6 for 3rd overtime and with the code below that is red it only shows the 1st overtime and 3rd overtime and skips the 2nd overtime..

been trying different things but not getting the right combo...also been looking at this thread http://php.net/manual/en/control-structures.if.php

 $player_scores = array();
$teams_scores = array(0, 0);
for($i=1;$i<=6;$i++){
	$ssresult = mysql_query("SELECT * FROM `goalinfo` WHERE gameid = '" . $id . "' AND goalperiod = '" . $i . "' ORDER BY goaltime");
	if($ssresult){

$test_ss = 1;

	[SIZE=4][/SIZE][COLOR="#FF0000"]if(($i == 4 || $i == 5 || $i == 6) && !mysql_numrows($ssresult)){[/COLOR]
		$test_ss = 0;
		}
if($test_ss == 1)
{		
	switch($i){
		case 1:$periodname = "1ST";break;
		case 2:$periodname = "2ND";break;
		case 3:$periodname = "3RD";break;
		case 4:$periodname = "1ST OVERTIME";break;
		case 5:$periodname = "2ND OVERTIME";break;
		case 6:$periodname = "3RD OVERTIME";break;
		}

here are different things i have tried so far
if(($i == 4 || $i == 5 || $i == 6) && !mysql_numrows($ssresult)){

or

if(($i == 4 || $i == 5) && ($i == 6) && !mysql_numrows($ssresult)){

or

if(($i == 4) || ($i == 5) || ($i == 6) && !mysql_numrows($ssresult)){

or

if(($i == 4 && $i == 5 && $i == 6) && !mysql_numrows($ssresult)){

[traq]

what does [man]mysql_numrows/man do? maybe you actually meant to use [man]mysql_num_rows/man?

--------------------------------------------------
# If at all possible, you should avoid using the mysql_* functions. #
Existing code should be updated to avoid performance and security problems.

php.net (official PHP website) wrote:

Warning
This extension is deprecated as of PHP 5.5.0, and is not recommended for writing new code as it will be removed in the future.
Instead, either the mysqli or PDO_MySQL extension should be used.
See also the MySQL API Overview for further help while choosing a MySQL API.

[/COLOR]

[msnhockey]

since i didn't originally write this part of the code I am assuming that the mysql_numrows() is there to get how many of the periods it needs to display.. i will look at the MySQL_num_rows() and see if I should be changing that part

[msnhockey]

i am assuming i will need to update the MySQL_numrows to this mysqli_stmt_num_rows()

[traq]

msnhockey;11027635 wrote:

since i didn't originally write this part of the code I am assuming that the mysql_numrows() is there to get how many of the periods it needs to display.. i will look at the MySQL_num_rows() and see if I should be changing that part

I would suspect it's a mistake, but look through your existing code and see if [font=monospace]mysql_numrows[/font] is defined anywhere (that would be a horrible naming choice for a user function, but hey - could happen).

msnhockey;11027637 wrote:

i am assuming i will need to update the MySQL_numrows to this mysqli_stmt_num_rows()

If you're talking about switching from ext/mysql to ext/mysqli, there a bit more to it than just adding "i" to all the function names.

In any case, I'd recommend getting this to work as-is before trying to update everything.

[msnhockey]

i see what you mean with just adding in the i

also i think the MySQL_numrows is a mistake... i skimmed through about 20 of the more popular pages i use and mysql_num_rows is used alittle but i see the $row lots

[Weedpacket]

Are you sure there's even a record for "second overtime"?

And what will you do if and when you get a fourth overtime, if the rules of the game allow for that possibility?

[msnhockey]

i can add in a fourth overtime but since i don't have a game with it i cant try it, but the way its supposed to work is if say the game went into 3rd overtime and there are no goals or penalaties in say Overtime 1 or overtime 2 it just says NO GOALS or NO PENALTIES.

in reality there could be as many overtimes as possible until someone scores a goal. usually 1 is needed, 2 overtimes =sometimes and 3 and over are very rare

this part of the code.. is it right if(($i == 4 || $i == 5 || $i == 6) && !mysql_numrows($ssresult)){

[msnhockey]

if this is correct... if(($i == 4 || $i == 5 || $i == 6) && !mysql_num_rows($ssresult)){

i am thinking weedpacket is correct on the theory that there is no record of goals and/or penalties so it is not displaying. now i have to make it displayable if no records are found and its showing a period greater than the one not displayed if that makes any sense

[traq]

I'm sorry: I didn't think that through enough. Please ignore me.

This may be deleted.