Sat, 10 Apr 2021 18:38:27 +0200
Applied some more code simplifications suggested by the new Simplify checker (super(Foo, self) => super()).
8186 | 1 | # -*- coding: utf-8 -*- |
2 | ||
3 | # Copyright (c) 2020 - 2021 Detlev Offenbach <detlev@die-offenbachs.de> | |
4 | # | |
5 | ||
6 | ||
7 | """ | |
8 | Module implementing message translations for the code style plugin messages | |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
9 | (simplify part). |
8186 | 10 | """ |
11 | ||
12 | from PyQt5.QtCore import QCoreApplication | |
13 | ||
14 | _simplifyMessages = { | |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
15 | # Python-specifics |
8186 | 16 | "Y101": QCoreApplication.translate( |
17 | "SimplifyChecker", | |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
18 | '''Multiple "isinstance()" calls which can be merged into a single ''' |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
19 | '''call for variable "{0}"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
20 | "Y102": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
21 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
22 | '''Use a single if-statement instead of nested if-statements'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
23 | "Y103": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
24 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
25 | '''Return the condition "{0}" directly'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
26 | "Y104": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
27 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
28 | '''Use "yield from {0}"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
29 | "Y105": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
30 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
31 | '''Use "with contextlib.suppress({0}):"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
32 | "Y106": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
33 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
34 | '''Handle error-cases first'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
35 | "Y107": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
36 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
37 | '''Don't use return in try/except and finally'''), |
8194
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
38 | "Y108": QCoreApplication.translate( |
8192
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
39 | "SimplifyChecker", |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
40 | '''Use ternary operator "{0} = {1} if {2} else {3}" ''' |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
41 | '''instead of if-else-block'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
42 | "Y109": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
43 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
44 | '''Use "{0} in {1}" instead of "{2}"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
45 | "Y110": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
46 | "SimplifyChecker", |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
47 | '''Use "any({0} for {1} in {2})"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
48 | "Y111": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
49 | "SimplifyChecker", |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
50 | '''Use "all({0} for {1} in {2})"'''), |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
51 | "Y112": QCoreApplication.translate( |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
52 | "SimplifyChecker", |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
53 | '''Use "{0}" instead of "{1}"'''), |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
54 | "Y113": QCoreApplication.translate( |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
55 | "SimplifyChecker", |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
56 | '''Use enumerate instead of "{0}"'''), |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
57 | "Y114": QCoreApplication.translate( |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
58 | "SimplifyChecker", |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
59 | '''Use logical or ("({0}) or ({1})") and a single body'''), |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
60 | "Y115": QCoreApplication.translate( |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
61 | "SimplifyChecker", |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
62 | '''Use context handler for opening files'''), |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
63 | "Y116": QCoreApplication.translate( |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
64 | "SimplifyChecker", |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
65 | '''Use a dictionary lookup instead of 3+ if/elif-statements: ''' |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
66 | '''return {0}'''), |
8192
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
67 | "Y117": QCoreApplication.translate( |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
68 | "SimplifyChecker", |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
69 | '''Use "{0}" instead of multiple with statements'''), |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
70 | "Y118": QCoreApplication.translate( |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
71 | "SimplifyChecker", |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
72 | '''Use "{0} in {1}" instead of "{0} in {1}.keys()"'''), |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
73 | "Y119": QCoreApplication.translate( |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
74 | "SimplifyChecker", |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
75 | '''Use a dataclass for "class {0}"'''), |
8194
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
76 | "Y120": QCoreApplication.translate( |
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
77 | "SimplifyChecker", |
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
78 | '''Use "class {0}:" instead of "class {0}(object):"'''), |
8210
b5903eaa7a7b
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8209
diff
changeset
|
79 | "Y121": QCoreApplication.translate( |
b5903eaa7a7b
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8209
diff
changeset
|
80 | "SimplifyChecker", |
b5903eaa7a7b
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8209
diff
changeset
|
81 | '''Use "class {0}({1}):" instead of "class {0}({1}, object):"'''), |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
82 | "Y122": QCoreApplication.translate( |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
83 | "SimplifyChecker", |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
84 | '''Use "{0}.get({1})" instead of "if {1} in {0}: {0}[{1}]"'''), |
8209
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
85 | |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
86 | # Python-specifics not part of flake8-simplify |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
87 | "Y181": QCoreApplication.translate( |
8202
df194f43119c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8195
diff
changeset
|
88 | "SimplifyChecker", |
df194f43119c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8195
diff
changeset
|
89 | '''Use "{0}" instead of "{1}"'''), |
8209
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
90 | "Y182": QCoreApplication.translate( |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
91 | "SimplifyChecker", |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
92 | '''Use "super()" instead of "{0}"'''), |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
93 | |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
94 | # Comparations |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
95 | "Y201": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
96 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
97 | '''Use "{0} != {1}" instead of "not {0} == {1}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
98 | "Y202": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
99 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
100 | '''Use "{0} == {1}" instead of "not {0} != {1}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
101 | "Y203": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
102 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
103 | '''Use "{0} not in {1}" instead of "not {0} in {1}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
104 | "Y204": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
105 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
106 | '''Use "{0} >= {1}" instead of "not ({0} < {1})"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
107 | "Y205": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
108 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
109 | '''Use "{0} > {1}" instead of "not ({0} <= {1})"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
110 | "Y206": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
111 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
112 | '''Use "{0} <= {1}" instead of "not ({0} > {1})"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
113 | "Y207": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
114 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
115 | '''Use "{0} < {1}" instead of "not ({0} >= {1})"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
116 | "Y208": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
117 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
118 | '''Use "{0}" instead of "not (not {0})"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
119 | |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
120 | "Y211": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
121 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
122 | '''Use "{1}" instead of "True if {0} else False"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
123 | "Y212": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
124 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
125 | '''Use "{1}" instead of "False if {0} else True"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
126 | "Y213": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
127 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
128 | '''Use "{0} if {0} else {1}" instead of "{1} if not {0} else {0}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
129 | |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
130 | "Y221": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
131 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
132 | '''Use "False" instead of "{0} and not {0}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
133 | "Y222": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
134 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
135 | '''Use "True" instead of "{0} or not {0}"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
136 | "Y223": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
137 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
138 | '''Use "True" instead of "... or True"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
139 | "Y224": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
140 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
141 | '''Use "False" instead of "... and False"'''), |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
142 | |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
143 | # Opinionated |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
144 | "Y301": QCoreApplication.translate( |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
145 | "SimplifyChecker", |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
146 | '''Use "{1} == {0}" instead of "{0} == {1}" (Yoda-condition)'''), |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
147 | |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
148 | # General Code Style |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
149 | "Y401": QCoreApplication.translate( |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
150 | "SimplifyChecker", |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
151 | '''Use keyword-argument instead of magic boolean'''), |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
152 | "Y402": QCoreApplication.translate( |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
153 | "SimplifyChecker", |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
154 | '''Use keyword-argument instead of magic number'''), |
8186 | 155 | } |
156 | ||
157 | _simplifyMessagesSampleArgs = { | |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
158 | # Python-specifics |
8186 | 159 | "Y101": ["foo"], |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
160 | "Y103": ["foo != bar"], |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
161 | "Y104": ["iterable"], |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
162 | "Y105": ["Exception"], |
8194
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
163 | "Y108": ["foo", "bar", "condition", "baz"], |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
164 | "Y109": ["foo", "[1, 42]", "foo == 1 or foo == 42"], |
8189
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
165 | "Y110": ["check", "foo", "iterable"], |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
166 | "Y111": ["check", "foo", "iterable"], |
17df5c8df8c1
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8186
diff
changeset
|
167 | "Y112": ["FOO", "foo"], |
8191
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
168 | "Y113": ["foo"], |
9125da0c227e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8189
diff
changeset
|
169 | "Y114": ["foo > 42", "bar < 42"], |
8203
dbcdb07cb600
Simplify translations: corrected some example values.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8202
diff
changeset
|
170 | "Y116": ["bar_dict.get(foo, 42)"], |
8192
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
171 | "Y117": ["with Foo() as foo, Bar() as bar:"], |
e1157bd8b4c2
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8191
diff
changeset
|
172 | "Y118": ["foo", "bar_dict"], |
8194
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
173 | "Y119": ["Foo"], |
b925628bf91f
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8192
diff
changeset
|
174 | "Y120": ["Foo"], |
8210
b5903eaa7a7b
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8209
diff
changeset
|
175 | "Y121": ["FooBar", "Foo"], |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
176 | "Y122": ["bar_dict", "'foo'"], |
8209
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
177 | |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
178 | # Python-specifics not part of flake8-simplify |
14470a65a52e
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8203
diff
changeset
|
179 | "Y181": ["foo += 42", "foo = foo + 42"], |
8218
7c09585bd960
Applied some more code simplifications suggested by the new Simplify checker (super(Foo, self) => super()).
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8211
diff
changeset
|
180 | "Y182": ["super()"], |
8202
df194f43119c
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8195
diff
changeset
|
181 | |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
182 | # Comparations |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
183 | "Y201": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
184 | "Y202": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
185 | "Y203": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
186 | "Y204": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
187 | "Y205": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
188 | "Y206": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
189 | "Y207": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
190 | "Y208": ["foo"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
191 | |
8203
dbcdb07cb600
Simplify translations: corrected some example values.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8202
diff
changeset
|
192 | "Y211": ["foo", "bool(foo)"], |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
193 | "Y212": ["foo", "not foo"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
194 | "Y213": ["foo", "bar"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
195 | |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
196 | "Y221": ["foo"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
197 | "Y222": ["foo"], |
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
198 | |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
199 | # Opinionated |
8195
db7f2badd374
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8194
diff
changeset
|
200 | "Y301": ["42", "foo"], |
8211
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
201 | |
8322a6f219ff
Code Style Checker
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
8210
diff
changeset
|
202 | # General Code Style |
8186 | 203 | } |