a conditional case

[akyuceisik]

$off = 'OFF'; chomp($off);  $on = 'ON'; chomp($on);    if(($fizrm == $off) && ($wizrm == $off))  {  system(' echo "Izmir RM Replication OFF" > /home/ahmet/repl_scr/izrmmail ');  }    elsif(($fizrm == $on) && ($wizrm == $on))  {  system(' echo "Izmir RM Replication ON" > /home/ahmet/repl_scr/izrmmail ');  }    elsif(($fizrm == $on) && ($wizrm == $off))  {  system(' echo "Izmir RM Replication PARTIAL" > /home/ahmet/repl_scr/izrmmail');  }    elsif(($fizrm == $off) && ($wizrm == $on))  {  system(' echo "Izmir RM Replication PARTIAL" > /home/ahmet/repl_scr/izrmmail');  }

I have a conditional code like this, but always it selects the first output although the results must be second or third.

Is there something i miss?

[numberwhun]

Here is a tip, when posting code, it is a really good idea to make it readable. The code that you posted is all on one line and severely lacks readability. Next time, please make it a lot more legible.

Regards,

Jeff

[numberwhun]

Ok, so you are aware of what I was speaking about in my last post, here is how you should have presented your code:

$off = 'OFF'; 
chomp($off);  

$on = 'ON'; 
chomp($on);

if(($fizrm == $off) && ($wizrm == $off))  { 
    system(' echo "Izmir RM Replication OFF" > /home/ahmet/repl_scr/izrmmail '); 
}
elsif(($fizrm == $on) && ($wizrm == $on))  {
    system(' echo "Izmir RM Replication ON" > /home/ahmet/repl_scr/izrmmail ');
}    elsif(($fizrm == $on) && ($wizrm == $off)) {
    system(' echo "Izmir RM Replication PARTIAL" > /home/ahmet/repl_scr/izrmmail');
}
elsif(($fizrm == $off) && ($wizrm == $on)) {
    system(' echo "Izmir RM Replication PARTIAL" > /home/ahmet/repl_scr/izrmmail');
}

Now, in looking at this, I notice you are using a numerical comparison operator (==) to compare text, when you shoudl be using "eq" instead. "eq" will compare text.

Make sure that you have the following pragmas in use:

use strict;
use warnings;

We have no way of knowing as we cannot see your entire script. If you did have them on, you should have gotten warnings about doing a numerical comparison on a string.

Also, just a note. You shouldn't have to "chomp" the variables as you are setting them in the script and not getting the input from command line. The "chomp" command would remove the carriage return that the user enters after typing their entry.


Regards,

Jeff

[numberwhun]

Ok, I have deleted your new post as it is a duplicate. Please do not post duplicate thread.

Regards,

Jeff

[akyuceisik]

It's OK now, thanks so much, also sorry for duplicate post.

[eWish]

So, did you figured out your problem?

--Kevin

[numberwhun]

So, did you figured out your problem?

--Kevin

I think he was using the wrong comparison operator as I had suggested to him, but can only assume as he didn't say.

Regards,

Jeff

[KevinADC]

Yea, if he uses == to compare strings this:

if(($fizrm == $off) && ($wizrm == $off))

will evaluate to:

if((0 == 0) && (0 == 0))

which will obviously evaluate to true and cause a problem.