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?
10 Answers
If the elements are hashable, you could use sets:
{a, b} == {y, x}
-
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.– hobbsOct 18, 2019 at 0:46
-
13On 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 needsorted
orCounter
. Oct 18, 2019 at 7:09 -
11I find this tricky to read (don't read the "{}" as "()"). Enough to comment it - and then the purpose is lost.– ÉdouardOct 18, 2019 at 11:20
-
9I 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.– marxinOct 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.– GloweyeOct 18, 2019 at 14:37
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.
-
12The 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? Oct 18, 2019 at 5:19 -
20@Brilliand If you're concerned about performance impact, Python is not the language for you. :-D– SneftelOct 18, 2019 at 10:35
-
1I 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
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}
-
,
creates a tuple, not a list. so(x, y)
and(a, b)
are tuples, same asx, y
anda, b
. 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.– ThomasJan 8, 2020 at 15:52
If the items aren't hashable, but support ordering comparisons, you could try:
sorted((x, y)) == sorted((a, b))
-
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.– dan04Oct 18, 2019 at 22:32
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.
-
-
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. Oct 19, 2019 at 0:03
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.
-
2
-
4This 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-tierOct 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-tierOct 18, 2019 at 17:54
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])
-
6Great 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
-
2It'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 takesO(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
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
-
3Written 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).– ÉdouardOct 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). Oct 19, 2019 at 1:17
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.
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.
-
counts
is equivalent tocollections.Counter
, which is implemented in C in Python 3 so should be faster, hopefully.– wjandreaJun 16, 2020 at 22:16
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 bothx,y
anda,b
in sorted order, then you could avoid the second branch. Note that creating a set will cause each of the four elementsx,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.((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.