114

I'm trying to evaluate ((x == a and y == b) or (x == b and y == a)) in Python, but it seems a bit verbose. Is there a more elegant way?

9
  • 8
    Depends on what type of objects x,y, a,b are: are they ints/floats/strings, arbitrary objects, or what? If they were builtin types and it was possible to keep both x,y and a,b in sorted order, then you could avoid the second branch. Note that creating a set will cause each of the four elements x,y, a,b to be hashed, which might or might not be trivial or have a performance implication depending entirely on what type of objects they are.
    – smci
    Oct 17, 2019 at 23:55
  • 18
    Keep in mind the people you work with and the kind of project you're developing. I use a little Python here and there. If someone coded one of the answers here I would totally have to google what's going on. Your conditional is readable pretty much no matter what.
    – user1162766
    Oct 18, 2019 at 13:54
  • 4
    I wouldn't bother with any of the replacements. They're more terse, but not as clear (IMHO) and I think will all be slower.
    – Barmar
    Oct 18, 2019 at 14:58
  • 24
    This is a classic XYAB problem.
    – Tashus
    Oct 18, 2019 at 15:53
  • 6
    And in general if you're dealing with order-independent collections of objects, use sets/dicts/etc. This really is an XY problem, we'd need to see the larger codebase it's drawn from. I agree that ((x == a and y == b) or (x == b and y == a)) may look yukky, but 1) its intent is crystal clear and intelligible to all non-Python programmers, not cryptic 2) interpreters/compilers will always handle it well and it can essentially never result in non-performant code, unlike the alternatives. So, 'more elegant' can have serious downsides too.
    – smci
    Oct 18, 2019 at 17:28

10 Answers 10

156

If the elements are hashable, you could use sets:

