138

I have 2 tables in my database. One is for orders, and one is for companies.

Orders has this structure:

OrderID     |     attachedCompanyIDs
------------------------------------
   1                     1,2,3
   2                     2,4

And Company has this structure:

CompanyID      |        name
--------------------------------------
    1                 Company 1
    2                 Another Company
    3                 StackOverflow
    4                 Nothing

To get an order's companies names, I can do a query as such:

SELECT name FROM orders,company
WHERE orderID = 1 AND FIND_IN_SET(companyID, attachedCompanyIDs)

That query works fine, but the following query does not.

SELECT name FROM orders,company
WHERE orderID = 1 AND companyID IN (attachedCompanyIDs)

Why does the first query work but not the second one?

The first query returns:

name
---------------
Company 1
Another Company
StackOverflow

The second query only returns:

name
---------------
Company 1

Why is this, why does the first query return all the companies, but the second query only returns the first one?

2

4 Answers 4

107
SELECT  name
FROM    orders,company
WHERE   orderID = 1
        AND companyID IN (attachedCompanyIDs)

attachedCompanyIDs is a scalar value which is cast into INT (type of companyID).

The cast only returns numbers up to the first non-digit (a comma in your case).

Thus,

companyID IN ('1,2,3') ≡ companyID IN (CAST('1,2,3' AS INT)) ≡ companyID IN (1)

In PostgreSQL, you could cast the string into array (or store it as an array in the first place):

SELECT  name
FROM    orders
JOIN    company
ON      companyID = ANY (('{' | attachedCompanyIDs | '}')::INT[])
WHERE   orderID = 1

and this would even use an index on companyID.

Unfortunately, this does not work in MySQL since the latter does not support arrays.

You may find this article interesting (see #2):

Update:

If there is some reasonable limit on the number of values in the comma separated lists (say, no more than 5), so you can try to use this query:

SELECT  name
FROM    orders
CROSS JOIN
        (
        SELECT  1 AS pos
        UNION ALL
        SELECT  2 AS pos
        UNION ALL
        SELECT  3 AS pos
        UNION ALL
        SELECT  4 AS pos
        UNION ALL
        SELECT  5 AS pos
        ) q
JOIN    company
ON      companyID = CAST(NULLIF(SUBSTRING_INDEX(attachedCompanyIDs, ',', -pos), SUBSTRING_INDEX(attachedCompanyIDs, ',', 1 - pos)) AS UNSIGNED)
8
  • 3
    Thanks for the explanation. I didn't realize the attachedCompanyIDs field was casted to an INT. Is there any way around this in MySQL? FIND_IN_SET works, but doesn't uses indexes, and may be slow with a lot of info in the Company table.
    – gen_Eric
    Commented Nov 11, 2010 at 15:43
  • 1
    Can you explain that update? What exactly does that do, because it seems to work.
    – gen_Eric
    Commented Nov 11, 2010 at 16:16
  • 1
    @Rocket: it strips pos items from the beginning of the CVS and casts the rest into integer.
    – Quassnoi
    Commented Nov 11, 2010 at 16:20
  • 12
    Thumbs up (y) for 10 things in MySQL (that won’t work as expected)
    – Smile
    Commented Feb 28, 2014 at 6:12
  • @Quassnoi, Why do you write CROSS JOIN? Aren't they all the same in MySQL?
    – Pacerier
    Commented Apr 18, 2015 at 7:08
14

attachedCompanyIDs is one big string, so mysql try to find company in this its cast to integer

when you use where in

so if comapnyid = 1 :

companyID IN ('1,2,3')

this is return true

but if the number 1 is not in the first place

 companyID IN ('2,3,1')

its return false

5

To get the all related companies name, not based on particular Id.

SELECT 
    (SELECT GROUP_CONCAT(cmp.cmpny_name) 
    FROM company cmp 
    WHERE FIND_IN_SET(cmp.CompanyID, odr.attachedCompanyIDs)
    ) AS COMPANIES
FROM orders odr
1

because the second query is looking for rows with the id's 1 OR 2 OR 3, the first query is looking for a one of the comma delimited values to exist in companyID,

and another problem here is you aren't joining the tables on a common key in your where so you are going to get a mutation of rows that = count(table1) * count(table2);

Your problem really exists with part 2 of my answer. (with your second query)

9
  • There are more rows in both tables than I am showing. In both tables, there the ID of the user you are logged in as, would joining on that help?
    – gen_Eric
    Commented Nov 11, 2010 at 15:30
  • Well you only need to change anything if your first query isn't returning the expected results. If the first query is returning the results that you want then there really isn't a problem. I thought you were just curious why the 2 don't show the same result.
    – superfro
    Commented Nov 11, 2010 at 15:34
  • @superfro, I am curious as to why the 2 don't show the same result.
    – gen_Eric
    Commented Nov 11, 2010 at 15:39
  • your second query is using a where IN (values) where the 'values' part come from the table, and its a string. The string is being evaluated as a bool true which = 1 which is why it only shows the first row.
    – superfro
    Commented Nov 11, 2010 at 15:43
  • 1
    If you are worried about performance, you should probably think about changing your database structure. You could add a joint table that contains 2 values, order_ID and company_ID instead of using the comma delimited list in the order table. This would allow you to select name from company left join order_companies on company.company_ID = order_companies.company_ID left join orders on order_companies.order_ID = order.order_ID where orders.order_ID = 1; This would use indexes.
    – superfro
    Commented Nov 11, 2010 at 16:05

Not the answer you're looking for? Browse other questions tagged or ask your own question.