Complex classes like LimitSubqueryOutputWalker 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 LimitSubqueryOutputWalker, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
44 | class LimitSubqueryOutputWalker extends SqlWalker |
||
45 | { |
||
46 | /** |
||
47 | * @var \Doctrine\DBAL\Platforms\AbstractPlatform |
||
48 | */ |
||
49 | private $platform; |
||
|
|||
50 | |||
51 | /** |
||
52 | * @var \Doctrine\ORM\Query\ResultSetMapping |
||
53 | */ |
||
54 | private $rsm; |
||
55 | |||
56 | /** |
||
57 | * @var array |
||
58 | */ |
||
59 | private $queryComponents; |
||
60 | |||
61 | /** |
||
62 | * @var int |
||
63 | */ |
||
64 | private $firstResult; |
||
65 | |||
66 | /** |
||
67 | * @var int |
||
68 | */ |
||
69 | private $maxResults; |
||
70 | |||
71 | /** |
||
72 | * @var \Doctrine\ORM\EntityManager |
||
73 | */ |
||
74 | private $em; |
||
75 | |||
76 | /** |
||
77 | * The quote strategy. |
||
78 | * |
||
79 | * @var \Doctrine\ORM\Mapping\QuoteStrategy |
||
80 | */ |
||
81 | private $quoteStrategy; |
||
82 | |||
83 | /** |
||
84 | * @var array |
||
85 | */ |
||
86 | private $orderByPathExpressions = []; |
||
87 | |||
88 | /** |
||
89 | * @var bool We don't want to add path expressions from sub-selects into the select clause of the containing query. |
||
90 | * This state flag simply keeps track on whether we are walking on a subquery or not |
||
91 | */ |
||
92 | private $inSubSelect = false; |
||
93 | |||
94 | /** |
||
95 | * Constructor. |
||
96 | * |
||
97 | * Stores various parameters that are otherwise unavailable |
||
98 | * because Doctrine\ORM\Query\SqlWalker keeps everything private without |
||
99 | * accessors. |
||
100 | * |
||
101 | * @param \Doctrine\ORM\Query $query |
||
102 | * @param \Doctrine\ORM\Query\ParserResult $parserResult |
||
103 | * @param array $queryComponents |
||
104 | */ |
||
105 | 51 | public function __construct($query, $parserResult, array $queryComponents) |
|
121 | |||
122 | /** |
||
123 | * Check if the platform supports the ROW_NUMBER window function. |
||
124 | * |
||
125 | * @return bool |
||
126 | */ |
||
127 | 51 | private function platformSupportsRowNumber() |
|
137 | |||
138 | /** |
||
139 | * Rebuilds a select statement's order by clause for use in a |
||
140 | * ROW_NUMBER() OVER() expression. |
||
141 | * |
||
142 | * @param SelectStatement $AST |
||
143 | */ |
||
144 | 3 | private function rebuildOrderByForRowNumber(SelectStatement $AST) |
|
166 | |||
167 | /** |
||
168 | * Walks down a SelectStatement AST node, wrapping it in a SELECT DISTINCT. |
||
169 | * |
||
170 | * @param SelectStatement $AST |
||
171 | * |
||
172 | * @return string |
||
173 | * |
||
174 | * @throws \RuntimeException |
||
175 | */ |
||
176 | 51 | public function walkSelectStatement(SelectStatement $AST) |
|
183 | |||
184 | /** |
||
185 | * Walks down a SelectStatement AST node, wrapping it in a SELECT DISTINCT. |
||
186 | * This method is for use with platforms which support ROW_NUMBER. |
||
187 | * |
||
188 | * @param SelectStatement $AST |
||
189 | * |
||
190 | * @return string |
||
191 | * |
||
192 | * @throws \RuntimeException |
||
193 | */ |
||
194 | 5 | public function walkSelectStatementWithRowNumber(SelectStatement $AST) |
|
195 | { |
||
196 | 5 | $hasOrderBy = false; |
|
197 | 5 | $outerOrderBy = ' ORDER BY dctrn_minrownum ASC'; |
|
198 | 5 | $orderGroupBy = ''; |
|
199 | 5 | if ($AST->orderByClause instanceof OrderByClause) { |
|
200 | 3 | $hasOrderBy = true; |
|
201 | 3 | $this->rebuildOrderByForRowNumber($AST); |
|
202 | } |
||
203 | |||
204 | 5 | $innerSql = $this->getInnerSQL($AST); |
|
205 | |||
206 | 5 | $sqlIdentifier = $this->getSQLIdentifier($AST); |
|
207 | |||
208 | 5 | if ($hasOrderBy) { |
|
209 | 3 | $orderGroupBy = ' GROUP BY ' . implode(', ', $sqlIdentifier); |
|
210 | 3 | $sqlIdentifier[] = 'MIN(' . $this->walkResultVariable('dctrn_rownum') . ') AS dctrn_minrownum'; |
|
211 | } |
||
212 | |||
213 | // Build the counter query |
||
214 | 5 | $sql = sprintf( |
|
215 | 5 | 'SELECT DISTINCT %s FROM (%s) dctrn_result', |
|
216 | 5 | implode(', ', $sqlIdentifier), |
|
217 | 5 | $innerSql |
|
218 | ); |
||
219 | |||
220 | 5 | if ($hasOrderBy) { |
|
221 | 3 | $sql .= $orderGroupBy . $outerOrderBy; |
|
222 | } |
||
223 | |||
224 | // Apply the limit and offset. |
||
225 | 5 | $sql = $this->platform->modifyLimitQuery( |
|
226 | $sql, |
||
227 | 5 | $this->maxResults, |
|
228 | 5 | $this->firstResult |
|
229 | ); |
||
230 | |||
231 | // Add the columns to the ResultSetMapping. It's not really nice but |
||
232 | // it works. Preferably I'd clear the RSM or simply create a new one |
||
233 | // but that is not possible from inside the output walker, so we dirty |
||
234 | // up the one we have. |
||
235 | 5 | foreach ($sqlIdentifier as $property => $alias) { |
|
236 | 5 | $this->rsm->addScalarResult($alias, $property); |
|
237 | } |
||
238 | |||
239 | 5 | return $sql; |
|
240 | } |
||
241 | |||
242 | /** |
||
243 | * Walks down a SelectStatement AST node, wrapping it in a SELECT DISTINCT. |
||
244 | * This method is for platforms which DO NOT support ROW_NUMBER. |
||
245 | * |
||
246 | * @param SelectStatement $AST |
||
247 | * @param bool $addMissingItemsFromOrderByToSelect |
||
248 | * |
||
249 | * @return string |
||
250 | * |
||
251 | * @throws \RuntimeException |
||
252 | */ |
||
253 | 46 | public function walkSelectStatementWithoutRowNumber(SelectStatement $AST, $addMissingItemsFromOrderByToSelect = true) |
|
297 | |||
298 | /** |
||
299 | * Finds all PathExpressions in an AST's OrderByClause, and ensures that |
||
300 | * the referenced fields are present in the SelectClause of the passed AST. |
||
301 | * |
||
302 | * @param SelectStatement $AST |
||
303 | */ |
||
304 | 40 | private function addMissingItemsFromOrderByToSelect(SelectStatement $AST) |
|
353 | |||
354 | /** |
||
355 | * Generates new SQL for statements with an order by clause |
||
356 | * |
||
357 | * @param array $sqlIdentifier |
||
358 | * @param string $innerSql |
||
359 | * @param string $sql |
||
360 | * @param OrderByClause $orderByClause |
||
361 | * |
||
362 | * @return string |
||
363 | */ |
||
364 | 46 | private function preserveSqlOrdering(array $sqlIdentifier, $innerSql, $sql, $orderByClause) |
|
365 | { |
||
366 | // If the sql statement has an order by clause, we need to wrap it in a new select distinct |
||
367 | // statement |
||
368 | 46 | if (! $orderByClause instanceof OrderByClause) { |
|
369 | 6 | return $sql; |
|
370 | } |
||
371 | |||
372 | // Rebuild the order by clause to work in the scope of the new select statement |
||
373 | /* @var array $orderBy an array of rebuilt order by items */ |
||
374 | 40 | $orderBy = $this->rebuildOrderByClauseForOuterScope($orderByClause); |
|
375 | |||
376 | // Build the select distinct statement |
||
377 | 40 | $sql = sprintf( |
|
378 | 40 | 'SELECT DISTINCT %s FROM (%s) dctrn_result ORDER BY %s', |
|
379 | 40 | implode(', ', $sqlIdentifier), |
|
380 | $innerSql, |
||
381 | 40 | implode(', ', $orderBy) |
|
382 | ); |
||
383 | |||
384 | 40 | return $sql; |
|
385 | } |
||
386 | |||
387 | /** |
||
388 | * Generates a new order by clause that works in the scope of a select query wrapping the original |
||
389 | * |
||
390 | * @param OrderByClause $orderByClause |
||
391 | * @return array |
||
392 | */ |
||
393 | 40 | private function rebuildOrderByClauseForOuterScope(OrderByClause $orderByClause) |
|
394 | { |
||
395 | $dqlAliasToSqlTableAliasMap |
||
396 | = $searchPatterns |
||
397 | = $replacements |
||
398 | = $dqlAliasToClassMap |
||
399 | = $selectListAdditions |
||
400 | = $orderByItems |
||
401 | 40 | = []; |
|
402 | |||
403 | // Generate DQL alias -> SQL table alias mapping |
||
404 | 40 | foreach(array_keys($this->rsm->aliasMap) as $dqlAlias) { |
|
405 | 40 | $dqlAliasToClassMap[$dqlAlias] = $class = $this->queryComponents[$dqlAlias]['metadata']; |
|
406 | 40 | $dqlAliasToSqlTableAliasMap[$dqlAlias] = $this->getSQLTableAlias($class->getTableName(), $dqlAlias); |
|
407 | } |
||
408 | |||
409 | // Pattern to find table path expressions in the order by clause |
||
410 | 40 | $fieldSearchPattern = '/(?<![a-z0-9_])%s\.%s(?![a-z0-9_])/i'; |
|
411 | |||
412 | // Generate search patterns for each field's path expression in the order by clause |
||
413 | 40 | foreach($this->rsm->fieldMappings as $fieldAlias => $fieldName) { |
|
414 | 40 | $dqlAliasForFieldAlias = $this->rsm->columnOwnerMap[$fieldAlias]; |
|
415 | 40 | $class = $dqlAliasToClassMap[$dqlAliasForFieldAlias]; |
|
416 | |||
417 | // If the field is from a joined child table, we won't be ordering |
||
418 | // on it. |
||
419 | 40 | if (!isset($class->fieldMappings[$fieldName])) { |
|
420 | 1 | continue; |
|
421 | } |
||
422 | |||
423 | 40 | $fieldMapping = $class->fieldMappings[$fieldName]; |
|
424 | |||
425 | // Get the proper column name as will appear in the select list |
||
426 | 40 | $columnName = $this->quoteStrategy->getColumnName( |
|
427 | $fieldName, |
||
428 | 40 | $dqlAliasToClassMap[$dqlAliasForFieldAlias], |
|
429 | 40 | $this->em->getConnection()->getDatabasePlatform() |
|
430 | ); |
||
431 | |||
432 | // Get the SQL table alias for the entity and field |
||
433 | 40 | $sqlTableAliasForFieldAlias = $dqlAliasToSqlTableAliasMap[$dqlAliasForFieldAlias]; |
|
434 | 40 | if (isset($fieldMapping['declared']) && $fieldMapping['declared'] !== $class->name) { |
|
435 | // Field was declared in a parent class, so we need to get the proper SQL table alias |
||
436 | // for the joined parent table. |
||
437 | 2 | $otherClassMetadata = $this->em->getClassMetadata($fieldMapping['declared']); |
|
438 | 2 | if (!$otherClassMetadata->isMappedSuperclass) { |
|
439 | 1 | $sqlTableAliasForFieldAlias = $this->getSQLTableAlias($otherClassMetadata->getTableName(), $dqlAliasForFieldAlias); |
|
440 | } |
||
441 | } |
||
442 | |||
443 | // Compose search/replace patterns |
||
444 | 40 | $searchPatterns[] = sprintf($fieldSearchPattern, $sqlTableAliasForFieldAlias, $columnName); |
|
445 | 40 | $replacements[] = $fieldAlias; |
|
446 | } |
||
447 | |||
448 | 40 | foreach($orderByClause->orderByItems as $orderByItem) { |
|
449 | // Walk order by item to get string representation of it |
||
450 | 40 | $orderByItemString = $this->walkOrderByItem($orderByItem); |
|
451 | |||
452 | // Replace path expressions in the order by clause with their column alias |
||
453 | 40 | $orderByItemString = preg_replace($searchPatterns, $replacements, $orderByItemString); |
|
454 | |||
455 | 40 | $orderByItems[] = $orderByItemString; |
|
456 | } |
||
457 | |||
458 | 40 | return $orderByItems; |
|
459 | } |
||
460 | |||
461 | /** |
||
462 | * getter for $orderByPathExpressions |
||
463 | * |
||
464 | * @return array |
||
465 | */ |
||
466 | 40 | public function getOrderByPathExpressions() |
|
470 | |||
471 | /** |
||
472 | * @param SelectStatement $AST |
||
473 | * |
||
474 | * @return string |
||
475 | * |
||
476 | * @throws \Doctrine\ORM\OptimisticLockException |
||
477 | * @throws \Doctrine\ORM\Query\QueryException |
||
478 | */ |
||
479 | 51 | private function getInnerSQL(SelectStatement $AST) |
|
500 | |||
501 | /** |
||
502 | * @param SelectStatement $AST |
||
503 | * |
||
504 | * @return array |
||
505 | */ |
||
506 | 51 | private function getSQLIdentifier(SelectStatement $AST) |
|
559 | |||
560 | /** |
||
561 | * {@inheritdoc} |
||
562 | */ |
||
563 | 41 | public function walkPathExpression($pathExpr) |
|
571 | |||
572 | /** |
||
573 | * {@inheritdoc} |
||
574 | */ |
||
575 | public function walkSubSelect($subselect) |
||
585 | } |
||
586 |