Report if too many lines in function body. Args: error: The function to call with any errors found. filename: The name of the current file. linenum: The number of the line to check.
(self, error, filename, linenum)
| 698 | self.lines_in_function += 1 |
| 699 | |
| 700 | def Check(self, error, filename, linenum): |
| 701 | """Report if too many lines in function body. |
| 702 | |
| 703 | Args: |
| 704 | error: The function to call with any errors found. |
| 705 | filename: The name of the current file. |
| 706 | linenum: The number of the line to check. |
| 707 | """ |
| 708 | if Match(r'T(EST|est)', self.current_function): |
| 709 | base_trigger = self._TEST_TRIGGER |
| 710 | else: |
| 711 | base_trigger = self._NORMAL_TRIGGER |
| 712 | trigger = base_trigger * 2**_VerboseLevel() |
| 713 | |
| 714 | if self.lines_in_function > trigger: |
| 715 | error_level = int(math.log(self.lines_in_function / base_trigger, 2)) |
| 716 | # 50 => 0, 100 => 1, 200 => 2, 400 => 3, 800 => 4, 1600 => 5, ... |
| 717 | if error_level > 5: |
| 718 | error_level = 5 |
| 719 | error(filename, linenum, 'readability/fn_size', error_level, |
| 720 | 'Small and focused functions are preferred:' |
| 721 | ' %s has %d non-comment lines' |
| 722 | ' (error triggered by exceeding %d lines).' % ( |
| 723 | self.current_function, self.lines_in_function, trigger)) |
| 724 | |
| 725 | def End(self): |
| 726 | """Stop analyzing function body.""" |
no test coverage detected