Есть массив из 15 элементов ['a','a','a', 'd', ... 'e']
, он подаётся на вход.
Если посмотреть по-другому, то можно представить его вот так:
---------elements----- -------keys---------------
[a][d][e][f][a]
| [0][3][6][9][12]
[a][e][f][e][g]
| [1][4][7][10][13]
[a][b][b][d][e]
| [2][5][8][11][14]
Задача: Найти линии в которых больше чем 3 одинаковых элемента подряд. Линий всего 10: три по строкам, пять по столбцам и по диагонали [0,4,8,10,12]
, [2,4,6,10,14]
.
Пример: в линии [0,1,2]
есть три одинаковых элемента (a
), в линии [2,4,6,10,14]
4 элемента (e
)
Функция возвращает линии и количество одинаковых элементов в ней.
Вопрос: Как можно упростить и сделать более понятной эту функцию? Не слишком ли длинный метод? Любые замечания приветсвуются.
public function findLines(array $input)
{
$checkingLines = [
[0,1,2],
[3,4,5],
[6,7,8],
[9,10,11],
[12,13,14],
[0,3,6,9,12],
[1,4,7,10,13],
[2,5,8,11,14],
[0,4,8,10,12],
[2,4,6,10,14]
];
$lines = array();
foreach ($checkingLines as $checkingLineNumber => $line) {
$matchesNumber = 0;
$matchesNumberMax = 0;
foreach ($line as $elementNumber => $element) {
if ($matchesNumberMax < $matchesNumber)
$matchesNumberMax = $matchesNumber;
$matchesNumber = 0;
$nextElement = $element;
$i = $elementNumber;
while ($input[$element] == $input[$nextElement] && $elementNumber < count($line)) {
$matchesNumber++;
$i++;
if (isset($line[$i]))
$nextElement = $line[$i];
else
break;
}
}
if ($matchesNumberMax >= 3) {
$lines[$checkingLineNumber] = [
'elements' => $line,
'consecutives' => $matchesNumberMax
];
}
}
return $lines;
}