Почему $total не меняется?

Question

[Виктор Таран]

Тotal всегда одно и тоже значение

<?php
require 'connect.php';

$random = array(
	'workweek' => array(-4,7),
	'holiday' => array(-16,-4)
);

mysql_query("SELECT * FROM `betaintranet`.`ad_stat_cleaned` LEFT JOIN `betaintranet`.`ad_c` ON ad_stat_cleaned.bid = ad_c.bid WHERE `date`='".date("Y-m-d")."'");?>

 <?while($row = mysql_fetch_assoc($q)){
	if($row['w_site'] == '1'){    			//Если это Найдиквартиру.ру
		if($row['ban_type'] == '1'){    	     
			$total = 1650000; 				// Баннеры ТГБ  172
		}elseif($row['ban_type'] == '2'){
			$total = 1500000; 				// Баннеры ТГБ  240
		}elseif($row['ban_type'] == '3'){
			$total = 1500000; 				// Баннеры ТГБ  330
		}elseif($row['ban_type'] == '4'){
			$total = 1650000; 				//Красная строка  
		}elseif($row['ban_type'] == '5'){
			$total = 1500000; 				//Баннер 940x90, середина
		}elseif($row['ban_type'] == '6'){
			$total = 1500000; 				//Брендирование 
		}elseif($row['ban_type'] == '7'){
			$total = 1500000; 				//Баннер 940x90, верхний
		}elseif($row['ban_type'] == '8'){
			$total = 1500000; 				//Баннер 940x90, нижний
		}elseif($row['ban_type'] == '101'){
			$total = 1500000; 				//Перетяжка шапка 1240х90 
		}elseif($row['ban_type'] == '102'){
			$total = 1500000; 				//Перетяжка под фильтром 1240х90 (фиксированная)
		}elseif($row['ban_type'] == '103'){
			$total = 1500000; 				//1-ый правый баннер 240х400
		}elseif($row['ban_type'] == '104'){
			$total = 1500000; 				//2-ый правый баннер 240х400
		}elseif($row['ban_type'] == '105'){
			$total = 1500000; 				 //3-ый правый баннер 240х400
		}elseif($row['ban_type'] == '106'){
			$total = 1500000; 				 //Перетяжка в подвале 1240х300
		}
	 }
}

$w = mysql_query("SELECT * FROM `betaintranet`.`ad_stat_cleaned` LEFT JOIN `betaintranet`.`ad_c` ON ad_stat_cleaned.bid = ad_c.bid WHERE `date`='".date("Y-m-d")."'");?>

 <?while($row = mysql_fetch_assoc($w))
 {
 	$now = time();
 	$daysInMonth = date('t', $now);
 	$dayOfWeek = date('w', mktime(0,0,0, date('m', $now),date('d', $now), date('y', $now)));
 	$isHoliday = $dayOfWeek == 6 || $dayOfWeek == 0;
 	$rand = $isHoliday ? $random['holiday'] : $random['workweek'];
 	$rand = rand($rand[0],$rand[1]);
 	$daily = $total / $daysInMonth;
 	$part = round($total / $daysInMonth / 100 * $rand);
 	$modification = round($row['modification'] + ($daily + $part) / 24);

 	$sql = "UPDATE `betaintranet`.`ad_stat_cleaned` SET `betaintranet`.`ad_stat_cleaned`.`modification` = '{$modification}' WHERE 
 				 `betaintranet`.`ad_stat_cleaned`.`bid` = '{$row['bid']}' AND 
 				 `betaintranet`.`ad_stat_cleaned`.`date` = '{$row['date']}'";
 	mysql_query($sql) or die(mysql_error());
}

Answer 1

[Максим]

Вероятно у вас идет сравнение числа со строкой. Приводите принудительно сравниваемые переменные к одному типу:
(int)$row['w_site'] === 1
(int)$row['ban_type'] === 1
и далее также по аналогии.
Плюс у вас цикл while будет возвращать всегда одно и то же значение. Потому как сохранится присваивание $total из последней итерации

Comments

[Виктор Таран]

