[RESOLVED] && operator

[dcjones]

HI all,

Have a very bed day, can anyone see what is wrong with this:

<?
$var1 = "1";
$var2 = "3";
$var3 = "0";
If ($var1 == "1")  && ($var2 > "2");
echo $var3 + $var1 + $var2;
?>

[leatherback]

everything...

I think you are looking for:

$var1 = 1;
$var2 = 3;
$var3 = 0; 

if(($var1 ==1)  && ($var2 > 2))
  {
  echo $var3 + $var1 + $var2; 
  }

[dcjones]

Hi and many thanks for your fast reply.

I adjusted my code and it work as simple as the code is. The production code I am working on is:

<? 	
if ($_SESSION['_amember_user']['secure_id_1'] > "1") 
print $_SESSION['_amember_user']['secure_id_1'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_2'] ;

else

if (($_SESSION['_amember_user']['secure_id_1' > "1"]) &&  ($_SESSION['_amember_user']['secure_id_2' > "1"]))
print $_SESSION['_amember_user']['secure_id_1'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_2'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_3'] ;

?>

I have tried to apply the same logic but with failure.

where am I going wrong I ask myself, and others.

[leatherback]

You go wrong when you try to compare the size of strings, when you mean to use integers/numbers; I do not know what is in your sessions, but i assume that -again- you try to do:

if ($SESSION['amember_user']['secure_id_1'] > 1)

Furtehrmore, I know you do not NEED to put brackets around single-line if-statements, but you SHOULD, as it will mess you over one day, when you think it influences 2 or 3 lines.

J.

[dcjones]

Hi again, and many thanks for your time.

What I am trying to do is test for content in the SESSION var (which is always going to be a number)

If the first SESSION var has content, display the second SESSION var. ie.

<?
if  ($_SESSION['_amember_user']['secure_id_1'] > "1")
print $_SESSION['_amember_user']['secure_id_1'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_2'] ;
?>

and then if the first SESSION var and the second SESSION var have content, then display the third SESSION var.

<?
if  ($_SESSION['_amember_user']['secure_id_1'] > "1")
print $_SESSION['_amember_user']['secure_id_1'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_2'] ;


else

if (($_SESSION['_amember_user']['secure_id_1' > "1"]) &&  ($_SESSION['_amember_user']['secure_id_2' > "1"]))

print $_SESSION['_amember_user']['secure_id_1'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_2'] ;?><br />
<? print $_SESSION['_amember_user']['secure_id_3'] ;

?>

Or am I doiung this completely wrong?

Many thansk again.

[leatherback]

did you read my other two posts?

Compare strings with strings, and numbers with numbers.

1 is a number
"1" is a string.

If you compare "2" with 1 you might be surprised of the output.

So. If your session contains numbers do this:

if ($_SESSION['_amember_user']['secure_id_1'] > 1) 

[laserlight]

If the first SESSION var has content

What do you mean by "has content"? It looks like you mean to say "If the first SESSION var is greater than 1".

As I understand it, you are looking to write something like:

<?php
if ($_SESSION['_amember_user']['secure_id_1'] > 1) {
    // First session variable > 1, so display second session variable.
    echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n";
    if ($_SESSION['_amember_user']['secure_id_2'] > 1) {
        // First and second session variables > 1, so display third variable.
        echo $_SESSION['_amember_user']['secure_id_3'] . "<br />\n";
    }
}
?>

[dcjones]

Hi and thansk again for all your help and replies.

I now have this:

<?php 
if ($_SESSION['_amember_user']['secure_id_1'] > 1) { 
    // First session variable > 1, so display second session variable. 
    echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
	echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 

if (($_SESSION['_amember_user']['secure_id_1'] > 1) && ($_SESSION['_amember_user']['secure_id_2'] > 1)) { 
    // First and second session variables > 1, so display third variable. 
     echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
	  echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 
	echo $_SESSION['_amember_user']['secure_id_3'] . "<br />\n"; 

} 
} 
?> 

What I need to do is to understand how the else and else if stantements work so I can display "if first condition is truem display this" and "if second condition is true display that".

So I have tried to unsert an elseif by doing this:

<?php 
if ($_SESSION['_amember_user']['secure_id_1'] > 1) { 
    // First session variable > 1, so display second session variable. 
    echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
	echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 
	}
	else
	{
	if (($_SESSION['_amember_user']['secure_id_1'] > 1) && ($_SESSION['_amember_user']['secure_id_2'] > 1)) { 
        // First and second session variables > 1, so display third variable. 
         echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
		  echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 
		echo $_SESSION['_amember_user']['secure_id_3'] . "<br />\n"; 
	} 
} 
?> 

The script works up to a point.

It does not show the content of "echo $SESSION['amember_user']['secure_id_3'] . "<br />\n"; " which I know has content.

The content of "secure_id_1" is 111111
The content of "secure_id_1" is 222222
The content of "secure_id_1" is 333333

Or should I be testing for any content i.e "Isset" I think and if the 1st variable has any content, display it and then test for content of the second variable, and if there is content display it and test for content in the third variable, and so on.

I am not to sure which is the correct way to do this.

Many thanks again.

[leatherback]

Hi Dereck,

What do the different session variables stand for? Or, what I really want to know: Is it a limited set of variables (e.g., only 3?). Than you could do it like you do. Otherwise you should think about using another level of arrays, as you will very quickly run in deep problems with your script.

Do you want to just show all the session_vars? Or only the 'next' if the revious ones were et?

[laserlight]

The problem is with your boolean logic. Here's what you're writing:

if ($x) {
    // #1
} else {
    if ($x && $y) {
        // #2
    }
}

If $x is true, #1 is reached. If $x is false, neither #1 nor #2 is reached, because ($x && $y) would also be false. In other words, #2 is unreachable code.

[dcjones]

Hi

I only want to display the next variable only if the previous ones are set.

I wanted to put this into an arry but i am sorry to say that I am not good at writing code for arrys.

Any ideas,

It's never said enough within the forum, I know your time is valuable and I do thank you.

I have reversed the code some what yo get it working>

<?php 
if (isset($_SESSION['_amember_user']['secure_id_3'])) { 
    // First session variable > 1, so display second session variable. 
    echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
	echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 
	echo $_SESSION['_amember_user']['secure_id_3'] . "<br />\n"; 
	}
	elseif

 (isset($_SESSION['_amember_user']['secure_id_1']) && ($_SESSION['_amember_user']['secure_id_2'])) { 
    // First and second session variables > 1, so display third variable. 
     echo $_SESSION['_amember_user']['secure_id_1'] . "<br />\n"; 
	  echo $_SESSION['_amember_user']['secure_id_2'] . "<br />\n"; 

} 

?> 

But your right when you say that I may get into some trouble if I go to deep. And there is going to be 8 varibles.

[laserlight]

if the 1st variable has any content, display it and then test for content of the second variable, and if there is content display it and test for content in the third variable, and so on.

Looks like you may need a loop. As leatherback noted, it may be better to have another inner array.

I only want to display the next variable only if the previous ones are set.

Nonetheless, you can write something like:

for ($i = 1; $i <= 3; ++$i) {
    if (isset($_SESSION['_amember_user']['secure_id_' . $i])) {
        echo $_SESSION['_amember_user']['secure_id_' . $i] . "<br />\n";
    } else {
        break; // Do not display remaining variables as well.
    }
}

[dcjones]

Hi Laserlight,

Thanks for your reply and I have replaced my original code by using yours and it work.

But I do have a problem

I have 8 variables, and I want to show them like this::

If var1 isset: display 1
If var2 isset: display 1 and 2
if var3 isset: display 1,2 and 3
if var4 isset: display 1,2,3 and 4
if var5 isset: display 1,2,3,4 and 5
if var6 isset: display 1,2,3,4,5 and 6
if var7 isset: display 1,2,3,4,5,6 and 7
if var8 isset: display 1,2,3,4,5,6,7 and 8

The code you kindly sent displays all the vars even if one is empty.

Does this make sense?

Many thanks for your time.

[laserlight]

You may need to switch isset() for !empty()

[dcjones]

Hi Laserlight.

A big thanks for your help. All is working OK.

I would like to thank all forum member who gave thier valuable time for helping.

[bradgrafelman]

Don't forget to mark this thread resolved.