Sun, 03 Dec 2023 19:46:34 +0100
Corrected some uses of dict.keys(), dict.values() and dict.items().
10052 | 1 | # -*- coding: utf-8 -*- |
2 | ||
3 | # Copyright (c) 2023 Detlev Offenbach <detlev@die-offenbachs.de> | |
4 | # | |
5 | ||
6 | """ | |
7 | Module implementing a checker for unused arguments, variables, ... . | |
8 | """ | |
9 | ||
10 | import ast | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
11 | import collections |
10052 | 12 | import copy |
13 | ||
14 | import AstUtilities | |
15 | ||
16 | ||
17 | class UnusedChecker: | |
18 | """ | |
19 | Class implementing a checker for unused arguments, variables, ... . | |
20 | """ | |
21 | ||
22 | Codes = [ | |
23 | ## Unused Arguments | |
24 | "U100", | |
25 | "U101", | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
26 | ## Unused Globals |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
27 | "U200", |
10052 | 28 | ] |
29 | ||
30 | def __init__(self, source, filename, tree, select, ignore, expected, repeat, args): | |
31 | """ | |
32 | Constructor | |
33 | ||
34 | @param source source code to be checked | |
35 | @type list of str | |
36 | @param filename name of the source file | |
37 | @type str | |
38 | @param tree AST tree of the source code | |
39 | @type ast.Module | |
40 | @param select list of selected codes | |
41 | @type list of str | |
42 | @param ignore list of codes to be ignored | |
43 | @type list of str | |
44 | @param expected list of expected codes | |
45 | @type list of str | |
46 | @param repeat flag indicating to report each occurrence of a code | |
47 | @type bool | |
48 | @param args dictionary of arguments for the various checks | |
49 | @type dict | |
50 | """ | |
51 | self.__select = tuple(select) | |
52 | self.__ignore = ("",) if select else tuple(ignore) | |
53 | self.__expected = expected[:] | |
54 | self.__repeat = repeat | |
55 | self.__filename = filename | |
56 | self.__source = source[:] | |
57 | self.__tree = copy.deepcopy(tree) | |
58 | self.__args = args | |
59 | ||
60 | # statistics counters | |
61 | self.counters = {} | |
62 | ||
63 | # collection of detected errors | |
64 | self.errors = [] | |
65 | ||
66 | checkersWithCodes = [ | |
67 | (self.__checkUnusedArguments, ("U100", "U101")), | |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
68 | (self.__checkUnusedGlobals, ("U200",)), |
10052 | 69 | ] |
70 | ||
71 | self.__checkers = [] | |
72 | for checker, codes in checkersWithCodes: | |
73 | if any(not (code and self.__ignoreCode(code)) for code in codes): | |
74 | self.__checkers.append(checker) | |
75 | ||
76 | def __ignoreCode(self, code): | |
77 | """ | |
78 | Private method to check if the message code should be ignored. | |
79 | ||
80 | @param code message code to check for | |
81 | @type str | |
82 | @return flag indicating to ignore the given code | |
83 | @rtype bool | |
84 | """ | |
85 | return code.startswith(self.__ignore) and not code.startswith(self.__select) | |
86 | ||
87 | def __error(self, lineNumber, offset, code, *args): | |
88 | """ | |
89 | Private method to record an issue. | |
90 | ||
91 | @param lineNumber line number of the issue | |
92 | @type int | |
93 | @param offset position within line of the issue | |
94 | @type int | |
95 | @param code message code | |
96 | @type str | |
97 | @param args arguments for the message | |
98 | @type list | |
99 | """ | |
100 | if self.__ignoreCode(code): | |
101 | return | |
102 | ||
103 | if code in self.counters: | |
104 | self.counters[code] += 1 | |
105 | else: | |
106 | self.counters[code] = 1 | |
107 | ||
108 | # Don't care about expected codes | |
109 | if code in self.__expected: | |
110 | return | |
111 | ||
112 | if code and (self.counters[code] == 1 or self.__repeat): | |
113 | # record the issue with one based line number | |
114 | self.errors.append( | |
115 | { | |
116 | "file": self.__filename, | |
117 | "line": lineNumber + 1, | |
118 | "offset": offset, | |
119 | "code": code, | |
120 | "args": args, | |
121 | } | |
122 | ) | |
123 | ||
124 | def run(self): | |
125 | """ | |
126 | Public method to check the given source against miscellaneous | |
127 | conditions. | |
128 | """ | |
129 | if not self.__filename: | |
130 | # don't do anything, if essential data is missing | |
131 | return | |
132 | ||
133 | if not self.__checkers: | |
134 | # don't do anything, if no codes were selected | |
135 | return | |
136 | ||
137 | for check in self.__checkers: | |
138 | check() | |
139 | ||
140 | ####################################################################### | |
141 | ## Unused Arguments | |
142 | ## | |
143 | ## adapted from: flake8-unused-arguments v0.0.13 | |
144 | ####################################################################### | |
145 | ||
146 | def __checkUnusedArguments(self): | |
147 | """ | |
148 | Private method to check function and method definitions for unused arguments. | |
149 | """ | |
150 | finder = FunctionFinder(self.__args["IgnoreNestedFunctions"]) | |
151 | finder.visit(self.__tree) | |
152 | ||
153 | for functionNode in finder.functionNodes(): | |
154 | decoratorNames = set(self.__getDecoratorNames(functionNode)) | |
155 | ||
156 | # ignore overload functions, it's not a surprise when they're empty | |
157 | if self.__args["IgnoreOverload"] and "overload" in decoratorNames: | |
158 | continue | |
159 | ||
160 | # ignore overridden functions | |
161 | if self.__args["IgnoreOverride"] and "override" in decoratorNames: | |
162 | continue | |
163 | ||
164 | # ignore abstractmethods, it's not a surprise when they're empty | |
165 | if self.__args["IgnoreAbstract"] and "abstractmethod" in decoratorNames: | |
166 | continue | |
167 | ||
168 | # ignore Qt slot methods | |
169 | if self.__args["IgnoreSlotMethods"] and ( | |
170 | "pyqtSlot" in decoratorNames or "Slot" in decoratorNames | |
171 | ): | |
172 | continue | |
173 | ||
10066
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
174 | if self.__args["IgnoreEventHandlerMethods"] and self.__isEventHandlerMethod( |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
175 | functionNode |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
176 | ): |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
177 | continue |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
178 | |
10052 | 179 | # ignore stub functions |
180 | if self.__args["IgnoreStubs"] and self.__isStubFunction(functionNode): | |
181 | continue | |
182 | ||
183 | # ignore lambdas | |
184 | if self.__args["IgnoreLambdas"] and isinstance(functionNode, ast.Lambda): | |
185 | continue | |
186 | ||
187 | # ignore __double_underscore_methods__() | |
188 | if self.__args["IgnoreDunderMethods"] and self.__isDunderMethod( | |
189 | functionNode | |
190 | ): | |
191 | continue | |
192 | ||
193 | for i, argument in self.__getUnusedArguments(functionNode): | |
194 | name = argument.arg | |
195 | if self.__args["IgnoreVariadicNames"]: | |
196 | if ( | |
197 | functionNode.args.vararg | |
198 | and functionNode.args.vararg.arg == name | |
199 | ): | |
200 | continue | |
201 | if functionNode.args.kwarg and functionNode.args.kwarg.arg == name: | |
202 | continue | |
203 | ||
204 | # ignore self or whatever the first argument is for a classmethod | |
205 | if i == 0 and ( | |
206 | name in ("self", "cls") or "classmethod" in decoratorNames | |
207 | ): | |
208 | continue | |
209 | ||
210 | lineNumber = argument.lineno | |
211 | offset = argument.col_offset | |
212 | ||
213 | errorCode = "U101" if name.startswith("_") else "U100" | |
214 | self.__error(lineNumber - 1, offset, errorCode, name) | |
215 | ||
216 | def __getDecoratorNames(self, functionNode): | |
217 | """ | |
218 | Private method to yield the decorator names of the function. | |
219 | ||
220 | @param functionNode reference to the node defining the function or lambda | |
221 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
222 | @yield decorator name | |
223 | @ytype str | |
224 | """ | |
225 | if isinstance(functionNode, ast.Lambda): | |
226 | return | |
227 | ||
228 | for decorator in functionNode.decorator_list: | |
229 | if isinstance(decorator, ast.Name): | |
230 | yield decorator.id | |
231 | elif isinstance(decorator, ast.Attribute): | |
232 | yield decorator.attr | |
233 | elif isinstance(decorator, ast.Call): | |
234 | if isinstance(decorator.func, ast.Name): | |
235 | yield decorator.func.id | |
236 | else: | |
237 | yield decorator.func.attr | |
238 | ||
239 | def __isStubFunction(self, functionNode): | |
240 | """ | |
241 | Private method to check, if the given function node defines a stub function. | |
242 | ||
243 | @param functionNode reference to the node defining the function or lambda | |
244 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
245 | @return flag indicating a stub function | |
246 | @rtype bool | |
247 | """ | |
248 | if isinstance(functionNode, ast.Lambda): | |
249 | return AstUtilities.isEllipsis(functionNode.body) | |
250 | ||
251 | statement = functionNode.body[0] | |
252 | if isinstance(statement, ast.Expr) and AstUtilities.isString(statement.value): | |
253 | if len(functionNode.body) > 1: | |
254 | # first statement is a docstring, let's skip it | |
255 | statement = functionNode.body[1] | |
256 | else: | |
257 | # it's a function with only a docstring, that's a stub | |
258 | return True | |
259 | ||
260 | if isinstance(statement, ast.Pass): | |
261 | return True | |
262 | if isinstance(statement, ast.Expr) and AstUtilities.isEllipsis(statement.value): | |
263 | return True | |
264 | ||
265 | if isinstance(statement, ast.Raise): | |
266 | # like 'raise NotImplementedError()' | |
267 | if ( | |
268 | isinstance(statement.exc, ast.Call) | |
269 | and hasattr(statement.exc.func, "id") | |
270 | and statement.exc.func.id == "NotImplementedError" | |
271 | ): | |
272 | return True | |
273 | ||
274 | # like 'raise NotImplementedError' | |
275 | elif ( | |
276 | isinstance(statement.exc, ast.Name) | |
277 | and hasattr(statement.exc, "id") | |
278 | and statement.exc.id == "NotImplementedError" | |
279 | ): | |
280 | return True | |
281 | ||
282 | return False | |
283 | ||
284 | def __isDunderMethod(self, functionNode): | |
285 | """ | |
286 | Private method to check, if the function node defines a special function. | |
287 | ||
288 | @param functionNode reference to the node defining the function or lambda | |
289 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
290 | @return flag indicating a special function | |
291 | @rtype bool | |
292 | """ | |
293 | if isinstance(functionNode, ast.Lambda): | |
294 | return False | |
295 | ||
296 | if not hasattr(functionNode, "name"): | |
297 | return False | |
298 | ||
299 | name = functionNode.name | |
300 | return len(name) > 4 and name.startswith("__") and name.endswith("__") | |
301 | ||
10066
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
302 | def __isEventHandlerMethod(self, functionNode): |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
303 | """ |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
304 | Private method to check, if the function node defines a Qt event handler. |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
305 | |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
306 | Qt event handler methods are assumed to end with 'Event' or have the name |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
307 | 'event' or 'eventFilter'. Only standard methodes (i.e. ast.FunctionDef) |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
308 | are assumed to be potential event handlers. |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
309 | |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
310 | @param functionNode reference to the node defining the function or lambda |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
311 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
312 | @return flag indicating a Qt event handler method |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
313 | @rtype bool |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
314 | """ |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
315 | if isinstance(functionNode, (ast.Lambda, ast.AsyncFunctionDef)): |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
316 | return False |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
317 | |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
318 | if not hasattr(functionNode, "name"): |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
319 | return False |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
320 | |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
321 | name = functionNode.name |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
322 | return name.endswith("Event") or name in ("event", "eventFilter") |
251638443e17
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10056
diff
changeset
|
323 | |
10052 | 324 | def __getUnusedArguments(self, functionNode): |
325 | """ | |
326 | Private method to get a list of unused arguments of the given function. | |
327 | ||
328 | @param functionNode reference to the node defining the function or lambda | |
329 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
330 | @return list of tuples of the argument position and the argument | |
331 | @rtype list of tuples of (int, ast.arg) | |
332 | """ | |
333 | arguments = list(enumerate(self.__getArguments(functionNode))) | |
334 | ||
335 | class NameFinder(ast.NodeVisitor): | |
336 | """ | |
337 | Class to find the used argument names. | |
338 | """ | |
339 | ||
340 | def visit_Name(self, name): | |
341 | """ | |
342 | Public method to check a Name node. | |
343 | ||
344 | @param name reference to the name node to be checked | |
345 | @type ast.Name | |
346 | """ | |
347 | nonlocal arguments | |
348 | ||
349 | if isinstance(name.ctx, ast.Store): | |
350 | return | |
351 | ||
352 | arguments = [ | |
353 | (argIndex, arg) for argIndex, arg in arguments if arg.arg != name.id | |
354 | ] | |
355 | ||
356 | NameFinder().visit(functionNode) | |
357 | return arguments | |
358 | ||
359 | def __getArguments(self, functionNode): | |
360 | """ | |
361 | Private method to get all argument names of the given function. | |
362 | ||
363 | @param functionNode reference to the node defining the function or lambda | |
364 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
365 | @return list of argument names | |
366 | @rtype list of ast.arg | |
367 | """ | |
368 | args = functionNode.args | |
369 | ||
370 | orderedArguments = [] | |
371 | ||
372 | # plain old args | |
373 | orderedArguments.extend(args.args) | |
374 | ||
375 | # *arg name | |
376 | if args.vararg is not None: | |
377 | orderedArguments.append(args.vararg) | |
378 | ||
379 | # *, key, word, only, args | |
380 | orderedArguments.extend(args.kwonlyargs) | |
381 | ||
382 | # **kwarg name | |
383 | if args.kwarg is not None: | |
384 | orderedArguments.append(args.kwarg) | |
385 | ||
386 | return orderedArguments | |
387 | ||
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
388 | ####################################################################### |
10056
ac1c214e0a05
Corrected some comments.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10054
diff
changeset
|
389 | ## Unused Globals |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
390 | ## |
10185
29e896e9e74e
Updated some codestyle checkers for imports and too complex annotations.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10127
diff
changeset
|
391 | ## adapted from: flake8-unused-globals v0.1.10 |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
392 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
393 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
394 | def __checkUnusedGlobals(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
395 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
396 | Private method to check for unused global variables. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
397 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
398 | errors = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
399 | loadCounter = GlobalVariableLoadCounter() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
400 | loadCounter.visit(self.__tree) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
401 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
402 | globalVariables = self.__extractGlobalVariables() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
403 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
404 | for varId, loads in loadCounter.getLoads(): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
405 | if varId in globalVariables and loads == 0: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
406 | storeInfo = loadCounter.getStoreInfo(varId) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
407 | errorInfo = (storeInfo.lineno - 1, storeInfo.offset, "U200", varId) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
408 | errors[varId] = errorInfo |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
409 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
410 | for node in self.__tree.body[::-1]: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
411 | if isinstance(node, ast.Assign): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
412 | for target in node.targets: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
413 | if isinstance(target, ast.Name) and target.id in errors: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
414 | errors.pop(target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
415 | elif ( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
416 | isinstance(node, ast.AnnAssign) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
417 | and isinstance(node.target, ast.Name) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
418 | and node.target.id in errors |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
419 | ): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
420 | errors.pop(node.target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
421 | else: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
422 | break |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
423 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
424 | if self.__args["IgnoreDunderGlobals"]: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
425 | # eliminate some special cases |
10373
093dcebe5ecb
Corrected some uses of dict.keys(), dict.values() and dict.items().
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10185
diff
changeset
|
426 | for name in list(errors): |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
427 | if name.startswith("__") and name.endswith("__"): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
428 | errors.pop(name) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
429 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
430 | for varId in errors: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
431 | self.__error(*errors[varId]) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
432 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
433 | def __extractGlobalVariables(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
434 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
435 | Private method to get the names of all global variables. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
436 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
437 | @return set containing the defined global variable names |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
438 | @rtype set of str |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
439 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
440 | variables = set() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
441 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
442 | for assignment in self.__tree.body: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
443 | if isinstance(assignment, ast.Assign): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
444 | for target in assignment.targets: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
445 | if isinstance(target, ast.Name): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
446 | variables.add(target.id) |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
447 | elif isinstance(assignment, ast.AnnAssign) and isinstance( |
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
448 | assignment.target, ast.Name |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
449 | ): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
450 | variables.add(assignment.target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
451 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
452 | return variables |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
453 | |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
454 | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
455 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
456 | ## Class used by 'Unused Arguments' |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
457 | ## |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
458 | ## adapted from: flake8-unused-arguments v0.0.13 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
459 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
460 | |
10052 | 461 | |
462 | class FunctionFinder(ast.NodeVisitor): | |
463 | """ | |
464 | Class to find all defined functions and methods. | |
465 | """ | |
466 | ||
467 | def __init__(self, onlyTopLevel=False): | |
468 | """ | |
469 | Constructor | |
470 | ||
471 | @param onlyTopLevel flag indicating to search for top level functions only | |
472 | (defaults to False) | |
473 | @type bool (optional) | |
474 | """ | |
475 | super().__init__() | |
476 | ||
477 | self.__functions = [] | |
478 | self.__onlyTopLevel = onlyTopLevel | |
479 | ||
480 | def functionNodes(self): | |
481 | """ | |
482 | Public method to get the list of detected functions and lambdas. | |
483 | ||
484 | @return list of detected functions and lambdas | |
485 | @rtype list of ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
486 | """ | |
487 | return self.__functions | |
488 | ||
489 | def __visitFunctionTypes(self, functionNode): | |
490 | """ | |
491 | Private method to handle an AST node defining a function or lambda. | |
492 | ||
493 | @param functionNode reference to the node defining a function or lambda | |
494 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
495 | """ | |
496 | self.__functions.append(functionNode) | |
497 | if not self.__onlyTopLevel: | |
498 | if isinstance(functionNode, ast.Lambda): | |
499 | self.visit(functionNode.body) | |
500 | else: | |
501 | for obj in functionNode.body: | |
502 | self.visit(obj) | |
503 | ||
504 | visit_AsyncFunctionDef = visit_FunctionDef = visit_Lambda = __visitFunctionTypes | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
505 | |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
506 | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
507 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
508 | ## Class used by 'Unused Globals' |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
509 | ## |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
510 | ## adapted from: flake8-unused-globals v0.1.9 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
511 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
512 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
513 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
514 | GlobalVariableStoreInfo = collections.namedtuple( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
515 | "GlobalVariableStoreInfo", ["lineno", "offset"] |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
516 | ) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
517 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
518 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
519 | class GlobalVariableLoadCounter(ast.NodeVisitor): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
520 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
521 | Class to find all defined global variables and count their usages. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
522 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
523 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
524 | def __init__(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
525 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
526 | Constructor |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
527 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
528 | super().__init__() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
529 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
530 | self.__loads = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
531 | self.__storeInfo = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
532 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
533 | def visit_Name(self, nameNode): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
534 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
535 | Public method to record the definition and use of a global variable. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
536 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
537 | @param nameNode reference to the name node to be processed |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
538 | @type ast.Name |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
539 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
540 | if isinstance(nameNode.ctx, ast.Load) and nameNode.id in self.__loads: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
541 | self.__loads[nameNode.id] += 1 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
542 | elif ( |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
543 | isinstance(nameNode.ctx, ast.Store) and nameNode.id not in self.__storeInfo |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
544 | ): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
545 | self.__loads[nameNode.id] = 0 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
546 | self.__storeInfo[nameNode.id] = GlobalVariableStoreInfo( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
547 | lineno=nameNode.lineno, offset=nameNode.col_offset |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
548 | ) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
549 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
550 | def getLoads(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
551 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
552 | Public method to get an iterator of the detected variable loads. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
553 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
554 | @return DESCRIPTION |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
555 | @rtype TYPE |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
556 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
557 | return self.__loads.items() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
558 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
559 | def getStoreInfo(self, variableId): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
560 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
561 | Public method to get the store info data of a given variable ID. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
562 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
563 | @param variableId variable ID to retrieve the store info for |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
564 | @type str |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
565 | @return named tuple containing the line number and column offset |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
566 | @rtype GlobalVariableStoreInfo |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
567 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
568 | try: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
569 | return self.__storeInfo[variableId] |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
570 | except KeyError: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
571 | return None |