Conditions | 1 |
Paths | 1 |
Total Lines | 90 |
Code Lines | 88 |
Lines | 0 |
Ratio | 0 % |
Tests | 89 |
CRAP Score | 1 |
Changes | 1 | ||
Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
33 | 54 | protected function getKeywords() |
|
34 | { |
||
35 | 54 | return array( |
|
36 | 54 | 'ALL', |
|
37 | 54 | 'ANALYSE', |
|
38 | 54 | 'ANALYZE', |
|
39 | 54 | 'AND', |
|
40 | 54 | 'ANY', |
|
41 | 54 | 'AS', |
|
42 | 54 | 'ASC', |
|
43 | 54 | 'AUTHORIZATION', |
|
44 | 54 | 'BETWEEN', |
|
45 | 54 | 'BINARY', |
|
46 | 54 | 'BOTH', |
|
47 | 54 | 'CASE', |
|
48 | 54 | 'CAST', |
|
49 | 54 | 'CHECK', |
|
50 | 54 | 'COLLATE', |
|
51 | 54 | 'COLUMN', |
|
52 | 54 | 'CONSTRAINT', |
|
53 | 54 | 'CREATE', |
|
54 | 54 | 'CURRENT_DATE', |
|
55 | 54 | 'CURRENT_TIME', |
|
56 | 54 | 'CURRENT_TIMESTAMP', |
|
57 | 54 | 'CURRENT_USER', |
|
58 | 54 | 'DEFAULT', |
|
59 | 54 | 'DEFERRABLE', |
|
60 | 54 | 'DESC', |
|
61 | 54 | 'DISTINCT', |
|
62 | 54 | 'DO', |
|
63 | 54 | 'ELSE', |
|
64 | 54 | 'END', |
|
65 | 54 | 'EXCEPT', |
|
66 | 54 | 'FALSE', |
|
67 | 54 | 'FOR', |
|
68 | 54 | 'FOREIGN', |
|
69 | 54 | 'FREEZE', |
|
70 | 54 | 'FROM', |
|
71 | 54 | 'FULL', |
|
72 | 54 | 'GRANT', |
|
73 | 54 | 'GROUP', |
|
74 | 54 | 'HAVING', |
|
75 | 54 | 'ILIKE', |
|
76 | 54 | 'IN', |
|
77 | 54 | 'INITIALLY', |
|
78 | 54 | 'INNER', |
|
79 | 54 | 'INTERSECT', |
|
80 | 54 | 'INTO', |
|
81 | 54 | 'IS', |
|
82 | 54 | 'ISNULL', |
|
83 | 54 | 'JOIN', |
|
84 | 54 | 'LEADING', |
|
85 | 54 | 'LEFT', |
|
86 | 54 | 'LIKE', |
|
87 | 54 | 'LIMIT', |
|
88 | 54 | 'LOCALTIME', |
|
89 | 54 | 'LOCALTIMESTAMP', |
|
90 | 54 | 'NATURAL', |
|
91 | 54 | 'NEW', |
|
92 | 54 | 'NOT', |
|
93 | 54 | 'NOTNULL', |
|
94 | 54 | 'NULL', |
|
95 | 54 | 'OFF', |
|
96 | 54 | 'OFFSET', |
|
97 | 54 | 'OLD', |
|
98 | 54 | 'ON', |
|
99 | 54 | 'ONLY', |
|
100 | 54 | 'OR', |
|
101 | 54 | 'ORDER', |
|
102 | 54 | 'OUTER', |
|
103 | 54 | 'OVERLAPS', |
|
104 | 54 | 'PLACING', |
|
105 | 54 | 'PRIMARY', |
|
106 | 54 | 'REFERENCES', |
|
107 | 54 | 'SELECT', |
|
108 | 54 | 'SESSION_USER', |
|
109 | 54 | 'SIMILAR', |
|
110 | 54 | 'SOME', |
|
111 | 54 | 'TABLE', |
|
112 | 54 | 'THEN', |
|
113 | 54 | 'TO', |
|
114 | 54 | 'TRAILING', |
|
115 | 54 | 'TRUE', |
|
116 | 54 | 'UNION', |
|
117 | 54 | 'UNIQUE', |
|
118 | 54 | 'USER', |
|
119 | 54 | 'USING', |
|
120 | 54 | 'VERBOSE', |
|
121 | 54 | 'WHEN', |
|
122 | 54 | 'WHERE' |
|
123 | 54 | ); |
|
126 |