Conditions | 1 |
Paths | 1 |
Total Lines | 112 |
Code Lines | 75 |
Lines | 0 |
Ratio | 0 % |
Changes | 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 |
||
142 | public function dataValidationFailed(): array |
||
143 | { |
||
144 | $incorrectInputMessage = 'This value must be a string.'; |
||
145 | $greaterThanMaxMessage = 'This value must contain at most 25 characters.'; |
||
146 | $notExactlyMessage = 'This value must contain exactly 25 characters.'; |
||
147 | $lessThanMinMessage = 'This value must contain at least 25 characters.'; |
||
148 | |||
149 | return [ |
||
150 | 'incorrect input, array' => [['not a string'], [new HasLength(min: 25)], ['' => [$incorrectInputMessage]]], |
||
151 | 'incorrect input, boolean (true)' => [true, [new HasLength(min: 25)], ['' => [$incorrectInputMessage]]], |
||
152 | 'incorrect input, boolean (false)' => [false, [new HasLength(min: 25)], ['' => [$incorrectInputMessage]]], |
||
153 | 'custom incorrect input message' => [ |
||
154 | false, |
||
155 | [new HasLength(min: 25, incorrectInputMessage: 'Custom incorrect input message.')], |
||
156 | ['' => ['Custom incorrect input message.']], |
||
157 | ], |
||
158 | 'custom incorrect input message with parameters' => [ |
||
159 | false, |
||
160 | [new HasLength(min: 25, incorrectInputMessage: 'Attribute - {attribute}, type - {type}.')], |
||
161 | ['' => ['Attribute - , type - bool.']], |
||
162 | ], |
||
163 | 'custom incorrect input message with parameters, attribute set' => [ |
||
164 | ['data' => false], |
||
165 | ['data' => new HasLength(min: 25, incorrectInputMessage: 'Attribute - {attribute}, type - {type}.')], |
||
166 | ['data' => ['Attribute - data, type - bool.']], |
||
167 | ], |
||
168 | |||
169 | [new SingleValueDataSet(new stdClass()), [new HasLength(min: 25)], ['' => [$incorrectInputMessage]]], |
||
170 | |||
171 | [str_repeat('x', 1250), [new HasLength(max: 25)], ['' => [$greaterThanMaxMessage]]], |
||
172 | [str_repeat('x', 125), [new HasLength(exactly: 25)], ['' => [$notExactlyMessage]]], |
||
173 | |||
174 | ['', [new HasLength(exactly: 25)], ['' => [$notExactlyMessage]]], |
||
175 | [ |
||
176 | str_repeat('x', 5), |
||
177 | [new HasLength(min: 10, max: 25)], |
||
178 | ['' => ['This value must contain at least 10 characters.']], |
||
179 | ], |
||
180 | [str_repeat('x', 13), [new HasLength(min: 25)], ['' => [$lessThanMinMessage]]], |
||
181 | ['', [new HasLength(min: 25)], ['' => [$lessThanMinMessage]]], |
||
182 | |||
183 | 'custom less than min message' => [ |
||
184 | 'ab', |
||
185 | [new HasLength(min: 3, lessThanMinMessage: 'Custom less than min message.')], |
||
186 | ['' => ['Custom less than min message.']], |
||
187 | ], |
||
188 | 'custom less than min message with parameters' => [ |
||
189 | 'ab', |
||
190 | [new HasLength(min: 3, lessThanMinMessage: 'Min - {min}, attribute - {attribute}, number - {number}.')], |
||
191 | ['' => ['Min - 3, attribute - , number - 2.']], |
||
192 | ], |
||
193 | 'custom less than min message with parameters, attribute set' => [ |
||
194 | ['data' => 'ab'], |
||
195 | [ |
||
196 | 'data' => new HasLength( |
||
197 | min: 3, |
||
198 | lessThanMinMessage: 'Min - {min}, attribute - {attribute}, number - {number}.', |
||
199 | ), |
||
200 | ], |
||
201 | ['data' => ['Min - 3, attribute - data, number - 2.']], |
||
202 | ], |
||
203 | |||
204 | 'custom greater than max message' => [ |
||
205 | 'abcd', |
||
206 | [new HasLength(max: 3, greaterThanMaxMessage: 'Custom greater than max message.')], |
||
207 | ['' => ['Custom greater than max message.']], |
||
208 | ], |
||
209 | 'custom greater than max message with parameters' => [ |
||
210 | 'abcd', |
||
211 | [ |
||
212 | new HasLength( |
||
213 | max: 3, |
||
214 | greaterThanMaxMessage: 'Max - {max}, attribute - {attribute}, number - {number}.', |
||
215 | ), |
||
216 | ], |
||
217 | ['' => ['Max - 3, attribute - , number - 4.']], |
||
218 | ], |
||
219 | 'custom greater than max message with parameters, attribute set' => [ |
||
220 | ['data' => 'abcd'], |
||
221 | [ |
||
222 | 'data' => new HasLength( |
||
223 | max: 3, |
||
224 | greaterThanMaxMessage: 'Max - {max}, attribute - {attribute}, number - {number}.', |
||
225 | ), |
||
226 | ], |
||
227 | ['data' => ['Max - 3, attribute - data, number - 4.']], |
||
228 | ], |
||
229 | |||
230 | 'custom not exactly message' => [ |
||
231 | 'abcd', |
||
232 | [new HasLength(exactly: 3, notExactlyMessage: 'Custom not exactly message.')], |
||
233 | ['' => ['Custom not exactly message.']], |
||
234 | ], |
||
235 | 'custom not exactly message with parameters' => [ |
||
236 | 'abcd', |
||
237 | [ |
||
238 | new HasLength( |
||
239 | exactly: 3, |
||
240 | notExactlyMessage: 'Exactly - {exactly}, attribute - {attribute}, number - {number}.', |
||
241 | ), |
||
242 | ], |
||
243 | ['' => ['Exactly - 3, attribute - , number - 4.']], |
||
244 | ], |
||
245 | 'custom not exactly message with parameters, attribute set' => [ |
||
246 | ['data' => 'abcd'], |
||
247 | [ |
||
248 | 'data' => new HasLength( |
||
249 | exactly: 3, |
||
250 | notExactlyMessage: 'Exactly - {exactly}, attribute - {attribute}, number - {number}.', |
||
251 | ), |
||
252 | ], |
||
253 | ['data' => ['Exactly - 3, attribute - data, number - 4.']], |
||
254 | ], |
||
279 |