Что то не так с переменными, в чем дело может быть ?

[horsethe]

Я немного помогаю на собственной инициативе в доработке игры. В общем задача была доработать файл помощи. что я собственно и сделала. Но пришлось немного изменить файл - добавить еще одну переменную. Итог: из трех переменных работает только 1. При ссылке на остальные переменные выводится пустая страница с одним только заглавием . Что из этого получилось можно посмотреть на http://rem-sot.ru/bot/faq.php (для теста заливала к себе на сайт)

http://www.188.ru/39749 тут скрипт сам .

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

[wwt]

выложите ваш php код пожалуйста вот сюда http://pastebin.com

[horsethe]

http://pastebin.com/H5NCm689 вроде все правильно сделала...

[Int]

http://pastebin.com/qGVfa35f

В рот мне ноги, и что из этого не работает?

[horsethe]

  On 9/7/2012 at 5:30 AM, Int said:

http://pastebin.com/qGVfa35f

В рот мне ноги, и что из этого не работает?

не отображается все что идет на переменных n и z а почему - понять не могу . вроде все правильно сделано.

[rus]

удалил

[horsethe]

elseif исправила . то же самое .

нет, не то же самое . с таким elseif не работает даже то что работало.

Edited September 7, 2012 by horsethe

[Veseloff]

Мне кажется, что надо переделать вообще всё. Это же ужас какой-то, а не код.

[horsethe]

  On 9/7/2012 at 6:37 AM, Veseloff said:

Мне кажется, что надо переделать вообще всё. Это же ужас какой-то, а не код.

может быть, но я опиралась на уже существующий код, потому и код такой . если подскажете как переделать - буду премного благодарна.

[wwt]

Жесть... руки бы оторвать тому кто это изначально писал =))))

1) Организовать хранилище данных, не важно что это будет, хотя бы простейшим массивом в отдельном файле как нибудь так:

<?
$content["m"]["zdaniya"] = "текст";
$content["m"]["zdaniya2"] = "текст";

$content["n"]["zdaniya"] = "текст";
$content["n"]["zdaniya2"] = "текст";

$content["z"]["zdaniya"] = "текст";
$content["z"]["zdaniya2"] = "текст";
//и т.д.
?>

2) В вашем файле подключаем это хранилище:

<?

if (isset($_REQUEST['m'])) $m = $_REQUEST['m'];
if (isset($_REQUEST['n'])) $n = $_REQUEST['n'];
if (isset($_REQUEST['z'])) $z = $_REQUEST['z'];

require_once("content.php");//подключаем файл с массивом.

if(!isset($m)){
   echo $content["m"][$m];//так выводим контент элемента массива
}
//далее вся ваша логическая структура
?>

Так вы хотя бы сможете править логическую структуру if...elseif...else , а с такой мешаниной как сейчас, там сам черт ногу сломит.

3) Некоторые условия вооще убрать из кода, ибо они станут ненужны к примеру вот такие моменты:

if(isset($z)){
   if($z==1){
      echo $content["z"][$z];
   }elseif($z==2){
      echo $content["z"][$z];
   }elseif($z==3){
      echo $content["z"][$z];
   }
}

заменяются просто обращением к элементу массива по ключу, сравнение будет уже ненужно если переменная $z установлена и элеиент с нужным ключем существует в массиве:

if(isset($z)){
   echo $content["z"][$z];
}

ЗЫ Конечно есть способы намного лучше этого, ну это будет хотя бы что-то, по сравнению с тем что есть сейчас )))

Edited September 7, 2012 by wwt

[hedgehog]

wwt,

мой первый опыт с PHP примерно так и выглядел ) Только контента было много, поэтому я импортировал его из инклудов. Т.е. у меня были несколько каталогов с контентом для основного блока (каталоги разбиты по категориям), плюс шапка/пдвал/сайдбар/блоки, которые подключались для всех страниц.

[wwt]

  On 9/7/2012 at 8:07 AM, hedgehog said:

wwt,

мой первый опыт с PHP примерно так и выглядел ) Только контента было много, поэтому я импортировал его из инклудов. Т.е. у меня были несколько каталогов с контентом для основного блока (каталоги разбиты по категориям), плюс шапка/пдвал/сайдбар/блоки, которые подключались для всех страниц.

ну как вариант можно хранилище не массивом, а файлами, и инклюдить. Тагда не будет геморроя с кавычками у контента в массиве.

[Int]

  On 9/7/2012 at 7:51 AM, wwt said:

if(!isset($m)){
   echo $content["m"][$m];//так выводим контент элемента массива
}
//далее вся ваша логическая структура
?>

Чего-то ты не то написал-то

А я мельком тот код просмотрел и вообще не увидел, где переменные передаются. По ссылкам потыкал, все открылись, текст есть.

М, так надо было в здания заходить.

А может быть, чтобы сервер не дёргать без дела, сделать эти разделы разворачивающимися через яваскрипт?

if (!isset($m))
{
	print ("");
}
elseif ($m=="zdaniya")
{
	if(!isset($n))
	{
		print("");
	}
}		//Вроде бы вот это вот косяк, эта скобка должна быть далеко внизу
elseif ($n=="ratusha")
{
	if(!isset($z))
	{
        print("");
	}
    elseif($z=='spy')
    {
        print("");
	}
	elseif ($z=='grabber')
	{
	}
	elseif ($z=…
}

Второй elseif не после той скобки воткнут, после какой надо. Я полагаю.

Может быть, переписать на switch-case? А то мозг сломать можно. Да и половина скобок лишняя. Хотя сейчас начнут кричать, что это эстетика

switch ($m)
{
case 'zdaniya': 
	switch($n)
	{
		case 'ratusha': require('ratusha.txt'); break;

	}
	break;
}

А вложенности внутри $n делать уже яваскриптом, например.

Ну и вместо

if (!isset($m))

проще делать так:

if (!isset($m)) $m = 'defalut';

switch($m)
{
	…
	…
	…
	default: require('default_m.txt');
}

Edited September 17, 2012 by Int

[horsethe]

ИЗвините, добавлю что есть такой критерий что бы ВСЕ было в одном файле... то есть разделять на несколько файлов нельзя...

Edited September 7, 2012 by horsethe

[wwt]

  On 9/7/2012 at 5:10 PM, Int said:

  On 9/7/2012 at 7:51 AM, wwt said:

if(!isset($m)){
   echo $content["m"][$m];//так выводим контент элемента массива
}
//далее вся ваша логическая структура
?>

Чего-то ты не то написал-то

та это просто пример был )) вообще занятно вышло согласен.

[horsethe]

Всем спасибо. С if'ами косяк был . нашла исправила . все работает. ну почти все, только res_print обособлять двойными кавычками - появляется ошибка синтаксиса . Ну там думаю можно просто текстом заменить.. хотя и халтурка получится наверное ))