implode function

[bob]

can anyone tell how to fix the following:

$some_array = implode(" OR p.product_id= ", $id_array)
$some_array = implode(" AND pt.product_id= ", $id_array)

in the end query should look like this:

blah blah WHERE p.product_id=6 AND pt.product_id=6 OR p.product_id=7 AND pt.product_id=7 OR .... ;

thx for now...

[kevinsequeira]

$some_array = "";
$some_array .= implode(" OR p.product_id= ", $id_array);
$some_array .= implode(" AND pt.product_id= ", $id_array) ;

EDIT : where are u getting the values of the numbers from ??

reg
kevin

[bob]

The values are taken from a row result

That worked... thx, only the first time the AND condition is left blank so there stands 51 in stead of AND p.product_id = 1

Query = SELECT DISTINCT pt.product_id, pt.kop, left( pt.beschrijving, 75 ) AS samenvatting FROM product_tekst AS pt, winkel AS w, product AS p WHERE pt.product_id = 1 OR pt.product_id = 4 OR pt.product_id = 51 AND p.product_id = 4 AND p.product_id = 5 AND p.winkel_id = w.winkel_id;

[YappyDog]

I think that code will give him all the OR stuff and then all the AND stuff. Which is probably not going to work. I don't think IMPLODE is the way to go........

I'd be more inclined to use a FOR LOOP or something to create the query correctly. Bob will need to worry about putting parentheses in there to get it correctly.

Something like:

$where_clause = "WHERE (";
for ($i=0, $i<count($id_array);$i++) {
if ($i > 0)
$where_clause .= " OR (";
$where_clause .= " p.product_id= " $id_array[$i] .
" AND pt.product_id = " . $id_array[$i] . ")";
}

[bob]

Thanks YappyDog,

That solved the problem.

[Weedpacket]

Instead of a big long chain of this=that OR this=theother OR this=somethingelse, it would be much nicer on the DBMS if instead this IN (that, theother, somethingelse) was used instead.