asked on

How do I fix this strpos statement?

How do I fix the STRPOS statement? What am I doing wrong here?

if ($Brand="PREMIUM POWER" AND (strpos($ModelArray,'Latitude'|'Vostro'|'ChemBook'|'Inspiron'|'JetBook'|'Precision'|'Studio'|'XPS')) !== false) {
    $NewBrand="Dell";
    }  else if ($Brand="PREMIUM POWER" AND (strpos($ModelArray,'ThinkPad')) !== false) {
    $NewBrand="IBM";
	}  else if ($Brand="PREMIUM POWER" AND (strpos($ModelArray,'Presario'|'Pavilion')) !== false) {
    $NewBrand="HP";
    } else {
	$NewBrand=$row5[Manufacturer];
	}

Open in new window

Marco Gasi

Line 3 must be

else if ($Brand="PREMIUM POWER" AND (strpos($ModelArray,'ThinkPad') !== false)) {

Open in new window

and line 5 must be:

}  else if ($Brand="PREMIUM POWER" AND (strpos($ModelArray,'Presario'|'Pavilion') !== false)) {

Open in new window

Chris Stanyon

There's a lot wrong with your code.

Firstly, a single = is an assignment - for compasrison your need a double ==

AND in your if statement should be &&

You can't use a single strpos call for multiple words - you need to loop through them. Create an array and loop through them one by one.

lawrence_dev

ASKER

Chris,
Please advise how to correctly structure. Thanks for your help!

Chris Stanyon

No worries. can you show me what's in $modelarray. Is it actually an array as the name suggests

lawrence_dev

ASKER

It is really not an 'array' per se. It is pipe delimited. Here it is:

ThinkPad i1700|ThinkPad i1720|ThinkPad i1721 etc

Thanks for your help!!

ASKER CERTIFIED SOLUTION

Chris Stanyon

membership

This solution is only available to members.