CREATE TABLE `ad_c` (
  `cid` int(10) NOT NULL,
  `ad_name` varchar(450) NOT NULL,
  `bid` int(10) NOT NULL,
  `ban_type` int(3) NOT NULL COMMENT '1-3 ТГБ, 4 - крстр, 5 - 940x90 середина, 6 - брендирование, 7 - 940x90 верхний, 8 - 940x90 - нижний',
  `obj_id` int(10) NOT NULL,
  `ra_id` int(10) NOT NULL,
  `dev_id` int(10) NOT NULL,
  `w_site` tinyint(1) NOT NULL COMMENT '1 - Nk, 2 - SPB, 3 - NP',
  `m_invoice` int(11) NOT NULL DEFAULT '0',
  `start_date` datetime NOT NULL,
  `end_date` datetime NOT NULL,
  `stopflag` tinyint(1) NOT NULL DEFAULT '0',
  `zaglusha` tinyint(1) NOT NULL DEFAULT '0' COMMENT 'является ли заглушкой',
  `phone` varchar(20) DEFAULT NULL COMMENT 'номер телефона для базы',
  `dyn_number` tinyint(1) NOT NULL,
  `whocreated` int(11) NOT NULL COMMENT 'ID создателя',
  `time_created` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP COMMENT 'Когда создал',
  `whoedited` int(11) DEFAULT NULL COMMENT 'ID редактора',
  `time_edited` timestamp NULL DEFAULT NULL COMMENT 'Когда редактировал'
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

Насколько я понимаю ban_type не принимал бы никаких значений поскольку условие не соблюдалось.

[Максим]

А, ну и еще момент. У вас цикл while будет возвращать всегда одно и то же значение. Потому как сохранится присваивание $total из последней итерации. (добавил в ответ выше)
Поэтому и не меняется значение) Но сравнение у вас всё равно некорректное идёт, я бы посоветовал вам это поправить.

[Виктор Таран]

Вот регуляции пожалуйста, баш пожалуйста, но пхп я даже не жуниор.
Можно на пальцах
И то что total не в whele я тоже понимаю и почему он принимает последнее значение тоже примерно понимаю, и да скорее всего в правы.
НО я не знаю как это написать ;)

[Максим]

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

[Виктор Таран]

логика простая
$daily = $total / $daysInMonth;
$part = round($totlal / $daysInMonth / 100 * $rand);
должны меняться от вида банера. сейчас у всех них одно значение, что там делает дальше это побоку там все работает.

[Максим]

Ну тогда вам нужно вот эту часть

while($row = mysql_fetch_assoc($w))
 {
 	$now = time();
 	$daysInMonth = date('t', $now);
 	$dayOfWeek = date('w', mktime(0,0,0, date('m', $now),date('d', $now), date('y', $now)));
 	$isHoliday = $dayOfWeek == 6 || $dayOfWeek == 0;
 	$rand = $isHoliday ? $random['holiday'] : $random['workweek'];
 	$rand = rand($rand[0],$rand[1]);
 	$daily = $total / $daysInMonth;
 	$part = round($total / $daysInMonth / 100 * $rand);
 	$modification = round($row['modification'] + ($daily + $part) / 24);

 	$sql = "UPDATE `betaintranet`.`ad_stat_cleaned` SET `betaintranet`.`ad_stat_cleaned`.`modification` = '{$modification}' WHERE 
 				 `betaintranet`.`ad_stat_cleaned`.`bid` = '{$row['bid']}' AND 
 				 `betaintranet`.`ad_stat_cleaned`.`date` = '{$row['date']}'";
 	mysql_query($sql) or die(mysql_error());
}

перенести в конец цикла while.

Но это такое себе решение. По-хорошему всё это дело нужно нормально отрефакторить.
А пока делайте бэкап базы данных перед тем как что-либо менять

Answer 2

[Adamos]

Он меняется - внутри первого цикла while.
К концу цикла он принимает значение в соответствии с последней строкой - и во втором while ему меняться не с чего.

P.S. Код, кстати, чудовищен. Колбаса if-ов заменяется одним компактным массивом и сравнением с ним. А в перспективе, при увеличении количества и/или произвольной динамике значений - выносом этих данных в базу.

Comments

[Виктор Таран]

Спасибо но я даже не разраб пхп, считайте что ваш сисадмин написал кривоватый код мало того мне на это убожество понадобилось около 2 дней ;(
Так что если есть возможность не мордой тыкать а показать как ;)

[Adamos]

Виктор Таран, если это временное решение - можно не полировать, работать-то оно будет.
Может, $total - это сумма и в if'ах должно быть не "=", а "+="?

[Виктор Таран]

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

[Adamos]

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

[Виктор Таран]

Adamos,
Да именно так и сделал, все заработало.
Всем спасибо