Complex classes like Runtime often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Runtime, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
26 | class Runtime |
||
27 | { |
||
28 | const DEBUG_ERROR_LOG = 1; |
||
29 | const DEBUG_ERROR_EXCEPTION = 2; |
||
30 | const DEBUG_TAGS = 4; |
||
31 | const DEBUG_TAGS_ANSI = 12; |
||
32 | const DEBUG_TAGS_HTML = 20; |
||
33 | |||
34 | /** |
||
35 | * LightnCandy runtime method for output debug info. |
||
36 | * |
||
37 | * @param string $v expression |
||
38 | * @param string $f runtime function name |
||
39 | * @param array<string,array|string|integer> $cx render time context |
||
40 | * |
||
41 | * @expect '{{123}}' when input '123', 'miss', array('flags' => array('debug' => Runtime::DEBUG_TAGS), 'runtime' => 'LightnCandy\\Runtime'), '' |
||
42 | * @expect '<!--MISSED((-->{{#123}}<!--))--><!--SKIPPED--><!--MISSED((-->{{/123}}<!--))-->' when input '123', 'wi', array('flags' => array('debug' => Runtime::DEBUG_TAGS_HTML), 'runtime' => 'LightnCandy\\Runtime'), false, false, function () {return 'A';} |
||
43 | */ |
||
44 | 9 | public static function debug($v, $f, $cx) { |
|
76 | |||
77 | /** |
||
78 | * LightnCandy runtime method for error |
||
79 | * |
||
80 | * @param array<string,array|string|integer> $cx render time context |
||
81 | * @param string $err error message |
||
82 | * |
||
83 | * @throws \Exception |
||
84 | */ |
||
85 | 8 | public static function err($cx, $err) { |
|
94 | |||
95 | /** |
||
96 | * LightnCandy runtime method for missing data error. |
||
97 | * |
||
98 | * @param array<string,array|string|integer> $cx render time context |
||
99 | * @param string $v expression |
||
100 | */ |
||
101 | 5 | public static function miss($cx, $v) { |
|
104 | |||
105 | /** |
||
106 | * LightnCandy runtime method for variable lookup. It is slower and only be used for instance property or method detection or lambdas. |
||
107 | * |
||
108 | * @param array<string,array|string|integer> $cx render time context |
||
109 | * @param array<array|string|integer> $base current variable context |
||
110 | * @param array<string|integer> $path array of names for path |
||
111 | * |
||
112 | * @return null|string Return the value or null when not found |
||
113 | * |
||
114 | * @expect null when input array('scopes' => array(), 'flags' => array('prop' => 0, 'method' => 0, 'mustlok' => 0)), 0, array('a', 'b') |
||
115 | * @expect 3 when input array('scopes' => array(), 'flags' => array('prop' => 0, 'method' => 0), 'mustlok' => 0), array('a' => array('b' => 3)), array('a', 'b') |
||
116 | * @expect null when input array('scopes' => array(), 'flags' => array('prop' => 0, 'method' => 0, 'mustlok' => 0)), (Object) array('a' => array('b' => 3)), array('a', 'b') |
||
117 | * @expect 3 when input array('scopes' => array(), 'flags' => array('prop' => 1, 'method' => 0, 'mustlok' => 0)), (Object) array('a' => array('b' => 3)), array('a', 'b') |
||
118 | */ |
||
119 | 336 | public static function v($cx, $base, $path) { |
|
164 | |||
165 | /** |
||
166 | * LightnCandy runtime method for {{#if var}}. |
||
167 | * |
||
168 | * @param array<string,array|string|integer> $cx render time context |
||
169 | * @param array<array|string|integer>|string|integer|null $v value to be tested |
||
170 | * @param boolean $zero include zero as true |
||
171 | * |
||
172 | * @return boolean Return true when the value is not null nor false. |
||
173 | * |
||
174 | * @expect false when input array(), null, false |
||
175 | * @expect false when input array(), 0, false |
||
176 | * @expect true when input array(), 0, true |
||
177 | * @expect false when input array(), false, false |
||
178 | * @expect true when input array(), true, false |
||
179 | * @expect true when input array(), 1, false |
||
180 | * @expect false when input array(), '', false |
||
181 | * @expect false when input array(), array(), false |
||
182 | * @expect true when input array(), array(''), false |
||
183 | * @expect true when input array(), array(0), false |
||
184 | */ |
||
185 | 54 | public static function ifvar($cx, $v, $zero) { |
|
188 | |||
189 | /** |
||
190 | * LightnCandy runtime method for {{#if var}} when {{../var}} used. |
||
191 | * |
||
192 | * @param array<string,array|string|integer> $cx render time context |
||
193 | * @param array<array|string|integer>|string|integer|null $v value to be tested |
||
194 | * @param boolean $zero include zero as true |
||
195 | * @param array<array|string|integer> $in input data with current scope |
||
196 | * @param Closure|null $truecb callback function when test result is true |
||
197 | * @param Closure|null $falsecb callback function when test result is false |
||
198 | * |
||
199 | * @return string The rendered string of the section |
||
200 | * |
||
201 | * @expect '' when input array('scopes' => array()), null, false, array(), null |
||
202 | * @expect '' when input array('scopes' => array()), null, false, array(), function () {return 'Y';} |
||
203 | * @expect 'Y' when input array('scopes' => array()), 1, false, array(), function () {return 'Y';} |
||
204 | * @expect 'N' when input array('scopes' => array()), null, false, array(), function () {return 'Y';}, function () {return 'N';} |
||
205 | */ |
||
206 | 1 | public static function ifv($cx, $v, $zero, $in, $truecb, $falsecb = null) { |
|
218 | |||
219 | /** |
||
220 | * LightnCandy runtime method for {{#unless var}} when {{../var}} used. |
||
221 | * |
||
222 | * @param array<string,array|string|integer> $cx render time context |
||
223 | * @param array<array|string|integer>|string|integer|null $var value be tested |
||
224 | * @param boolean $zero include zero as true |
||
225 | * @param array<array|string|integer>|string|integer|null $in input data with current scope |
||
226 | * @param Closure $truecb callback function when test result is true |
||
227 | * @param Closure|null $falsecb callback function when test result is false |
||
228 | * |
||
229 | * @return string Return rendered string when the value is not null nor false. |
||
230 | * |
||
231 | * @expect '' when input array('scopes' => array()), null, false, array(), null |
||
232 | * @expect 'Y' when input array('scopes' => array()), null, false, array(), function () {return 'Y';} |
||
233 | * @expect '' when input array('scopes' => array()), 1, false, array(), function () {return 'Y';} |
||
234 | * @expect 'Y' when input array('scopes' => array()), null, false, array(), function () {return 'Y';}, function () {return 'N';} |
||
235 | * @expect 'N' when input array('scopes' => array()), true, false, array(), function () {return 'Y';}, function () {return 'N';} |
||
236 | */ |
||
237 | 1 | public static function unl($cx, $var, $zero, $in, $truecb, $falsecb = null) { |
|
240 | |||
241 | /** |
||
242 | * LightnCandy runtime method for {{^var}} inverted section. |
||
243 | * |
||
244 | * @param array<string,array|string|integer> $cx render time context |
||
245 | * @param array<array|string|integer>|string|integer|null $v value to be tested |
||
246 | * |
||
247 | * @return boolean Return true when the value is not null nor false. |
||
248 | * |
||
249 | * @expect true when input array(), null |
||
250 | * @expect false when input array(), 0 |
||
251 | * @expect true when input array(), false |
||
252 | * @expect false when input array(), 'false' |
||
253 | * @expect true when input array(), array() |
||
254 | * @expect false when input array(), array('1') |
||
255 | */ |
||
256 | 36 | public static function isec($cx, $v) { |
|
259 | |||
260 | /** |
||
261 | * LightnCandy runtime method for {{{var}}} . |
||
262 | * |
||
263 | * @param array<string,array|string|integer> $cx render time context |
||
264 | * @param array<array|string|integer>|string|integer|null $v value to be output |
||
265 | * |
||
266 | * @return string The raw value of the specified variable |
||
267 | * |
||
268 | * @expect true when input array('flags' => array('jstrue' => 0, 'mustlam' => 0, 'lambda' => 0)), true |
||
269 | * @expect 'true' when input array('flags' => array('jstrue' => 1)), true |
||
270 | * @expect '' when input array('flags' => array('jstrue' => 0, 'mustlam' => 0, 'lambda' => 0)), false |
||
271 | * @expect 'false' when input array('flags' => array('jstrue' => 1)), false |
||
272 | * @expect 'false' when input array('flags' => array('jstrue' => 1)), false, true |
||
273 | * @expect 'Array' when input array('flags' => array('jstrue' => 1, 'jsobj' => 0)), array('a', 'b') |
||
274 | * @expect 'a,b' when input array('flags' => array('jstrue' => 1, 'jsobj' => 1, 'mustlam' => 0, 'lambda' => 0)), array('a', 'b') |
||
275 | * @expect '[object Object]' when input array('flags' => array('jstrue' => 1, 'jsobj' => 1)), array('a', 'c' => 'b') |
||
276 | * @expect '[object Object]' when input array('flags' => array('jstrue' => 1, 'jsobj' => 1)), array('c' => 'b') |
||
277 | * @expect 'a,true' when input array('flags' => array('jstrue' => 1, 'jsobj' => 1, 'mustlam' => 0, 'lambda' => 0)), array('a', true) |
||
278 | * @expect 'a,1' when input array('flags' => array('jstrue' => 0, 'jsobj' => 1, 'mustlam' => 0, 'lambda' => 0)), array('a',true) |
||
279 | * @expect 'a,' when input array('flags' => array('jstrue' => 0, 'jsobj' => 1, 'mustlam' => 0, 'lambda' => 0)), array('a',false) |
||
280 | * @expect 'a,false' when input array('flags' => array('jstrue' => 1, 'jsobj' => 1, 'mustlam' => 0, 'lambda' => 0)), array('a',false) |
||
281 | */ |
||
282 | 272 | public static function raw($cx, $v) { |
|
313 | |||
314 | /** |
||
315 | * LightnCandy runtime method for {{var}} . |
||
316 | * |
||
317 | * @param array<string,array|string|integer> $cx render time context |
||
318 | * @param array<array|string|integer>|string|integer|null $var value to be htmlencoded |
||
319 | * |
||
320 | * @return string The htmlencoded value of the specified variable |
||
321 | * |
||
322 | * @expect 'a' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a' |
||
323 | * @expect 'a&b' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a&b' |
||
324 | * @expect 'a'b' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a\'b' |
||
325 | */ |
||
326 | 44 | public static function enc($cx, $var) { |
|
329 | |||
330 | /** |
||
331 | * LightnCandy runtime method for {{var}} , and deal with single quote to same as handlebars.js . |
||
332 | * |
||
333 | * @param array<string,array|string|integer> $cx render time context |
||
334 | * @param array<array|string|integer>|string|integer|null $var value to be htmlencoded |
||
335 | * |
||
336 | * @return string The htmlencoded value of the specified variable |
||
337 | * |
||
338 | * @expect 'a' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a' |
||
339 | * @expect 'a&b' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a&b' |
||
340 | * @expect 'a'b' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), 'a\'b' |
||
341 | * @expect '`a'b' when input array('flags' => array('mustlam' => 0, 'lambda' => 0)), '`a\'b' |
||
342 | */ |
||
343 | 201 | public static function encq($cx, $var) { |
|
346 | |||
347 | /** |
||
348 | * LightnCandy runtime method for {{#var}} section. |
||
349 | * |
||
350 | * @param array<string,array|string|integer> $cx render time context |
||
351 | * @param array<array|string|integer>|string|integer|null $v value for the section |
||
352 | * @param array<array|string|integer>|string|integer|null $in input data with current scope |
||
353 | * @param boolean $each true when rendering #each |
||
354 | * @param Closure $cb callback function to render child context |
||
355 | * @param Closure|null $else callback function to render child context when {{else}} |
||
356 | * |
||
357 | * @return string The rendered string of the section |
||
358 | * |
||
359 | * @expect '' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), false, false, false, function () {return 'A';} |
||
360 | * @expect '' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), null, null, false, function () {return 'A';} |
||
361 | * @expect 'A' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), true, true, false, function () {return 'A';} |
||
362 | * @expect 'A' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 0, 0, false, function () {return 'A';} |
||
363 | * @expect '-a=' when input array('scopes' => array(), 'flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array('a'), array('a'), false, function ($c, $i) {return "-$i=";} |
||
364 | * @expect '-a=-b=' when input array('scopes' => array(), 'flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array('a','b'), array('a','b'), false, function ($c, $i) {return "-$i=";} |
||
365 | * @expect '' when input array('scopes' => array(), 'flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 'abc', 'abc', true, function ($c, $i) {return "-$i=";} |
||
366 | * @expect '-b=' when input array('scopes' => array(), 'flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array('a' => 'b'), array('a' => 'b'), true, function ($c, $i) {return "-$i=";} |
||
367 | * @expect '1' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 'b', 'b', false, function ($c, $i) {return count($i);} |
||
368 | * @expect '1' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 1, 1, false, function ($c, $i) {return print_r($i, true);} |
||
369 | * @expect '0' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 0, 0, false, function ($c, $i) {return print_r($i, true);} |
||
370 | * @expect '{"b":"c"}' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array('b' => 'c'), array('b' => 'c'), false, function ($c, $i) {return json_encode($i);} |
||
371 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array(), 0, true, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
372 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), array(), 0, false, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
373 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), false, 0, true, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
374 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), false, 0, false, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
375 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), '', 0, true, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
376 | * @expect 'cb' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), '', 0, false, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
377 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 0, 0, true, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
378 | * @expect 'cb' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), 0, 0, false, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
379 | * @expect 'inv' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), new stdClass, 0, true, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
380 | * @expect 'cb' when input array('flags' => array('spvar' => 0, 'mustlam' => 0, 'lambda' => 0)), new stdClass, 0, false, function ($c, $i) {return 'cb';}, function ($c, $i) {return 'inv';} |
||
381 | * @expect '268' when input array('scopes' => array(), 'flags' => array('spvar' => 1, 'mustlam' => 0, 'lambda' => 0), 'sp_vars'=>array('root' => 0)), array(1,3,4), 0, false, function ($c, $i) {return $i * 2;} |
||
382 | * @expect '038' when input array('scopes' => array(), 'flags' => array('spvar' => 1, 'mustlam' => 0, 'lambda' => 0), 'sp_vars'=>array('root' => 0)), array(1,3,'a'=>4), 0, true, function ($c, $i) {return $i * $c['sp_vars']['index'];} |
||
383 | */ |
||
384 | 151 | public static function sec($cx, $v, $in, $each, $cb, $else = null) { |
|
496 | |||
497 | /** |
||
498 | * LightnCandy runtime method for {{#with var}} . |
||
499 | * |
||
500 | * @param array<string,array|string|integer> $cx render time context |
||
501 | * @param array<array|string|integer>|string|integer|null $v value to be the new context |
||
502 | * @param array<array|string|integer>|string|integer|null $in input data with current scope |
||
503 | * @param Closure $cb callback function to render child context |
||
504 | * @param Closure|null $else callback function to render child context when {{else}} |
||
505 | * |
||
506 | * @return string The rendered string of the token |
||
507 | * |
||
508 | * @expect '' when input array(), false, false, function () {return 'A';} |
||
509 | * @expect '' when input array(), null, null, function () {return 'A';} |
||
510 | * @expect '{"a":"b"}' when input array(), array('a'=>'b'), array('a'=>'c'), function ($c, $i) {return json_encode($i);} |
||
511 | * @expect '-b=' when input array(), 'b', array('a'=>'b'), function ($c, $i) {return "-$i=";} |
||
512 | */ |
||
513 | 13 | public static function wi($cx, $v, $in, $cb, $else = null) { |
|
522 | |||
523 | /** |
||
524 | * LightnCandy runtime method for {{> partial}} . |
||
525 | * |
||
526 | * @param array<string,array|string|integer> $cx render time context |
||
527 | * @param string $p partial name |
||
528 | * @param array<array|string|integer>|string|integer|null $v value to be the new context |
||
529 | * |
||
530 | * @return string The rendered string of the partial |
||
531 | * |
||
532 | */ |
||
533 | 56 | public static function p($cx, $p, $v, $sp = '') { |
|
555 | |||
556 | /** |
||
557 | * LightnCandy runtime method for custom helpers. |
||
558 | * |
||
559 | * @param array<string,array|string|integer> $cx render time context |
||
560 | * @param string $ch the name of custom helper to be executed |
||
561 | * @param array<array> $vars variables for the helper |
||
562 | * @param string $op the name of variable resolver. should be one of: 'raw', 'enc', or 'encq'. |
||
563 | * |
||
564 | * @return string The rendered string of the token |
||
565 | * |
||
566 | * @expect '---' when input array('helpers' => array('a' => function ($i) {return "-$i[0]-";})), 'a', array(array('-'),array()), 'raw' |
||
567 | * @expect '-&-' when input array('helpers' => array('a' => function ($i) {return "-$i[0]-";})), 'a', array(array('&'),array()), 'enc' |
||
568 | * @expect '-'-' when input array('helpers' => array('a' => function ($i) {return "-$i[0]-";})), 'a', array(array('\''),array()), 'encq' |
||
569 | * @expect '-b-' when input array('helpers' => array('a' => function ($i,$j) {return "-{$j['a']}-";})), 'a', array(array(),array('a' => 'b')), 'raw' |
||
570 | */ |
||
571 | 22 | public static function ch($cx, $ch, $vars, $op) { |
|
574 | |||
575 | /** |
||
576 | * LightnCandy runtime method to handle response of custom helpers. |
||
577 | * |
||
578 | * @param string|array<string,array|string|integer> $ret return value from custom helper |
||
579 | * @param string $op the name of variable resolver. should be one of: 'raw', 'enc', or 'encq'. |
||
580 | * |
||
581 | * @return string The rendered string of the token |
||
582 | * |
||
583 | * @expect '-&-' when input '-&-', 'raw' |
||
584 | * @expect '-&'-' when input '-&\'-', 'enc' |
||
585 | * @expect '-&'-' when input '-&\'-', 'encq' |
||
586 | * @expect '-&'-' when input array('-&\'-'), 'enc' |
||
587 | * @expect '-&'-' when input array('-&\'-'), 'encq' |
||
588 | * @expect '-&-' when input array('-&-', false), 'enc' |
||
589 | * @expect '-&-' when input array('-&-', false), 'raw' |
||
590 | * @expect '-&-' when input array('-&-', 'raw'), 'enc' |
||
591 | * @expect '-&'-' when input array('-&\'-', 'encq'), 'raw' |
||
592 | */ |
||
593 | 144 | public static function chret($ret, $op) { |
|
609 | |||
610 | /** |
||
611 | * LightnCandy runtime method for Handlebars.js style custom helpers. |
||
612 | * |
||
613 | * @param array<string,array|string|integer> $cx render time context |
||
614 | * @param string $ch the name of custom helper to be executed |
||
615 | * @param array<array|string|integer>|string|integer|null $vars variables for the helper |
||
616 | * @param string $op the name of variable resolver. should be one of: 'raw', 'enc', or 'encq'. |
||
617 | * @param boolean $inverted the logic will be inverted |
||
618 | * @param Closure|null $cb callback function to render child context |
||
619 | * @param Closure|null $else callback function to render child context when {{else}} |
||
620 | * |
||
621 | * @return string The rendered string of the token |
||
622 | */ |
||
623 | 124 | public static function hbch($cx, $ch, $vars, $op, $inverted, $cb = null, $else = null) { |
|
703 | |||
704 | /** |
||
705 | * LightnCandy runtime method for block custom helpers. |
||
706 | * |
||
707 | * @param array<string,array|string|integer> $cx render time context |
||
708 | * @param string $ch the name of custom helper to be executed |
||
709 | * @param array<array|string|integer>|string|integer|null $vars variables for the helper |
||
710 | * @param array<array|string|integer>|string|integer|null $in input data with current scope |
||
711 | * @param boolean $inverted the logic will be inverted |
||
712 | * @param Closure $cb callback function to render child context |
||
713 | * @param Closure|null $else callback function to render child context when {{else}} |
||
714 | * |
||
715 | * @return string The rendered string of the token |
||
716 | * |
||
717 | * @expect '4.2.3' when input array('blockhelpers' => array('a' => function ($cx) {return array($cx,2,3);})), 'a', array(0, 0), 4, false, function($cx, $i) {return implode('.', $i);} |
||
718 | * @expect '2.6.5' when input array('blockhelpers' => array('a' => function ($cx,$in) {return array($cx,$in[0],5);})), 'a', array('6', 0), 2, false, function($cx, $i) {return implode('.', $i);} |
||
719 | * @expect '' when input array('blockhelpers' => array('a' => function ($cx,$in) {})), 'a', array('6', 0), 2, false, function($cx, $i) {return implode('.', $i);} |
||
720 | */ |
||
721 | 4 | public static function bch($cx, $ch, $vars, $in, $inverted, $cb, $else = null) { |
|
748 | } |
||
749 | |||
750 |
This check looks from parameters that have been defined for a function or method, but which are not used in the method body.