Комментарии на: Рефакторинг: введение объясняющих переменных

От: novice

novice — Sun, 09 Nov 2008 15:30:50 +0000

2 iwan: не воспринимает скорее всего не браузер, а php-интерпретатор ) Тут конечно будет ошибка, если держать этот код не в классе, в котором есть поля _quantity и _itemPrice. А вообще этот код просто для примера приведен, его даже пробовать не стоит Достаточно понять смысл приема.

От: iwan

iwan — Sun, 09 Nov 2008 10:22:31 +0000

Нашёл ошибку, по крайне мере у меня не воспринимает браузер пока не исправил: #
# function quantityDiscount() {
# return max(0, $this->_quantity – 500) * $this->_itemPrice * 0.06;
Кто нить уже пробовал код?

От: Игорь

Игорь — Mon, 03 Nov 2008 16:02:10 +0000

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