Method: pylint.extensions.for_any_all.ConsiderUsingAnyOrAllChecker.visit_for
Calls: 30, Exceptions: 2, Paths: 6Back
Path 1: 12 calls (0.4)
For (12)
None (12)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return
Path 2: 8 calls (0.27)
For (8)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return
Path 3: 3 calls (0.1)
For (3)
None (3)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return
Path 4: 3 calls (0.1)
For (3)
None (3)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return
Path 5: 2 calls (0.07)
For (2)
None (2)
GeneratorExit (2)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return
Path 6: 2 calls (0.07)
For (2)
None (2)
1@only_required_for_messages("consider-using-any-or-all")
2 def visit_for(self, node: nodes.For) -> None:
3 if len(node.body) != 1: # Only If node with no Else
4 return
5 if not isinstance(node.body[0], nodes.If):
6 return
7
8 if_children = list(node.body[0].get_children())
9 if any(isinstance(child, nodes.If) for child in if_children):
10 # an if node within the if-children indicates an elif clause,
11 # suggesting complex logic.
12 return
13
14 node_after_loop = node.next_sibling()
15
16 if self._assigned_reassigned_returned(node, if_children, node_after_loop):
17 final_return_bool = node_after_loop.value.name
18 suggested_string = self._build_suggested_string(node, final_return_bool)
19 self.add_message(
20 "consider-using-any-or-all",
21 node=node,
22 args=suggested_string,
23 confidence=HIGH,
24 )
25 return
26
27 if self._if_statement_returns_bool(if_children, node_after_loop):
28 final_return_bool = node_after_loop.value.value
29 suggested_string = self._build_suggested_string(node, final_return_bool)
30 self.add_message(
31 "consider-using-any-or-all",
32 node=node,
33 args=suggested_string,
34 confidence=HIGH,
35 )
36 return