Здравейте,

за домашна имам да изкарам минималното и максималното число от даден масив, като НЕ МОГА да изпозвам функциите min и max а само с функцията foreach.

<?php

$numbers = array(376,23,56,34,56,234);
$min = PHP_INT_MAX;
$max = -PHP_INT_MAX;
foreach ($numbers as $number)
{
	if ($number > $max)
		$max = $number;
	if ($number < $min)
		$min = $number;
}

var_dump($min, $max);

?>

<?php

$numbers = array(376,23,56,34,56,234);
$min = PHP_INT_MAX;
$max = -PHP_INT_MAX;
foreach ($numbers as $number)
{
	if ($number > $max)
		$max = $number;
	if ($number < $min)
		$min = $number;
}

var_dump($min, $max);

?>

Кофти е да се ползват тия константи за инициализация... По-добре е да се инициализират с първия елемнт от масива.

Кофти е да се ползват тия константи за инициализация... По-добре е да се инициализират с първия елемнт от масива.

Какво имаш в предвид?

Какво имаш в предвид?

Ами, че е по-добре $min и $max да се инициализират с първия елемент на масива, вместо с някакви гранични константи. Особено пък в език, който не е много придирчив към типовете. И даденият код ще работи, но с това което предлагам, алгоритъмът ще може да работи и за сравняване на данни, за които не знаеш границите или работиш с потребителска функция за сравнение. Примерно ако горе решиш да минеш на числа с плаваща запетая, трябва да смениш и констаните, в моя вариант сменяш само данните.

Редактирано 7 декември 201411 г. от flare (преглед на промените)

Ами, че е по-добре $min и $max да се инициализират с първия елемент на масива, вместо с някакви гранични константи. Особено пък в език, който не е много придирчив към типовете. И даденият код ще работи, но с това което предлагам, алгоритъмът ще може да работи и за сравняване на данни, за които не знаеш границите или работиш с потребителска функция за сравнение. Примерно ако горе решиш да минеш на числа с плаваща запетая, трябва да смениш и констаните, в моя вариант сменяш само данните.

<?php

$numbers = array(376,23,56,34,56,234);
$min =$numbers[0];
$max =$min*(-1);
foreach ($numbers as $number)
{
	if ($number > $max)
		$max = $number;
	if ($number < $min)
		$min = $number;
}

var_dump($min, $max);

?>

Така ли?

Редактирано 7 декември 201411 г. от rado99 (преглед на промените)

...

Къде съм казал да умножаваш по -1?

Къде съм казал да умножаваш по -1?

-$min не е иденично с $min*(-1)

-$min не е иденично с $min*(-1)

Аз казах че в началото на цикъла двете променливи $min и $max трябва да са равни на първия елемент. Нe $max да е равно на $min с обратен знак. Така както си го направил е направо грешно.

Редактирано 7 декември 201411 г. от flare (преглед на промените)

...

Лаконично, но недопустимо.

и според условието, но по-засукано.

успех

Много благодаря за всички отзовали се... Оправих се с условието благодарение на уточненията от @flare и отчасти кода на gothicrock, за което много ви благодаря!