Сообщений в теме: 10

[TelecomMedia]

Подскажите в чем ошибка:

if ($line['Договор'] == '0' && $line['Сумма оплат'] == '0,00') {
	$line['Категория'] = 'Реквизиты';
}if ($line['Договор'] == '1' && $line['Сумма оплат'] == '0,00') {
	$line['Категория'] = 'Новый';
}if ($line['Сумма оплат'] != '0,00' && $line['Количество оплат'] != '0') {
	$line['Категория'] = 'Действующий';
}if ($line['Среднемесячное кол-во рассылок'] > '1') {
	$line['Категория'] = 'Активно работающий';
}if ($line['Сумма оплат'] > '50000' && $line['Среднемесячный трафик'] > '20000') {
	$line['Категория'] = 'VIP';
}

Доп действие должно присваивать Категорию в зависимости от условий. Но категория проставляется некорректно и практически всегда = "Действующий"
Прикладываю скрин

Прикрепленные изображения

• 

Сообщение отредактировал fabrika-r: 29 Апрель 2014 - 10:54

[CbCoder]

Условия исключают друг друга? Тогда должны быть elseif, начиная со второй строки.

[TelecomMedia]

Сделал с elseif. Все тоже самое получается.

[CbCoder]

"0,00" - все данные внутри базы хранятся в международном формате (точнее американском, т.к. они законодатели в it-сфере), в частности в цифрах используется точка. Более, того, "0.00" равно просто нулю, поэтому можно и рекомендуется писать так:

$line['Сумма оплат'] == 0

Примечание: это все относится к полям с типом "число", в текстовых полях все хранится так как было записано, без преобразований.

[TelecomMedia]

Да, спасибо, заработало.

[TelecomMedia]

Не срабатывает вот эта часть кода:

}elseif ($line['Среднемесячное кол-во рассылок'] > '1') {
	$line['Категория'] = 'Активно работающий';
}elseif ($line['Сумма оплат'] > '50000' && $line['Среднемесячный трафик'] > '20000') {
	$line['Категория'] = 'VIP';
}

Хотя есть значения в записях, которые должны были перевести поле Категория в значения "Активно работающий" или "VIP"

Сообщение отредактировал fabrika-r: 30 Апрель 2014 - 09:22

[CbCoder]

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

[TelecomMedia]

Все кавычки с чисел убрал, не помогло.

[CbCoder]

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

Например:

если яблоко зеленое, то крикнуть "ку" один раз
иначе если банан желтый, то махнуть рукой два раза

Так вот, если яблоко зеленое, то выполнится только "ку один раз", а до махания рукой дело не дойдет, даже если банан и в самом деле желтый.

[TelecomMedia]

))))

Т.е., чтобы воторое условие сработало должно быть так:

если яблоко зеленое, то крикнуть "ку" один раз
иначе если банан желтый и яблоко зеленое, то махнуть рукой два раза


так?

[CbCoder]

Не совсем. Речь о том, что если выполняется условие "если", то "иначе" в любом случае будет проигнорировано. Если нужно проверять оба условия независимо, то их нужно писать отдельно. В вашем случае будет:

если яблоко зеленое, то крикнуть "ку" один раз
если банан желтый и яблоко зеленое, то махнуть рукой два раза

Тогда до проверки цвета банана дело дойдет. Но, следует учесть, что если яблоко зеленое, то выполнятся оба условия, и второе может заменить первое, если к примеру заполняется одно и тоже поле. Так что важно расставить верно последовательность условий.