Можно ли вынести повторяющийся цикл foreach в отдельную функцию?

Question

[daylight6]

Возможно ли как-то вынести повторяющий цикл foreach из функций? Нужно написать отдельную функцию, где будет foreach, и в ней вызывать уже те, что в примере ниже? Будет ли код отрабатывать быстрее, если так сделать?

function getItemsDate()
{
	foreach ($posts as $item) {
		// ...
	}
	return $result;
}
function getItemsCategory()
{
	foreach ($posts as $item) {
		// ...
	}
	return $result2;
}

Comments

[John Didact]

Можно. Не благодари.

[Slava Rozhnev]

Можно. Зачем?

[Sergo Zar]

И ещё: нету смысла делать в разных функциях разные переменные (в смысле result, result2 и т.д

Это:

function myOneFunction(){
 $result = 2+2;
 return $result;
}
function myTwoFunction(){
 $result2 = 2+3;
 return $result2;
}

И это:

function myOneFunction(){
 $result = 2+2;
 return $result;
}
function myTwoFunction(){
 $result = 2+3;
 return $result;
}

Будут работать одинаково.
P.s. это так. На заметку

Answer 1

[Sergo Zar]

function posts_items($posts){
foreach ($posts as $item) {
    // ...
  }
// return то что тебе надо получить или то что получилось в цикле
}

Например

function getItemsDate()
{
  return posts_items($posts);

}
function getItemsCategory()
{
  return posts_items($posts);
}

P.s. надеюсь поймёте. я бы написал точнее если бы твоего кода было больше

Comments

[daylight6]

Код, который пытаюсь написать, вот:

function countAverageNumberOfWords()
{
	$allWordsCount = null;

	foreach ($this->posts as $item) {
		$itemWordsCount = count(preg_split('/\s+/', $item['description']));
		$allWordsCount += $itemWordsCount;
	}

	$averageWordsCount = $allWordsCount / count($this->posts);

	return $averageWordsCount;
}

function getItemsSortBySections()
{
	foreach ($this->posts as $item) {
		$arSections[$item['category']][] = $item;
	}
	return $arSections;
}

Это я пытаюсь выполнить задание, где мне сказали, что у меня цикл крутится один и тот же в каждой функции, и его надо вынести куда-то отдельно, чтобы был проход один раз.
Вот я не понимаю, с этими функциями это реально сделать или нет, и все итак нормально...

[Rsa97]

daylight6, Во второй функции у вас какой-то бред. Вы создаёте массив $sectionList, а затем на его основе точно такой же массив $arSections.

[daylight6]

Rsa97, ой, и правда. Лишнее, спасибо.

[Vitsliputsli]

daylight6,

Это я пытаюсь выполнить задание, где мне сказали, что у меня цикл крутится один и тот же в каждой функции, и его надо вынести куда-то отдельно, чтобы был проход один раз.
Вот я не понимаю, с этими функциями это реально сделать или нет, и все итак нормально...

от ваc хотят примерно это:

private int $averageWordsCount = 0;
private array $arSections = [];

function postsProcess(): void
{
	$allWordsCount = 0;
	foreach ($this->posts as $item) {
		$allWordsCount += count(preg_split('/\s+/', $item['description']));
		$this->arSections[$item['category']][] = $item;
	}
	$this->averageWordsCount = $allWordsCount / count($this->posts);
}

function countAverageNumberOfWords(): int
{
	return $this->averageWordsCount;
}

function getItemsSortBySections(): array
{
	return $this->arSections;
}

[daylight6]

Vitsliputsli, спасибо!