Tue, 23 May 2023 13:33:47 +0200
Corrected some comments.
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 | ||
174 | # ignore stub functions | |
175 | if self.__args["IgnoreStubs"] and self.__isStubFunction(functionNode): | |
176 | continue | |
177 | ||
178 | # ignore lambdas | |
179 | if self.__args["IgnoreLambdas"] and isinstance(functionNode, ast.Lambda): | |
180 | continue | |
181 | ||
182 | # ignore __double_underscore_methods__() | |
183 | if self.__args["IgnoreDunderMethods"] and self.__isDunderMethod( | |
184 | functionNode | |
185 | ): | |
186 | continue | |
187 | ||
188 | for i, argument in self.__getUnusedArguments(functionNode): | |
189 | name = argument.arg | |
190 | if self.__args["IgnoreVariadicNames"]: | |
191 | if ( | |
192 | functionNode.args.vararg | |
193 | and functionNode.args.vararg.arg == name | |
194 | ): | |
195 | continue | |
196 | if functionNode.args.kwarg and functionNode.args.kwarg.arg == name: | |
197 | continue | |
198 | ||
199 | # ignore self or whatever the first argument is for a classmethod | |
200 | if i == 0 and ( | |
201 | name in ("self", "cls") or "classmethod" in decoratorNames | |
202 | ): | |
203 | continue | |
204 | ||
205 | lineNumber = argument.lineno | |
206 | offset = argument.col_offset | |
207 | ||
208 | errorCode = "U101" if name.startswith("_") else "U100" | |
209 | self.__error(lineNumber - 1, offset, errorCode, name) | |
210 | ||
211 | def __getDecoratorNames(self, functionNode): | |
212 | """ | |
213 | Private method to yield the decorator names of the function. | |
214 | ||
215 | @param functionNode reference to the node defining the function or lambda | |
216 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
217 | @yield decorator name | |
218 | @ytype str | |
219 | """ | |
220 | if isinstance(functionNode, ast.Lambda): | |
221 | return | |
222 | ||
223 | for decorator in functionNode.decorator_list: | |
224 | if isinstance(decorator, ast.Name): | |
225 | yield decorator.id | |
226 | elif isinstance(decorator, ast.Attribute): | |
227 | yield decorator.attr | |
228 | elif isinstance(decorator, ast.Call): | |
229 | if isinstance(decorator.func, ast.Name): | |
230 | yield decorator.func.id | |
231 | else: | |
232 | yield decorator.func.attr | |
233 | ||
234 | def __isStubFunction(self, functionNode): | |
235 | """ | |
236 | Private method to check, if the given function node defines a stub function. | |
237 | ||
238 | @param functionNode reference to the node defining the function or lambda | |
239 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
240 | @return flag indicating a stub function | |
241 | @rtype bool | |
242 | """ | |
243 | if isinstance(functionNode, ast.Lambda): | |
244 | return AstUtilities.isEllipsis(functionNode.body) | |
245 | ||
246 | statement = functionNode.body[0] | |
247 | if isinstance(statement, ast.Expr) and AstUtilities.isString(statement.value): | |
248 | if len(functionNode.body) > 1: | |
249 | # first statement is a docstring, let's skip it | |
250 | statement = functionNode.body[1] | |
251 | else: | |
252 | # it's a function with only a docstring, that's a stub | |
253 | return True | |
254 | ||
255 | if isinstance(statement, ast.Pass): | |
256 | return True | |
257 | if isinstance(statement, ast.Expr) and AstUtilities.isEllipsis(statement.value): | |
258 | return True | |
259 | ||
260 | if isinstance(statement, ast.Raise): | |
261 | # like 'raise NotImplementedError()' | |
262 | if ( | |
263 | isinstance(statement.exc, ast.Call) | |
264 | and hasattr(statement.exc.func, "id") | |
265 | and statement.exc.func.id == "NotImplementedError" | |
266 | ): | |
267 | return True | |
268 | ||
269 | # like 'raise NotImplementedError' | |
270 | elif ( | |
271 | isinstance(statement.exc, ast.Name) | |
272 | and hasattr(statement.exc, "id") | |
273 | and statement.exc.id == "NotImplementedError" | |
274 | ): | |
275 | return True | |
276 | ||
277 | return False | |
278 | ||
279 | def __isDunderMethod(self, functionNode): | |
280 | """ | |
281 | Private method to check, if the function node defines a special function. | |
282 | ||
283 | @param functionNode reference to the node defining the function or lambda | |
284 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
285 | @return flag indicating a special function | |
286 | @rtype bool | |
287 | """ | |
288 | if isinstance(functionNode, ast.Lambda): | |
289 | return False | |
290 | ||
291 | if not hasattr(functionNode, "name"): | |
292 | return False | |
293 | ||
294 | name = functionNode.name | |
295 | return len(name) > 4 and name.startswith("__") and name.endswith("__") | |
296 | ||
297 | def __getUnusedArguments(self, functionNode): | |
298 | """ | |
299 | Private method to get a list of unused arguments of the given function. | |
300 | ||
301 | @param functionNode reference to the node defining the function or lambda | |
302 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
303 | @return list of tuples of the argument position and the argument | |
304 | @rtype list of tuples of (int, ast.arg) | |
305 | """ | |
306 | arguments = list(enumerate(self.__getArguments(functionNode))) | |
307 | ||
308 | class NameFinder(ast.NodeVisitor): | |
309 | """ | |
310 | Class to find the used argument names. | |
311 | """ | |
312 | ||
313 | def visit_Name(self, name): | |
314 | """ | |
315 | Public method to check a Name node. | |
316 | ||
317 | @param name reference to the name node to be checked | |
318 | @type ast.Name | |
319 | """ | |
320 | nonlocal arguments | |
321 | ||
322 | if isinstance(name.ctx, ast.Store): | |
323 | return | |
324 | ||
325 | arguments = [ | |
326 | (argIndex, arg) for argIndex, arg in arguments if arg.arg != name.id | |
327 | ] | |
328 | ||
329 | NameFinder().visit(functionNode) | |
330 | return arguments | |
331 | ||
332 | def __getArguments(self, functionNode): | |
333 | """ | |
334 | Private method to get all argument names of the given function. | |
335 | ||
336 | @param functionNode reference to the node defining the function or lambda | |
337 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
338 | @return list of argument names | |
339 | @rtype list of ast.arg | |
340 | """ | |
341 | args = functionNode.args | |
342 | ||
343 | orderedArguments = [] | |
344 | ||
345 | # plain old args | |
346 | orderedArguments.extend(args.args) | |
347 | ||
348 | # *arg name | |
349 | if args.vararg is not None: | |
350 | orderedArguments.append(args.vararg) | |
351 | ||
352 | # *, key, word, only, args | |
353 | orderedArguments.extend(args.kwonlyargs) | |
354 | ||
355 | # **kwarg name | |
356 | if args.kwarg is not None: | |
357 | orderedArguments.append(args.kwarg) | |
358 | ||
359 | return orderedArguments | |
360 | ||
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
361 | ####################################################################### |
10056
ac1c214e0a05
Corrected some comments.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10054
diff
changeset
|
362 | ## Unused Globals |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
363 | ## |
10056
ac1c214e0a05
Corrected some comments.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10054
diff
changeset
|
364 | ## adapted from: flake8-unused-globals v0.1.9 |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
365 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
366 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
367 | def __checkUnusedGlobals(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
368 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
369 | Private method to check for unused global variables. |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
370 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
371 | errors = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
372 | loadCounter = GlobalVariableLoadCounter() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
373 | loadCounter.visit(self.__tree) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
374 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
375 | globalVariables = self.__extractGlobalVariables() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
376 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
377 | for varId, loads in loadCounter.getLoads(): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
378 | if varId in globalVariables and loads == 0: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
379 | storeInfo = loadCounter.getStoreInfo(varId) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
380 | errorInfo = (storeInfo.lineno - 1, storeInfo.offset, "U200", varId) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
381 | errors[varId] = errorInfo |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
382 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
383 | for node in self.__tree.body[::-1]: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
384 | if isinstance(node, ast.Assign): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
385 | for target in node.targets: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
386 | if isinstance(target, ast.Name) and target.id in errors: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
387 | errors.pop(target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
388 | elif ( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
389 | isinstance(node, ast.AnnAssign) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
390 | and isinstance(node.target, ast.Name) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
391 | and node.target.id in errors |
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 | errors.pop(node.target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
394 | else: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
395 | break |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
396 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
397 | if self.__args["IgnoreDunderGlobals"]: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
398 | # eliminate some special cases |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
399 | for name in list(errors.keys()): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
400 | if name.startswith("__") and name.endswith("__"): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
401 | errors.pop(name) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
402 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
403 | for varId in errors: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
404 | self.__error(*errors[varId]) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
405 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
406 | def __extractGlobalVariables(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
407 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
408 | Private method to get the names of all global variables. |
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 | @return set containing the defined global variable names |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
411 | @rtype set of str |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
412 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
413 | variables = set() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
414 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
415 | for assignment in self.__tree.body: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
416 | if isinstance(assignment, ast.Assign): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
417 | for target in assignment.targets: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
418 | if isinstance(target, ast.Name): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
419 | variables.add(target.id) |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
420 | elif isinstance(assignment, ast.AnnAssign) and isinstance( |
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
421 | assignment.target, ast.Name |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
422 | ): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
423 | variables.add(assignment.target.id) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
424 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
425 | return variables |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
426 | |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
427 | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
428 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
429 | ## Class used by 'Unused Arguments' |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
430 | ## |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
431 | ## adapted from: flake8-unused-arguments v0.0.13 |
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 | |
10052 | 434 | |
435 | class FunctionFinder(ast.NodeVisitor): | |
436 | """ | |
437 | Class to find all defined functions and methods. | |
438 | """ | |
439 | ||
440 | def __init__(self, onlyTopLevel=False): | |
441 | """ | |
442 | Constructor | |
443 | ||
444 | @param onlyTopLevel flag indicating to search for top level functions only | |
445 | (defaults to False) | |
446 | @type bool (optional) | |
447 | """ | |
448 | super().__init__() | |
449 | ||
450 | self.__functions = [] | |
451 | self.__onlyTopLevel = onlyTopLevel | |
452 | ||
453 | def functionNodes(self): | |
454 | """ | |
455 | Public method to get the list of detected functions and lambdas. | |
456 | ||
457 | @return list of detected functions and lambdas | |
458 | @rtype list of ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
459 | """ | |
460 | return self.__functions | |
461 | ||
462 | def __visitFunctionTypes(self, functionNode): | |
463 | """ | |
464 | Private method to handle an AST node defining a function or lambda. | |
465 | ||
466 | @param functionNode reference to the node defining a function or lambda | |
467 | @type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda | |
468 | """ | |
469 | self.__functions.append(functionNode) | |
470 | if not self.__onlyTopLevel: | |
471 | if isinstance(functionNode, ast.Lambda): | |
472 | self.visit(functionNode.body) | |
473 | else: | |
474 | for obj in functionNode.body: | |
475 | self.visit(obj) | |
476 | ||
477 | visit_AsyncFunctionDef = visit_FunctionDef = visit_Lambda = __visitFunctionTypes | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
478 | |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
479 | |
10053
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
480 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
481 | ## Class used by 'Unused Globals' |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
482 | ## |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
483 | ## adapted from: flake8-unused-globals v0.1.9 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
484 | ####################################################################### |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
485 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
486 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
487 | GlobalVariableStoreInfo = collections.namedtuple( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
488 | "GlobalVariableStoreInfo", ["lineno", "offset"] |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
489 | ) |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
490 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
491 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
492 | class GlobalVariableLoadCounter(ast.NodeVisitor): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
493 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
494 | 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
|
495 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
496 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
497 | def __init__(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
498 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
499 | Constructor |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
500 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
501 | super().__init__() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
502 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
503 | self.__loads = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
504 | self.__storeInfo = {} |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
505 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
506 | def visit_Name(self, nameNode): |
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 | 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
|
509 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
510 | @param nameNode reference to the name node to be processed |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
511 | @type ast.Name |
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 | 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
|
514 | self.__loads[nameNode.id] += 1 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
515 | elif ( |
10054
d7a47f0cff2b
Regenerated source documentation.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10053
diff
changeset
|
516 | 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
|
517 | ): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
518 | self.__loads[nameNode.id] = 0 |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
519 | self.__storeInfo[nameNode.id] = GlobalVariableStoreInfo( |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
520 | lineno=nameNode.lineno, offset=nameNode.col_offset |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
521 | ) |
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 | def getLoads(self): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
524 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
525 | 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
|
526 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
527 | @return DESCRIPTION |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
528 | @rtype TYPE |
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 | return self.__loads.items() |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
531 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
532 | def getStoreInfo(self, variableId): |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
533 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
534 | 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
|
535 | |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
536 | @param variableId variable ID to retrieve the store info for |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
537 | @type str |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
538 | @return named tuple containing the line number and column offset |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
539 | @rtype GlobalVariableStoreInfo |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
540 | """ |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
541 | try: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
542 | return self.__storeInfo[variableId] |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
543 | except KeyError: |
9914b7b4b11c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
10052
diff
changeset
|
544 | return None |