Pythagorean triplets using python's list comprehension

2.3k Views Asked by At

I can find out Pythagorean triplets using for loop as follows:

def triplet(n): # Find all the Pythagorean triplets between 1 and n (inclusive)
  for a in range(n+1):
    for b in range(a):
      for c in range(b):
        if a*a == b*b + c*c:
          print(a, b, c)

I wanted to replace this with a one-liner using list comprehension and tried the following piece:

[a, b, c in range(n+1), range(a), range(b) if a*a == b*b + c*c]

But, I get a syntax error on the closing square bracket. I tried to change the list into tuple using simple brackets, but with no success. May I know how to get it right?

2

There are 2 best solutions below

0
On BEST ANSWER

I think you mean

[(a,b,c) for a in range(n+1) for b in range(a) for c in range(b) if a*a == b*b + c*c]

That at least is syntactically valid.

2
On

Notice: This solution is only for the problem when a + b + c <= N

Asssume that a<=b<=c, this version is a little faster:

triplet = [(a,b,c) for a in range(1,N//3+1) for b in range(a,N//2+1) for c in range(b,N-1) if a**2 + b**2 == c**2]