Форум Сайтостроение Веб-строительство

Не работает код на PHP Version 5.3.13

123
LEOnidUKG
На сайте с 25.11.2006
Offline
1762
LEOnidUKG
#11

перед:

if ($r)

напишите:

$r=intval($_GET['r']);

✅ Мой Телеграм канал по SEO, оптимизации сайтов и серверов: https://t.me/leonidukgLIVE ✅ Качественное и рабочее размещение SEO статей СНГ и Бурж: https://getmanylinks.ru/ ✅ Настройка и оптимизация серверов https://getmanyspeed.ru/
AL
На сайте с 02.12.2010
Offline
26
Al.lexx
#12
LEOnidUKG:
перед:

if ($r)

напишите:

$r=intval($_GET['r']);

Спасибо! Помогло!!! Ура! :)

B
На сайте с 14.04.2012
Offline
35
bimbo
#13

ой,ошибся тут

<?php
if ($r==21)
{
include ("razdels/2/menu.php");
}
elseif ($r==22)
{
include ("razdels/2/2.php");
}
elseif ($r==23)
{
include ("razdels/2/3.php");
}
elseif ($r==24)
{
include ("razdels/2/4.php");
}
elseif ($r==25)
{
include ("razdels/2/5.php");
}
elseif ($r==26)
{
include ("razdels/2/6.php");
}
elseif ($r==27)
{
include ("razdels/2/7.php");
}
elseif ($r==31)
{
include ("razdels/3/menu.php");
}
elseif ($r==32)
{
include ("razdels/3/2.php");
}
elseif ($r==33)
{
include ("razdels/3/3.php");
}
elseif ($r==34)
{
include ("razdels/3/4.php");
}
elseif ($r==35)
{
include ("razdels/3/5.php");
}
elseif ($r==36)
{
include ("razdels/3/6.php");
}
elseif ($r==37)
{
include ("razdels/3/7.php");
}
elseif ($r==38)
{
include ("razdels/3/8.php");
}
elseif ($r==39)
{
include ("razdels/3/9.php");
}
elseif ($r==310)
{
include ("razdels/3/10.php");
}
elseif ($r==41)
{
include ("razdels/4/menu.php");
}
elseif ($r==42)
{
include ("razdels/4/2.php");
}
elseif ($r==43)
{
include ("razdels/4/3.php");
}
elseif ($r==44)
{
include ("razdels/4/4.php");
}
elseif ($r==45)
{
include ("razdels/4/5.php");
}
elseif ($r==46)
{
include ("razdels/4/6.php");
}
elseif ($r==47)
{
include ("razdels/4/7.php");
}
elseif ($r==48)
{
include ("razdels/4/8.php");
}
elseif ($r==49)
{
include ("razdels/4/9.php");
}
elseif ($r==410)
{
include ("razdels/4/10.php");
}
elseif ($r==411)
{
include ("razdels/4/11.php");
}


else
{
include ("razdels/1/1.php");
}


?>
K
На сайте с 30.07.2012
Offline
0
knowill
#14

юзайте swithc. за такой код растреливать без суда и следствия...

LEOnidUKG
На сайте с 25.11.2006
Offline
1762
LEOnidUKG
#15
knowill:
юзайте swithc. за такой код растреливать без суда и следствия...

Почему расстреливать?! Никогда не понимал таких людей. 🍿

Можно ли сделать проще, чем ТС? Можно.

Будет ли это в современном мире быстрее и удобней для конечно пользователя? Нет.

K
На сайте с 30.07.2012
Offline
0
knowill
#16
LEOnidUKG:
Почему расстреливать?! Никогда не понимал таких людей. 🍿

Можно ли сделать проще, чем ТС? Можно.
Будет ли это в современном мире быстрее и удобней для конечно пользователя? Нет.

1, в данном случае правильно использовать switch (выигрыш в производительности). с таким объемом, похоже очень большой

2. имена переменных "$r", уже растрел

