Форум PHP программистов > конструкция if

[ Поиск ] - [ Пользователи ] - [ Календарь ]

DMX

21.03.2009 - 10:52

Проблема с данным кодом.

PHP

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

   if ($do="edit") {

        $edit_id=intval($_GET['id']);
        include ("../news_config.php");
        $conn = mysql_query("SELECT * FROM `news` WHERE `post_id` = '$edit_id'")
    or die(mysql_error());
        $row = mysql_fetch_array($conn);
        $link="editpost.php?id=$edit_id";
   }

   else
     {

        $post_date=date("y-m-d h:i:s");
        $row[post_name]=null;
        $row[post_short]=null;
        $row[post_full]=null;
        $row[post_id]=null;
        $link="addpost.php";
   }

Хочу оформить добавление и редактирование постов в один файл. для этого и сделал данную конструкцию. Посредством GET я получаю действие которое необходимо сделать, но скрипт всегда возвращает

Код

$link="editpost.php?id=$edit_id"

может кто подскажет в чём проблема у меня?

Спустя 21 минута, 40 секунд (21.03.2009 - 10:14) twin написал(а):

if($do=="edit")

Спустя 8 минут, 25 секунд (21.03.2009 - 10:22) DMX написал(а):

Цитата (twin @ 21.03.2009 - 07:14)

if($do=="edit")

не помогает, хотя в соседнем скрипте нормально работает, правда содержимое if { .... } и else {...} оставляет include. Dозможно ли это из-за неправильного конфига сервера?

ps юзаю денвер

Спустя 2 часа, 6 минут, 37 секунд (21.03.2009 - 12:29) Kuliev написал(а):

Цитата (DMX @ 21.03.2009 - 07:22)

Цитата (twin @ 21.03.2009 - 07:14)

if($do=="edit")

PHP

<?
 $do = intval($_GET['do']); // Здесь ты объявляешь ее как число

// потом делаешь условие 

   if ($do == "edit"); // Пытаешься число прировнять к строке !!

Денвер тут не причем!

Спустя 15 минут, 39 секунд (21.03.2009 - 12:44) sergeiss написал(а):

Во-первых, после назначения переменной $do не помешает вывести ее значение, чтобы посмотреть, что же там на самом деле записано.
Во-вторых, фраза "if($do=="edit") на помогает" звучит как... Как полный маразм

В-третьих, с какого перепугу $do будет равно 'edit', если только что в эту переменную записано целое число?
В-четвертых, откуда в блоке else берётся массив $row[]? Из блока if он не может "перейти".

Короче говоря, думай, исправляй

PS. По некоторым пунктам уже ответили, пока писал ответ

Но исправлять не буду.

Спустя 13 минут, 11 секунд (21.03.2009 - 12:57) DMX написал(а):

Цитата (sergeiss @ 21.03.2009 - 09:44)

PS. По некоторым пунктам уже ответили, пока писал ответ

Но исправлять не буду.

1) посмотрю сейчас)
3)посредством GET я получаю значения для действия.
4) данная переменная подставляеца в значение textarea. Т.Е. если я добавляю новость то, то данное поле пустое, если же я редактирую, то данные подставляюца из переменной, полученной mysql запросом.

Kuliev,
а как тогда лучше сделать?

Спустя 7 минут, 33 секунды (21.03.2009 - 13:05) sergeiss написал(а):

А... Насчет $row[] понял

Но вот насчет установки целой величины в переменную и далее тут же сравнение ее со строкой - я не понял. Это явная логическая ошибка.

Спустя 9 минут, 20 секунд (21.03.2009 - 13:14) DMX написал(а):

Спасибо что на мысли навели

вывел значение, а оно 0 оказалось. почитал FAQ и понял что intval для чисел, а trim для текста

ну или както так.

Спустя 21 минута, 33 секунды (21.03.2009 - 13:36) Kuliev написал(а):

Цитата (DMX @ 21.03.2009 - 09:57)

3)посредством GET я получаю значения для действия.
4) данная переменная подставляеца в значение textarea. Т.Е. если я добавляю новость то, то данное поле пустое, если же я редактирую, то данные подставляюца из переменной, полученной mysql запросом.

--> Какое значение ты получаешь $_GET[]

Спустя 10 минут, 54 секунды (21.03.2009 - 13:47) Kuliev написал(а):

Пот тебе один вариант как сделать, можешь им воспользоваться, а можешь и нет!

Лови суть идеи!

PHP

<?
function links()
{

   echo " <a href='".$_SERVER['PHP_SELF']."'?show=new>Создать новость</a><p>
<a href='".$_SERVER['PHP_SELF']."'?show=update>редактировать новость</a>";

}

links();


function new_news ()
{


 // Здесь форма для заполнения и т.д.

}


function update ()
{

  // Здесь выводишь форму уже с теми значения которые надо отредактировать
//т.е. берешь из базы и подставляешь!

}

// выбор функции

switch ($_GET['show'])
{

case new     : new_news();  break;
case update : update();       break;

}

Спустя 19 минут, 18 секунд (21.03.2009 - 14:06) DMX написал(а):

Kuliev,
спс. принял к сведению. исправил ошибки в твоём листинге и пробираюсь дальше

Спустя 6 минут, 33 секунды (21.03.2009 - 14:13) Kuliev написал(а):

Цитата (DMX @ 21.03.2009 - 11:06)

Kuliev,
спс. принял к сведению. исправил ошибки в твоём листинге и пробираюсь дальше

торопился )))

Спустя 8 минут, 18 секунд (21.03.2009 - 14:21) DMX написал(а):

Kuliev,
а я думал это спецом для самостоятельной работы

щас пробую. переделать это, чтоб красиво было

Спустя 35 минут, 47 секунд (21.03.2009 - 14:57) Kuliev написал(а):

Цитата (DMX @ 21.03.2009 - 11:21)

Kuliev,
а я думал это спецом для самостоятельной работы

щас пробую. переделать это, чтоб красиво было

Ну ты хотел чтобы все в одном скрипти было! Вот тебе вариант!

Спустя 6 минут, 51 секунда (21.03.2009 - 15:04) sergeiss написал(а):

Цитата (DMX @ 21.03.2009 - 14:21)

а я думал это спецом для самостоятельной работы

Считай, что это была "лакмусовая бумажка"

: проверка на самостоятельность мышления.

Спустя 25 минут, 8 секунд (21.03.2009 - 15:29) DMX написал(а):

Цитата (Kuliev @ 21.03.2009 - 11:57)

Ну ты хотел чтобы все в одном скрипти было! Вот тебе вариант!

это уже продолжение моей мысли пошло.

и правда удобнее через функции писать))

Быстрый ответ:

Здесь расположена полная версия этой страницы.