with statements can be merged PTC-W0062
Anti-pattern
Major
5 months ago
—
2 years old
Consider merging collapsible
With statements` 73 c2000 = date.replace(year=2000)
74 coffset = iemre.daily_offset(c2000)
75
76 with ncopen(fn) as nc: 77 with ncopen(iemre.get_dailyc_ncname()) as cnc:
78 res.append(
79 {
Description
Nested with statements can be merged into a single compound with statement.
This issue is raised only when there are no other statements between the nested with statements.
Merging collapsible with statements can decrease cognitive complexity, improving code readability.
Bad practice
with open("file1", "w") as file1:
with open("file2", "w") as file2:
doSomething()
Preferred
with open("file1", "w") as file1, open("file2", "w") as file2:
doSomething()