[FINAL] Birthday Zodiac Optimization

[Xpert]

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

Ставить поверх оригинального аддона.

Drupal refresh

[Siava]

Файл table_update.php запускал кучу раз, но так и не дождался окончания обновления. У некоторых посетителей знак зодиака так и не выставился. День и месяц рождения например, 29.12. , 31.12. , 26.12. , и т.д.. примерно конец декабря.

[Xpert]

Там по 50 пользователей за раз. Если при запуске не появляется сообщение о том, что все пользователи обработаны, значит нужно запустить еще...

Добавлено спустя 2 минуты 7 секунд:

Если нужно запускать еще, то видим

Done, start script one more time to process next portion of users

Если все обработаны, то

Done, all users were proceed

[Siava]

Xpert
Я запускал, обновлял и т.д. раз 50. у меня остаётся около 20 пользователей, у которых никак не добавляется зодиак. Всего пользователей более 700.

Добавлено спустя 1 минуту 58 секунд:

Только что выполнил ровно 50 раз, но всё-равно "Done, start script one more time to process next portion of users"

[Xpert]

А что у них в полях user_birthday и user_zodiac ?

[Siava]

Xpert
в user_zodiac у них 0, а в user_birthday 7664, 6570, 4742, 6204, 5838, ... (первых 5 перечислил).

Добавлено спустя 11 часов 37 минут 11 секунд:

И ещё сегодня нашёл.
В БД:
user_birthday 6319
user_zodiac 4

День Рождения: 21.04.1987
Зодиак: Овен

Но ведь это должен быть телец

Добавлено спустя 3 минуты 30 секунд:

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

[Xpert]

Ошибся я немного в коде подсчета зодиака видимо...
Меняем ф-ю get_zodiac на такую

Код: Выделить всё

function get_zodiac($date)
{
	global $zodiacdates;

	$zodiac = 0;

	if ( $date == 999999 ) 
	{
		return $zodiac;
	}

	$date = realdate('md', $date);	

	for ($i = 0; $i < 26; $i+=2)
	{
		if ($date >= $zodiacdates[$i] && $date <= $zodiacdates[$i+1])
		{
			$zodiac = ($i/2) + 1;
			break;
		}
	}

	if ($zodiac == 13)
	{
		$zodiac = 1;
	}

	return $zodiac;
}

Потом делаем

Код: Выделить всё

UPDATE phpbb_users SET user_zodiac = 0 WHERE 1

... и проверяем все с самого начала.

Siava
Проверь у себя все пожалуйста, включая тот глюк с Овеном...
Действительно серверу полегче стало?

[Siava]

Done, all users were proceed

Теперь всё обновилось успешно, глюк с овном пропал, исправился на тельца. Об ошибках буду сообщать.

Нагрузку мод действительно снял неплохую. Определил на глаз, когда обычно посетителей на форуме 50-60, то Load Average в среднем доходит до 1.5-2, а с оптимизацией держится на уровне 1-1.5.

[Xpert]

Спасибо... обновил версию до 1.0.1, изменения - только эта функция. Чего бы еще такого прооптимизировать?

[Гога]

Код: Выделить всё

#
#-----[ FIND ]----------------------------------------
#
$zodiacs = array (

#
#-----[ IN-LINE FIND ]----------------------------------------
#
'Capricorn'

#
#-----[ IN-LINE BEFORE, ADD ]----------------------------------------
#
1 =>

У меня два элемента 'Capricorn' (первый и последний). Какой трогать?
Должно ли быть так:

Код: Выделить всё

$zodiacs = array (1 => 'Capricorn', 'Aquarius', 'Pisces', 'Aries', 'Taurus', 'Gemini', 'Cancer', 'Leo', 'Virgo', 'Libra', 'Scorpio', 'Sagittarius', 'Capricorn');

[Xpert]

Трогать первый, Гога, ну не тебе такие вопросы задавать.
Решил все же вести нумерацию с 1... это проще, чем делать значение по умолчанию NULL или -1.

[Поручик]

Xpert, а вы не против, если я вашу идею прикручу к мому аналогичному адд-ону

[lusever]

Сделайте плз полную версию, чтоб поверх не ставить.

[Xpert]

Нет, на здоровье.

[Xpert]

Полную версию делать не буду, нет желания. Оно и поверх неплохо ставится.