Как сделать цикл в классе?

Question

[ne-programmist]

Есть такой код:

# БД:
# name | age |
# ------------
# John | 20  |
# Bob  | 20  |
# July | 30  |

class Filter
{
	public function age()
	{
		$users = # запрос в бд

		foreach ($users as $user)
		{
			if ($_GET['age'] == 20)
			{
				return $user->name;
			}
			elseif ($_GET['age'] == 30)
			{
				return $user->name;
			}
		}
	}
}

echo Filter::age($user->name);

Результат:
John
А нужно:
John, Bob

Как сделать?

Comments

[QNA-1976]

возраст параметром передать в метод и использовать в условии выборки

[BATPYIIIKOB]

QNA-1976, очевидно у него несколько возрастов, прийдётся передавать массивом, и далее сравнивать со значениями массива

[ne-programmist]

FanatPHP, потому что $_GET['age'] равно 20
запрос в БД выглядит так:

$users = R::findAll('users', 'age = ?', [$_GET['age']]);

[Sergej]

Сложить в массив и вывести нужное в конце метода.

$result = [];
foreach...;
if () $result[] = $user->name;

...

[FanatPHP]

У тебя запрос УЖЕ отфильтровал всех тех кому 20 лет и УЖЕ вернул массив. Никаких "циклов" не нужно

[ne-programmist]

FanatPHP, в смысле не нужно? а как их мне вывести?

[romaaa32]

ne-programmist, запрос вернул имена 20-летних пользователей в массиве? как вывести массив?

[ne-programmist]

romaaa32, лайк) спасибо

[romaaa32]

ne-programmist, напишите в ответах как вы решили свою проблему и отметьте ее как решение. Может кому-то пригодится.

Answer 1

[BATPYIIIKOB]

Вообще, правильно - это сделать 2 запроса на выборку из БД и не городить огород

return $user->name;
возвращает одно значение. У пользователя не может 2 имени. В данном случае вы прерываете выполнения цикла (выходом из функции) при соблюдении условия.
Если необходимо не прерывать цикл а собрать все значения по условию, добавляете в новый массив необходимые значения (имена), и только в конце return

Массив:

$user_age = [
   ["20"] => ["Иван","Света"],
   ["30"] => ["Катя"]   
]

Comments

[Антон Шаманов]

сделать 2 запроса на выборку из БД и не городить огород

:)

[ne-programmist]

Антон Шаманов, где сделать 2 запроса?

[Антон Шаманов]

ne-programmist, вначале спокойно освой работу с массивами.

[BATPYIIIKOB]

ne-programmist, получай данные по мере их необходимости, т.е.
public function age( age ) - передай сюда возраст, и далее запросом вытащи данные из БД.
После ,по необходимости, выполни второй запрос (точно также):

$filter = new Filter();
$result = $filter->age(25);

Какой смысл использовать классы и не использовать ООП и его возможности? Класс ради класса?

И по $_GET['age'], что будет, если$_GET['age'] не существует? Где проверки? Сделай проверку на существование (з.ы. сделай валидацию), передай его как параметр метода и получи ответ.

З.Ы.: Возраст также можно передавать массивом значений

[Антон Шаманов]

BATPYIIIKOB, подобный подход оправдан только для полей вроде TEXT и то далеко не всегда