Как внести изменения записи в БД, на SQL PHP?

Question

[Александр Иванов]

Создаю код внесения изменений в БД, обновление не происходит.
Скорее всего кусок $query_array["news_content"] не верно записан. Видимо по новой его объявить, но не знаю как.

<?php 
$host = "localhost";
$dbname = "dogs";
$dbuser = "root";
$dbpass = "";
$dbtable = "news";
$dbtable_id = "7"; 
$db = mysql_connect($host,$dbuser,$dbpass) or die (mysql_error());
mysql_select_db($dbname,$db);
$query = mysql_query("SELECT * FROM ".$dbtable." WHERE news_id="
.$dbtable_id."");
$query_array = mysql_fetch_array($query);
if(isset($_POST['cur'])){
echo "запрос отпавлен<br />";
$updQuery = mysql_query("UPDATE ".$dbname.".".$dbtable
." SET `news_content` = ".$query_array["news_content"]
." WHERE ".$dbtable.".news_id =".$dbtable_id.";");
}
?>
<form method="POST">
<textarea><?php echo $query_array["news_content"] ?></textarea><br />
<input type="submit" name="cur"/>
</form>

Answer 1

[andreyqin]

Замените свой кусок кода на мой:

$query = mysql_query("SELECT * FROM ".$dbtable." WHERE news_id="
.$dbtable_id);
$query_array = mysql_fetch_array($query);
if(isset($_POST['cur'])){
echo "запрос отпавлен<br />";
$updQuery = mysql_query('UPDATE '.$dbtable
.' SET `news_content` = "'.$query_array["news_content"]
.'" WHERE news_id ='.$dbtable_id);
}

Comments

[Александр Иванов]

Никаких изменений...

[Александр Иванов]

Я и учусь, на него. Твой комент неуместен.

[andreyqin]

@cimonlebedev, а, ну у вас $dbtable_id записано как строка, а в базе, наверное, это число. Либо в $query_array["news_content"] ничего не приходит.

[FanatPHP]

@andreyqin $dbtable_id записано как строка, а в базе, наверное, это число - ну и что?

[Александр Иванов]

@andreyqin я заменил $dbtable_id на 7 но без изменений.
Может надо где-то unset() применить?

[andreyqin]

@FanatPHP, извиняюсь, запрос выполнится и со строкой.
@cimonlebedev, зачем? Вы проверьте, что у вас в переменной $query_array["news_content"] перед запросом.

[Александр Иванов]

@andreyqin @FanatPHP Я внёс предложение от статьи www.phpsql.ru/writing-adminky

[andreyqin]

@cimonlebedev, не знаю что вы там внесли, но это плохой код. Погуглите на тему "php best practices" и первое, что вы должны прочитать - getjump.github.io/ru-php-the-right-way

[Александр Иванов]

@andreyqin спасибо статья полезная

Answer 2

[FanatPHP]

Покажи человеку, который писал этот код, вот эту ссылку: phpfaq.ru/debug
Пусть прочтет и попытается понять.

Comments

[Александр Иванов]

я его и писал

[FanatPHP]

@cimonlebedev Судя по этому ответу, "читать и понимать" в процесс обучения на программиста не входит? В таком случае вынужден констатировать, что затея обречена на провал. Если взять говнокод, написанный идиотом, который не понимает что делает, и попытаться скопировать его, так же без малейшего понимания - то никакого программиста не получится. Я гарантирую это.

Answer 3

[Александр Иванов]

В этой статье www.phpsql.ru/writing-adminky корректировка или update выглядит так:

//Корректировка
if(isset($_POST[`edit`])){
  $query="UPDATE blog SET tema=`$_POST[tema]`,text=`$_POST[text]`  WHERE id=`$_POST[id]`";
  $result = mysql_query($query);
  unset($row);  
}

Поэтому я думаю думаю, unset не хватает.

Comments

[FanatPHP]

офигеть логика. просто на грани фантастики. Слушай, тебе рано еще обновление писать. Тебе надо сначала понять, что делает твой код. Чтобы хотя бы примерно отличать, чем отличается unset от mysql_query. А то так гадать можно долго.

[Александр Иванов]

Спасибо за комментарии, но ни одного стоящего предложения не было. Тема закрыта.

[FanatPHP]

Стоящий комментарий можно дать только к стоящему вопросу. Вопрос вида "я взял чудовищный говнокод, написанный каким-то идиотом, и что-то в нем переделал без малейшего понимания. Почему у меня не работает?" стоящим назвать никак нельзя. подумай об этом на досуге.

[FanatPHP]

В порядке благотворительности. "Стоящий" комментарий: вызов unset что в исходном коде, что в твоём, не несёт абсолютно никакого смысла.

[FanatPHP]

Да, кстати, ещё один стоящий комментарий был. Надо просто уметь извлекать информацию. Если надо, могу повторить отдельно: сайт phpsql.ru написан идиотом. Не пей из этой лужи Иванушка - таким же козленочком стагешь.

[Александр Иванов]

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

[FanatPHP]

Это уже лучше. В любом случае, твоя проблема в том, что ты очень пассивен. Ты ждёшь, что к тебе прилетит волшебник, и на блюдечке принесет ответ. Увы, но после окончания детского сада в жизни любого человека происходит перелом, после которого такое развитие событий становится маловероятным. Дальше решать свои проблемы приходится самому. И даже при наличии сторонней помощи не получится просто сидеть сложа ручки и ждать, чтобы чужой дядя все за тебя сделал. Надо и самому стараться. Надо читать, что тебе пишут, и пробовать. Я понимаю, что andreyqin выкатил кучу идиотских догадок, большая часть которых не имеет смысла. Но всё равно, рабочий код нужен тебе, а не ему. И проверять его догадки тоже тебе. Тем более что среди них была одна осмысленная: "проверьте, что у вас в переменной $query_array["news_content"]" - очень здравая мысль. А заодно и вообще подумать, что это за переменная и с какой стати в ней вообще что-то должно быть. А потом подумать, где на самом деле надо искать данные из формы. А потом посмотреть на форму и подумать, чего в ней не хватает. Все эти действия описаны по ссылке, которую я дал.

[Александр Иванов]

@FanatPHP основная проблема была в отсутствии кавычек:
'".$query_array["news_content"]."'

[andreyqin]

@cimonlebedev, я же вам поставил эти кавычки в своем ответе

[Александр Иванов]

@andreyqin наверное я скопировал криво... Зато узнал много нового)
Спасибо, отмечу как решение.