3. цифры, думаю тоже нет смысла объяснять

4. путь razdels/2/4.php

5. include должен быть или require или require_once

6. кавычки двойные

7. весь код, сплошной баг

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

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

Как увеличить трафик с Сервис Rookee продлевает акцию Особенности SEO для сайтов-агрегаторов
LEOnidUKG
На сайте с 25.11.2006
Offline
1762
LEOnidUKG
#17
knowill:
1, в данном случае правильно использовать switch (выигрыш в производительности). с таким объемом, похоже очень большой
2. имена переменных "$r", уже растрел
3. цифры, думаю тоже нет смысла объяснять
4. путь razdels/2/4.php
5. include должен быть или require или require_once
6. кавычки двойные
7. весь код, сплошной баг
8, автор хотябы прокоментировать бы потрудился, сам же забудет через пару месяцев что он тут написал
будет как минимум в плане поддержки и развития для пользователей, и дешевле для владельца

Да же спорить не буду. Набор бреда.

Coolmac
На сайте с 08.09.2011
Offline
60
Coolmac
#18
1, в данном случае правильно использовать switch (выигрыш в производительности). с таким объемом, похоже очень большой
2. имена переменных "$r", уже растрел
3. цифры, думаю тоже нет смысла объяснять
4. путь razdels/2/4.php
5. include должен быть или require или require_once
6. кавычки двойные
7. весь код, сплошной баг
8, автор хотябы прокоментировать бы потрудился, сам же забудет через пару месяцев что он тут написал
будет как минимум в плане поддержки и развития для пользователей, и дешевле для владельца

Вам сюда, думайте перед тем как писать.

Доступные vps/ds с грамотной тех. поддержкой (https://my.friendhosting.net/pl.php?11304)
IL
На сайте с 20.04.2007
Offline
435
ivan-lev
#19
knowill:
юзайте swithc. за такой код растреливать без суда и следствия...

Имхо, код не лишён недостатков.. но switch в данном случае не намного лучше.

Соглашусь с LEOnidUKG

LEOnidUKG:
офигеть и не поленились же сколько кода писать :)

Как-то так бы сделал.. Потом просто файлики закидывать в razdels:

$r = $_GET['r'];
// вставить нормальную обработку 404 ошибки.. 
// или обращение к другому файлу..  в общем, как задумано
if ($r!=(string)(int)$r) 
    die('404 ошибка');
$razdel = $r[0];  
$file = substr($r,1);
// если $razdel и $file не требуются дальше, их можно не присваивать, 
//а сразу находить $filename 
echo $filename = "razdels/$razdel/$file.php"; 
if (is_file ($filename))
    include $filename;
LEOnidUKG:
Можно ли сделать проще, чем ТС? Можно.
Будет ли это в современном мире быстрее и удобней для конечно пользователя? Нет.

По-моему, это "быстрее и удобнее" для конечного программиста, который (если) будет поддерживать код. С учётом того, что для "универсализации" времени потребуется не так много.

По поводу

$r=intval($_GET['r']);

Тут есть потенциальная "уязвимость" с точки зрения SEO.

?r=1 и ?r=1any - две одинаковых страницы (=дубли) с разным URL-ом. Не факт, что они появятся (хотя, у более-менее успешного сайта "доброжелатели" находятся довольно часто) Скорее всего, со временем поисковик разберётся и выкинет из индекса всё кроме одной, но не факт, что оставит нужную.

... :) Облачные серверы от RegRu - промокод 3F85-3D10-806D-7224 ( http://levik.info/regru )
Страница выпала из индекса. Новый URL при выборе Переезд страницы в рамках
E
На сайте с 21.09.2009
Offline
283
easywind
#20

ivan-lev, у меня на сайте и вот так появлялось

?r=

так и не понял откуда. Пришлось дописать рефреш на 404 страницу.

123

Авторизуйтесь или зарегистрируйтесь, чтобы оставить комментарий