{a, b} == {y, x}
15
  • 21
    @Graham no it doesn't, because there are exactly two items in the right hand set. If both are a, there's no b, and if both are b, there's no a, and in either case the sets can't be equal.
    – hobbs
    Oct 18, 2019 at 0:46
  • 13
    On the other hand, if we had three elements on each side and we needed to test whether they could be matched up one to one, then sets wouldn't work. {1, 1, 2} == {1, 2, 2}. At that point, you need sorted or Counter. Oct 18, 2019 at 7:09
  • 11
    I find this tricky to read (don't read the "{}" as "()"). Enough to comment it - and then the purpose is lost.
    – Édouard
    Oct 18, 2019 at 11:20
  • 9
    I know it's python but creating two new sets just to compare the values seems like an overkill to me... Just define a function which does it and call when needed.
    – marxin
    Oct 18, 2019 at 13:18
  • 6
    @marxin A function would be even bigger overkill than 2 simple set constructions. And less readable with 4 arguments.
    – Gloweye
    Oct 18, 2019 at 14:37
62

I think the best you could get is to package them into tuples:

if (a, b) == (x, y) or (a, b) == (y, x)

Or, maybe wrap that in a set lookup

if (a, b) in {(x, y), (y, x)}

Just since it was mentioned by a couple comments, I did some timings, and tuples and sets appear to perform identically here when the lookup fails:

from timeit import timeit

x = 1
y = 2
a = 3
b = 4

>>> timeit(lambda: (a, b) in {(x, y), (y, x)}, number=int(5e7))
32.8357742

>>> timeit(lambda: (a, b) in ((x, y), (y, x)), number=int(5e7))
31.6169182

Although tuples are actually faster when the lookup succeeds:

x = 1
y = 2
a = 1
b = 2

>>> timeit(lambda: (a, b) in {(x, y), (y, x)}, number=int(5e7))
35.6219458

>>> timeit(lambda: (a, b) in ((x, y), (y, x)), number=int(5e7))
27.753138700000008

I chose to use a set because I'm doing a membership lookup, and conceptually a set is a better fit for that use-case than a tuple. If you measured a significant different between the two structures in a particular use case, go with the faster one. I don't think performance is a factor here though.

11
  • 12
    The tuple method looks very clean. I'd be concerned about the performance impact of using a set. You could do if (a, b) in ((x, y), (y, x)), though?
    – Brilliand
    Oct 18, 2019 at 5:19
  • 20
    @Brilliand If you're concerned about performance impact, Python is not the language for you. :-D
    – Sneftel
    Oct 18, 2019 at 10:35
  • 1
    I really like the first method, two tuple comparisons. It's easy to parse (one clause at a time) and each clause is very simple. And top of which, it should be relatively efficient. Oct 18, 2019 at 11:18
  • Is there any reason to prefer the set solution in the answer to the tuple solution from @Brilliand? Oct 18, 2019 at 15:28
  • A set requires that objects be hashable, so a tuple would be a more general solution with fewer dependencies. The performance, while probably not generally important, may also look very different when dealing with large objects with expensive hashing and equality checks. Oct 19, 2019 at 15:23
32

Tuples make it slightly more readable:

(x, y) == (a, b) or (x, y) == (b, a)

This gives a clue: we're checking whether the sequence x, y is equal to the sequence a, b but ignoring ordering. That's just set equality!

{x, y} == {a, b}
2
  • , creates a tuple, not a list. so (x, y) and (a, b) are tuples, same as x, y and a, b.
    – kissgyorgy
    Jan 8, 2020 at 13:24
  • I meant "list" in the sense of "ordered sequence of elements", not in the sense of the Python list type. Edited because indeed this was confusing.
    – Thomas
    Jan 8, 2020 at 15:52
28

If the items aren't hashable, but support ordering comparisons, you could try:

sorted((x, y)) == sorted((a, b))
2
  • Since this works with hashable items as well (right?), it's a more global solution. Oct 18, 2019 at 15:34
  • 6
    @CarlWitthoft: No. There are types that are hashable but not sortable: complex, for example.
    – dan04
    Oct 18, 2019 at 22:32
28

The most elegant way, in my opinion, would be

(x, y) in ((a, b), (b, a))

This is a better way than using sets, i.e. {a, b} == {y, x}, as indicated in other answers because we don't need to think if the variables are hashable.

2
  • How does this differ from this earlier answer?
    – scohe001
    Oct 18, 2019 at 19:38
  • 5
    @scohe001 It uses a tuple where the earlier answer uses a set. That earlier answer did consider this solution, but declined to list it as a recommendation.
    – Brilliand
    Oct 19, 2019 at 0:03
25

If these are numbers, you can use (x+y)==(a+b) and (x*y)==(a*b).

If these are comparable items, you can use min(x,y)==min(a,b) and max(x,y)==max(a,b).

But ((x == a and y == b) or (x == b and y == a)) is clear, safe, and more general.

3
  • 2
    Haha, symmetric polynomials ftw!
    – Carsten S
    Oct 18, 2019 at 16:06
  • 4
    This is the correct answer, in particular the last sentence. Keep it simple, this should not require multiple new iterables or anything like that. For those that want to use sets, take a look at the implementation of set objects and then imagine trying to run this in a tight loop...
    – Z4-tier
    Oct 18, 2019 at 17:52
  • 4
    @RazvanSocol OP didn't say what the data types are, and this answer does qualify the solutions that are type-dependent.
    – Z4-tier
    Oct 18, 2019 at 17:54
23

As a generalization to more than two variables we can use itertools.permutations. That is instead of

(x == a and y == b and z == c) or (x == a and y == c and z == b) or ...

we can write

(x, y, z) in itertools.permutations([a, b, c])

And of course the two variable version:

(x, y) in itertools.permutations([a, b])
2
  • 6
    Great answer. It is worth pointing out here (for those that haven't done much with generators before) that this is very memory efficient, as only one permutation is created at a time, and the "in" check would stop and return True immediately after a match is found. Oct 20, 2019 at 0:01
  • 2
    It's also worth pointing out that the complexity of this method is O(N*N!); For 11 variables, this can take over a second to finish. (I posted a faster method, but it still takes O(N^2), and starts taking over a second on 10k variables; So it seems this can either be done fast or generally (wrt. hashability/orderability), but not both :P) Oct 21, 2019 at 6:57
15

You can use tuples to represent your data and then check for set inclusion, like:

def test_fun(x, y):
    test_set = {(a, b), (b, a)}

    return (x, y) in test_set
2
  • 3
    Written as a one-liner and with a list (for non-hashable items), I would consider this to be the best answer. (although I'm a complete Python newbie).
    – Édouard
    Oct 18, 2019 at 11:02
  • 1
    @Édouard Now there's another answer that's essentially that (just with a tuple instead of a list, which is more efficient anyway).
    – Brilliand
    Oct 19, 2019 at 1:17
11

You already got the most readable solution. There are other ways to express this, perhaps with less characters, but they are less straight-forward to read.

Depending on what the values actually represent your best bet is to wrap the check in a function with a speaking name. Alternatively or in addition, you can model the objects x,y and a,b each in dedicated higher class objects that you then can compare with the logic of the comparison in a class equality check method or a dedicated custom function.

5

It seems the OP was only concerned with the case of two variables, but since StackOverflow is also for those who search for the same question later, I'll try to tackle the generic case here in some detail; One previous answer already contains a generic answer using itertools.permutations(), but that method leads to O(N*N!) comparisons, since there are N! permutations with N items each. (This was the main motivation for this answer)

First, let's summarize how some of the methods in previous answers apply to the generic case, as motivation for the method presented here. I'll be using A to refer to (x, y) and B to refer to (a, b), which can be tuples of arbitrary (but equal) length.

set(A) == set(B) is fast, but only works if the values are hashable and you can guarantee that one of the tuples doesn't contain any duplicate values. (Eg. {1, 1, 2} == {1, 2, 2}, as pointed out by @user2357112 under @Daniel Mesejo's answer)

The previous method can be extended to work with duplicate values by using dictionaries with counts, instead of sets: (This still has the limitation that all values need to be hashable, so e.g. mutable values like list won't work)

def counts(items):
    d = {}
    for item in items:
        d[item] = d.get(item, 0) + 1
    return d

counts(A) == counts(B)

sorted(A) == sorted(B) doesn't require hashable values, but is slightly slower, and requires orderable values instead. (So e.g. complex won't work)

A in itertools.permutations(B) doesn't require hashable or orderable values, but like already mentioned, it has O(N*N!) complexity, so even with just 11 items, it can take over a second to finish.

So, is there a way to be as general, but do it considerably faster? Why yes, by "manually" checking that there's the same amount of each item: (The complexity of this one is O(N^2), so this isn't good for large inputs either; On my machine, 10k items can take over a second - but with smaller inputs, like 10 items, this is just as fast as the others)

def unordered_eq(A, B):
    for a in A:
        if A.count(a) != B.count(a):
            return False
    return True

To get the best performance, one might want to try the dict-based method first, fall back to the sorted-based method if that fails due to unhashable values, and finally fall back to the count-based method if that too fails due to unorderable values.

1
  • counts is equivalent to collections.Counter, which is implemented in C in Python 3 so should be faster, hopefully.
    – wjandrea
    Jun 16, 2020 at 22:16

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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