Полезные мелочи для продвижения

Развивайте тему, думаю многим будет полезно

Для кого? Если для других, то с таким кодом как в первом скрипте - не нужно!

count внутри цикла... Да еще и с чужого сервера.

Кроме того для этой задачи лучшее решение это .htaccess

Что не так?

Конкретно для этой задачи не лучшее. Написал же, использую для десятка сайтов сразу. Править в 10 .htaccess'ах или в одном перечне? (txt-файле)

Так положите на своём :)

count($bad) можно выполнить всего лишь 1 раз

.htaccess выполнит эту задачу намного быстрей.

А если сервер с файлом упадёт?

От парсеров данная конструкция не спасет, но может создать проблемы нормальным ботам и возникнет проблема с индексацией страниц 😕

Ок.

Если же вы подразумеваете, что надо до запуска цикла переменной присвоить количество массива $bad, а потом уже эту переменную использовать в цикле, то аргументируйте, плиз, для моего просвещения.

Еще раз пишу, для 10 сайтов надо вносить правки в 10 файлов htaccess, а при данном подходе лишь в одном файле.

У меня VPS и все сайты лежат на одном сервере.

P.S.: с моего сервера не надо использовать, размещайте на своём.

---------- Добавлено 26.11.2018 в 19:13 ----------

Не, цель именно нагрузку снизить. Ahrefs зашел и погнал тысячами новый сайт листить, а у меня там ~ 600k страниц, в итоге на других сайтах пошли тормоза. Или еще один, не помню какой конкретно, что-то из ссылочного, пошел сканировать с интервалом не более секунды, отсюда и текст до exit'a.

UPD: неправильно понял вашу реплику. Значит как это работает: рефереры логируются, потом я их просматриваю. Когда идут подряд запросы от одного типа бота, я ищу о нем информацию и решаю, блокировать ли. Нормальным ботам это создаст помехи, если внести их в black list.

Думаю, что речь о том, что сейчас количество слов вычисляется на каждой итерации.

Можно сделать это перед началом цикла один раз. Небольшая оптимизация. :)

$bad = file_get_contents('http://joysea.ru/bad.txt');
$bad = explode(',' ,$bad);
$bad_count = count($bad);

for ($i=0; $i<$bad_count; $i++) {
	if (substr_count($_SERVER['HTTP_USER_AGENT'],$bad[$i]) > 0) {
		echo 'Please leave my server forever and fuck yourself.';
		exit();
	}
}

Не используйте for там, где это не нужно. Есть foreach.

Тогда, наверное, можно и без foreach решить.

$bad = file_get_contents('http://joysea.ru/bad.txt');

$bad = explode(',' ,$bad);

if (in_array($_SERVER['HTTP_USER_AGENT'], $bad)) {

	exit('Please leave my server forever and fuck yourself.');

}

Не надо там ничего считать. Достаточно substr().