Улучшение кода

[premiums]

Улучшение кода

Скажите отцы PHP (и не только)!
Как коректней написать такой код.

<?
/*Если $str* = on то $p1 =1 И текст не появляется */
if ($str.$str1.$str2.$str3.$str4.$str5 == True) {
$p1 = 1;
}

if ( !isset($p1) )
echo "Текст";

?>

 

[Фанат]

if (!empty($str)) echo "Текст";

$str - массив.
только я не понял фразы "Если $str* = on " имеется в виду "все он" или "любая он"?

 

[premiums]

Автор оригинала: Фанат
if (!empty($str)) echo "Текст";

$str - массив.
только я не понял фразы "Если $str* = on " имеется в виду "все он" или "любая он"?

К примеру ссылка
http://site.com/?mod_page?name=Dilers&dil=4str3=on

если равто он то в говорит True примерно так))
Тут каждая $str* это для вывода на определённой странице куска нужного кода

 

[Фанат]

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

прежде, чем что-то оптимизировать, надо понять смысл

 

[premiums]

Автор оригинала: Фанат
а теперь поясни смысл этой стр3.
смысл первых двух переменных очевиден и ясен.
а эта стр как-то ник селу ни к городу.

прежде, чем что-то оптимизировать, надо понять смысл

Когда в ссылке не задействована $str3 то в страницу вставляется кусок кода (боковое меню) из if ( !isset($p1) )
echo "Текст";

 

[Фанат]

зачем тебе переменная $p1?

 

[premiums]

Поясню в коде.

<?
/*Если $str*в ссылке = on то $p1 =1 И текст не появляется */
if ($str.$str1.$str2.$str3.$str4.$str5 == True) {

$p1 = 1; //Если она ровна 1 то меню на данной стр. нет.

}

if ( !isset($p1) )
echo "Текст";

?>

 

[Фанат]

повторю вопрос.
зачем тебе переменная $p1?

 

[premiums]

если $p1 не равняется 1 то меню будет появлятся на всех страницах. А мне надо на определенных стр. меню убрать.

 

[Фанат]

но ведь это появление зависит не от $p1, а от $str1

 

[sayber]

Если в ссылке содержется переменная $str которая равна on то
if ($str# == True) то есть октевирована. Если октевирована то PHP смотрит что содержется в блоке { } и актевирует $p1=1

if ( !isset($p1) ) Этот иф смотрит активирована ли переменная $p1 если да то выполняется echo "";

 

[Фанат]

спасибо.
а теперь, с вашего позволения, я подожду ответа от premiums

 

[premiums]

Фанат Sayber это мой домашний аккаунт) У меня дома 3 компа и на работе один

 

[Фанат]

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

 

[premiums]

Нет просто тот что premiums это рабочий и его зарегестрировали без меня. Я просто зашел на форум и смотрю что уже активирован а когда дал 1 вопрос понял что не мой аккаунт.
Но теперь похоже мой.

Извеняюсь за такую непонятку и захломление БД)

 

[Фанат]

ну так зайди под своим
и общайся под одним каким-нибудь

 

[premiums]

ОК, но давай перервем нашу беседу и обратим внимание на саму суть вопроса.

 

[Фанат]

я думаю, это бессмысленно.

 

[premiums]

) Почему? Оно же работает
Или как лучше написать данное действие?

Я изменил значение с True na False

 

[Фанат]

зачем?
работает - и пускай работает