2019-07-22 Python

Code Smells

Python Code Smells#

Search for = []

Example:

results = []
for item in data:
    result = my_func(item)
    if result:
        results.append(result)

Can become:

results = [
    my_func(item) for item in data
    if my_func(item)
]

Using the := (walrus) it can become:

results = [
    y for item in data
    if (y := my_func(item))
]

chunk = file.read(8192)
while chunk:
    process(chunk)
    chunk = file.read(8192)

becomes:

while chunk := file.read(8192):
    process(chunk)

Why do this:

match = re.match(data)
group = match.group(1) if match else None

can be made into a single line:

group = re.match(data).group(1) if re.match(data) else None

with walrus:

group = match.group(1) if (match := re.match(data)) else None

x = y = z = 0

(z := (y := (x := 0 )))

a[i] = x

a[i] := x

self.rest = []

self.rest := []