Почему php скрипт игнорирует функцию?

Question

[Don2Quixote]

<?php
	$arr = array(1, 2, 3, 4, 5);
	function geomPr($b1, $q) {
		$arr[1] = $b1;
		for ($n = 1; $n < 11; $n++) {
			if ($n == 1) {
				$b_previous = $b1;
			} else {
				$b_previous = $b_previous * $q;
				$b = $b_previous * $q;
				$arr[$n] = $b;
			}  
		}
	}

	geomPr(2, 2);
	print $arr[2];
?>

В начале кода в массиве указаны значения. arr[2] = 3. В конце кода должно вывестись значение arr[2]. По идее, это значени должно быть равно 4. Но выводится 3. (Потому что в начале кода в массиве указано такое значение.) То есть вызов функции geomPr(2, 2); Игнорируется компилятором. Почему так происходит и как это исправить?

Answer 1

[Stalker_RED]

Вот он, ваш$arr[2]


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

Замыкания в php есть, но они не применяются к каждой функции автоматически, как в js, и если вам они действительно нужны, придется делать это самостоятельно, как-то так: https://ideone.com/HvsrC2

И вообще, в таких простых случаях проще передать массив как параметр: https://ideone.com/JMIntQ

А еще лучше так: https://ideone.com/aWaK70
В таком случае более явно видно, что функция изменит массив $arr.

Comments

[Don2Quixote]

Но ведь массив arr[] - он внешний. Функция его меняет, то есть она не обязана что то возвращать, так как в ходе своего выполнения она изменяет массив из вне.

[Don2Quixote]

Расскажите тогда, как исправить ситуацию )

[Stalker_RED]

Артём Зиновьев, это не js. Вы не передаете массив в функцию, вы не возвращаете результат.

Замыкания в php есть, но они не применяются к каждой функции автоматически, как в js, и если вам они действительно нужны, придется делать это самостоятельно, как-то так: https://ideone.com/HvsrC2

И вообще, в таких простых случаях проще передать массив как параметр.

А еще лучше так: https://ideone.com/aWaK70
В таком случае более явно видно, что функция изменит массив $arr.

[Stalker_RED]

Артём Зиновьев, дополнил ответ.

[Don2Quixote]

Stalker_RED, К сожалению, с вашим решением вылетала ошибка, но я нашел такой вариант:

<?php
	function geomPr($b1, $q) {
		global $arr;
		$arr[1] = $b1;
		for ($n = 1; $n < 11; $n++) {
			if ($n == 1) {
				$b_previous = $b1;
			} else {
				$b_previous = $b_previous * $q;
				$b = $b_previous * $q;
				$arr[$n] = $b;
			}  
		}
	}

	geomPr(2, 2);
	print $arr[1]."<br>";
	print $arr[2]."<br>";
	print $arr[3]."<br>";
	print $arr[4]."<br>";
?>

Я объявил сам массив внутри функции как глобальный.

[Stalker_RED]

Артём Зиновьев, ну круто, предавайте привет в двадцатый век.

[Don2Quixote]

Stalker_RED, А чем это решение вам не угодило ?)
Оно как то уступает по функциональности/скорости другим решениям ?)

[Stalker_RED]

Артём Зиновьев, По скорости скорее всего не уступает.

1. Глобальные переменные в большинстве случаев нарушают инкапсуляцию. К ним открыт неконтролируемый доступ отовсюду.
   
2. В большом проекте при обилии глобальных переменных возникает путаница в именах. Глобальную переменную же видно отовсюду, надо, чтобы отовсюду было понятно, зачем она.
   
3. Глобальные переменные в большинстве случаев нарушают принцип инверсии зависимостей (или делают возможным его нарушение).
   
4. Глобальные переменные ухудшают масштабируемость проекта.
   
5. Глобальные переменные ухудшают читаемость кода (в каком-то конкретно взятом месте непонятно, нужна ли какая-то конкретная глобальная переменная, или нет).
   
6. Глобальные переменные приводят к трудноуловимым ошибкам. Примеры: нежелательное изменение её значения в другом месте/другим потоком, ошибочное использование глобальной переменной для промежуточных вычислений из-за совпадения имен, возвращение функцией неправильного значения при тех же параметрах (оказывается, она зависима от глобальной переменной, а ее кто-то поменял).
   
7. Глобальные переменные создают большие сложности при использовании модульного тестирования.
   
8. Глобальные переменные увеличивают число прямых и косвенных связей в системе, делая её поведение труднопредсказуемым, а её саму - сложной для понимания и развития.
   

Копипаста оттуда: почему глобальные переменные это плохо

В любом более-менее крупном проекте, главные вопросы при чтении кода это не "насколько он быстр", а "что вообще делает эта штука, как она это делает, и ЗАЧЕМ". Вы конечно можете возразить, что у вас пока маленькие проекты, и таких вопросов не возникает. Но если вы в будущем будете работать над чем-то сложным, то стоит ли сейчас развивать в себе плохие привычки?

Answer 2

[kally]

Вы к внешней переменной $arr обращаетесь.
Внутри функции $arr = $b1;
В конце функции необходимо вернуть значение

return $arr[$n];
}

а при печати:

$geomPr = geomPr(2, 2);
echo $geomPr;

Если Вы не возвращаете значение return $arr[$n]; внутри функции, то система не знает, что печатать или брать на вычисление.