Strpos returning other result than expected - always true

Question

I have looked as an example provided here How do I check if a string contains a specific word? and tried to implement the same logic in my code.

However, the results that I get is not what I would have expected.

My code looks following

foreach ($files as $file_name) {
        require_once "Classes/PHPExcel.php";
        $tmpfname = "./product_files/".$file_name;
        //echo $tmpfname;
        //die();
        $excelReader = PHPExcel_IOFactory::createReaderForFile($tmpfname);
        $excelObj = $excelReader->load($tmpfname);
        $worksheet = $excelObj->getSheet(0);
        $lastRow = $worksheet->getHighestRow();
        echo $file_name;
         if (strpos($file_name, 'aroundnoon' !== false)) {
                    echo 'Aroundnoon </br>' ;
                 }
         else if (strpos($file_name, 'classic_drinks' !== false)) {
            echo 'Classic Drinks </br>' ;
         }
         else 
         {
             echo 'This is stupid.. </br>';
         }
}

The file names are aroundnoon.xlsx and classic_drinks.xlsx however when I run this file, I get the following results

Where am I going wrong?

I think you mean `if (strpos($file_name, 'aroundnoon') !== false)` — B001ᛦ, Jan 08 '20 at 10:45
You have a misplaced `)` in both your `strpos()` if-statements. — Tom Udding, Jan 08 '20 at 10:45
That would solve my problem... Thank you both! Can one of you post your comment as an answer so I can accept it? Thanks — Artur L, Jan 08 '20 at 10:48

score -1 · Accepted Answer · answered Jan 08 '20 at 10:57

As mentioned in the comments, you have misplaced the ) in both of the strpos() calls. Check the code below :

$files = [

    'aroundnoon.xlsx',
    'classic_drinks.xlsx'
];

foreach ($files as $file_name) {
    if (strpos($file_name, 'aroundnoon') !== false) {
        echo 'Aroundnoon </br>' ;
    }
    else if (strpos($file_name, 'classic_drinks') !== false) {
        echo 'Classic Drinks </br>' ;
    }
    else {
        echo 'This is stupid.. </br>';
   }
}

Check the output here : https://ideone.com/zpW8Mg

I am wondering who has put a downvote here and there is no reason is specified. — Happy Coder, Jan 09 '20 at 06:54

Strpos returning other result than expected - always true

1 Answers1