CODEWITHSUNDEEP
php
Nick
Nick

Reputation: 1

ELSE statement acting funny

I'm having a problem with a line of code. My teacher doesn't even see the problem and I've been fighting it for almost a week and a half. any help would be greatly appreciated.

The code:

{
  if (count($_POST['CINS']) > 0)
  {
    echo "<h2>Your CINS picks are:</h2>\n";
    echo "<ul>\n"; 
    foreach ($_POST['CINS'] as $element)
    {
      echo "\t<li>$element</li>\n";
    } // end of FOREACH statement
    echo "</ul>\n";
  } // end of IF count CINS

  if (count($_POST['CINT']) > 0 ) 
  {
    echo "<h2>Your CINT picks are:</h2>\n";
    echo "<ul>\n"; 
    foreach ($_POST['CINT'] as $element2)
    {
      echo "\t<li>$element2</li>\n";
    } // End of FOREACH CINT
    echo "</ul>\n";
  } // End of IF for CINT
  else
  {
    echo "CINT = " . count($_POST['CINT']) . " CINS = " . count($_POST['CINS']) . "<br />\n";
    echo __LINE__;
    if ((count($_POST['CINT'] == 0)) and (count($_POST['CINS'] == 0)))  // This is where the problem lies. It's showing up the echo statements even when CINS has a count of 1. but if CINT has a count of 1, the echo statements do not show up. 
    {
      echo "<h2>No classes</h2>\n";
      echo "<p>You need to pick a class from BOTH CINT and CINS to be a well rounded student.</p>\n";
      echo "CINT = " . count($_POST['CINT']) . " CINS = " . count($_POST['CINS']) . "<br />\n";
    }
  } // END ELSE COUNT CINS
}
?>

Upvotes: 0

Views: 154

Answers (4)

Dominik Ras
Dominik Ras

Reputation: 511

Perhaps parentheses need attention...

Your

if ((count($_POST['CINT'] == 0)) and (count($_POST['CINS'] == 0)))

might work better as 

if ((count($_POST['CINT']) == 0) && (count($_POST['CINS']) == 0))

Give it shot - reply with outcome.

Upvotes: 0

Kermit
Kermit

Reputation: 34063

You don't need all those parentheses:

if(
    count($_POST['CINT']) == 0 AND
    count($_POST['CINS']) == 0
    )

Look how some indentation goes a long way:

{
    if (count($_POST['CINS']) > 0) {
        echo "<h2>Your CINS picks are:</h2>\n";
        echo "<ul>\n"; 
        foreach ($_POST['CINS'] as $element) {
            echo "\t<li>$element</li>\n";
        }

        echo "</ul>\n";
    }

    if (count($_POST['CINT']) > 0 ) {
        echo "<h2>Your CINT picks are:</h2>\n";
        echo "<ul>\n"; 
        foreach ($_POST['CINT'] as $element2) {
            echo "\t<li>$element2</li>\n";
        }

        echo "</ul>\n";
    }
else {
    echo "CINT = " . count($_POST['CINT']) . " CINS = " . count($_POST['CINS']) . "<br />\n";
    echo __LINE__;

    if(
        count($_POST['CINT']) == 0 AND
        count($_POST['CINS']) == 0
        ) {
        echo "<h2>No classes</h2>\n";
        echo "<p>You need to pick a class from BOTH CINT and CINS to be a well rounded student.</p>\n";
        echo "CINT = " . count($_POST['CINT']) . " CINS = " . count($_POST['CINS']) . "<br />\n";
    }
}

Upvotes: 1

Oliver
Oliver

Reputation: 2864

I think, you want this:

if (count($_POST['CINS']) > 0)
{
    …
} // End of IF for CINS

if (count($_POST['CINT']) > 0 ) 
{
    …
} // End of IF for CINT

if (count($_POST['CINS']) == 0 || count($_POST['CINT']) == 0 ) 
{
    …
}

Upvotes: 0

Dino Babu
Dino Babu

Reputation: 5809

misplaced brackets

if ((count($_POST['CINT'] == 0)) and (count($_POST['CINS'] == 0))) -> wrong

if ((count($_POST['CINT']) == 0) and (count($_POST['CINS']) == 0))

Upvotes: 3

Related Questions