PHP != and == operators

Question

This has boggled me for a while. I am running through a directory and echo'ing out its contents, and I want to exclude the ".." and "." files.

Now, this code works:

if ($files = scandir("temp/"))
{
    foreach ($files as $file)
    {
        if ($file == ".." OR $file == ".")
        {
        }
        else {
            echo $file;
            echo "<br>";
        }
    }
}

But this doesn't...

if ($files = scandir("temp/"))
{
    foreach ($files as $file)
    {
        if ($file != ".." OR $file != ".")
        {
            echo $file;
            echo "<br>";
        }
    }
}

For obvious reasons the second lump of code is more what I want, because I really hate having the true statement do nothing.

You really need to choose an accepted answer on some your questions. — Josh Mein, Jan 28 '10 at 14:12

score 22 · Answer 1 · edited Jan 03 '16 at 15:36

22

If you negate a condition consisting of two single conditions and a conjunction ("and" or "or"), you need to negate each condition separately and use the other conjunction.

So try this instead:

if ($file != ".." AND $file != ".")

edited Jan 03 '16 at 15:36

Peter Mortensen

30,738
21
105
131

answered Nov 02 '09 at 20:24

Franz

11,353
8
48
70

score 9 · Answer 2 · answered Nov 02 '09 at 20:28

9

This is one of deMorgan's Laws.

not (A OR B) = (not A) AND (not B)

The change you are making is a refactoring called Reverse Conditional

answered Nov 02 '09 at 20:28

Ewan Todd

7,315
26
33

score 4 · Answer 3 · answered Nov 02 '09 at 20:25

4

They're not opposites...

Check out de Morgan's laws.

if($file != ".." OR $file != ".")

should be

if($file != ".." AND $file != ".")

answered Nov 02 '09 at 20:25

Austin Salonen

49,173
15
109
139

score 2 · Answer 4 · edited Jan 03 '16 at 15:38

2

You have to negate the entire expression, just like -(-x + 2) in math negates everything:

if ($file == ".." OR $file == ".")

Is not the negation of

if ($file != ".." OR $file != ".")

Because you didn't negate the OR. The opposite of OR is AND, resulting a:

if ($file != ".." AND $file != ".")

edited Jan 03 '16 at 15:38

Peter Mortensen

30,738
21
105
131

answered Nov 02 '09 at 20:25

LiraNuna

64,916
15
117
140

score 2 · Answer 5 · edited Jan 03 '16 at 15:39

2

$file != ".." evaluates to true. Instead, simply use an AND operator:

if ( $file != '..' && $file != '.' ) { }

However, I would use DirectoryIterator instead:

foreach (new DirectoryIterator('temp') as $fileInfo) {
    if ($fileInfo->isDot())
        continue;
    echo $fileInfo->getFilename() . "<br>\n";
}

edited Jan 03 '16 at 15:39

Peter Mortensen

30,738
21
105
131

answered Nov 02 '09 at 20:26

meder omuraliev

183,342
71
393
434

score 1 · Answer 6 · edited Jan 03 '16 at 15:37

1

This:

    if ($file != ".." OR $file != ".")

should be:

    if ($file != ".." && $file != ".")

edited Jan 03 '16 at 15:37

Peter Mortensen

30,738
21
105
131

answered Nov 02 '09 at 20:25

schneck

10,556
11
49
74

score 1 · Answer 7 · edited Jan 03 '16 at 15:40

1

Alternatively, you could always use DirectoryIterator and specifically its isDot method.

edited Jan 03 '16 at 15:40

Peter Mortensen

30,738
21
105
131

answered Nov 02 '09 at 20:32

Peter Bailey

105,256
31
182
206

PHP != and == operators

7 Answers7

Linked