Не обновляет выбранное поле в бд по id

@Kameramann · Регистрация: 06.12.2016

Студворк — интернет-сервис помощи студентам

Почему то не работает
Пытаюсь обновить выбранное поле в бд через форму

HTML5
1
2
3
4
5
6
7
8
9
10
11
<form  method='POST' action="../connect.php">
           <select  type="select" name="status">
           <option value="" selected="selected" disabled="">Выберите значение</option>
        <option VALUE="В обработке">В обработке</option>
        <option VALUE="Завершено">Завершено</option>
     </select>
     </div>
          <div class="modal-footer">
          <div class="btn-animate">
            <input  class="btn-signin" type="submit" name="submit" value="ДОБАВИТЬ" >
          </div></form>

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 <?php
 $dbcon = mysql_connect("localhost", "root", ""); 
    mysql_select_db("users", $dbcon);
  if (!$dbcon)
  {
    echo "<p>Произошла ошибка при подсоединении к MySQL!</p>".mysql_error(); exit();
    } else {
    if (!mysql_select_db("users", $dbcon))
    {
    echo("<p>Выбранной базы данных не существует! коннект</p>");
    }
  }
  if(isset($_POST['submit'])) 
  if (isset($_POST['status'])) { $status = $_POST['status']; if ($status == '') { unset($status);} } 
   if (isset($_POST['id'])) { $id = $_POST['id']; if ($id == '') { unset($id);} } 
  $query = mysql_query("UPDATE tecnikal_zakaz
   SET status = '".$status."',
   
   WHERE id='$id'");
        if ($query) {
                echo "<div align=center>Данные успешно обновлены!</div><br>";
                } else {
                echo "<div align=center>Невозможно сохранение данных в БД!</div><br>";
                }
        mysql_close($dbcon);
   ?>

@gbsoftware · 29.08.2018, 09:40

Ошибка в запросе, запятую перед WHERE уберите:

PHP
1
$query = mysql_query("UPDATE tecnikal_zakaz SET status = '$status' WHERE id=$id");

@Para bellum · 29.08.2018, 09:46

Не по теме:

Kameramann, и переходите, пока не поздно, на PDO или MySQLi. Расширение mysql было актуально лет семь назад.
Пять лет назад было признано устаревшим.

@Kameramann · 29.08.2018, 09:48 **[ТС]**

а как брать id текущей записи?
Обновляет но только если явно указать что надо обновить 78 запись.
А надо чтобы он сам обновлял открытую запись

Добавлено через 52 секунды
на 7 версию я перешел бы, если кодов и примеров было бы достаточно

@gbsoftware · 29.08.2018, 10:01

Что значит "открытую запись"?
Примеров по PDO или MySQLi более чем предостаточно, как собственно и по 7 версии PHP к каковой они не имеют отношения.

@Kameramann · 29.08.2018, 10:03 **[ТС]**

Он должен брать сам id записи и обновить ее, а не явно указывать номер записи

@gbsoftware · 29.08.2018, 10:06

откуда он должен брать это id? вы форму на основании базы строите? записи в базе откуда берутся?

@Kameramann · 29.08.2018, 10:11 **[ТС]**

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

@gbsoftware · 29.08.2018, 10:26

ну вот при выводе записи из бд пишите ее в скрытое поле формы и отправляйте вместе с формой, например как-то так:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?php
$query = mysqli_query($dbcon, "SELECT * FROM tecnikal_zakaz");
while ($record = mysqli_fetch_assoc($query)):
?>
<form  method='POST' action="../connect.php">
    <label>Заявка № <?php echo $record['id']; ?></label>
    <select  type="select" name="status">
        <option value="" selected="selected" disabled="">Выберите значение</option>
        <option VALUE="В обработке">В обработке</option>
        <option VALUE="Завершено">Завершено</option>
    </select>
    <input type="hidden" value="<?php echo $record['id']; ?>" name="id">
    </div>
    <div class="modal-footer">
        <div class="btn-animate">
            <input  class="btn-signin" type="submit" name="submit" value="ДОБАВИТЬ" >
        </div>
</form>
<?php
endwhile;
?>

а потом уже обрабатываете форму, как-то например так:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<?php
$dbcon = mysqli_connect("localhost", "root", "", "users");
if (!$dbcon)
{
    echo "<p>Произошла ошибка при подсоединении к MySQL!</p>" . mysqli_error($dbcon);
    exit();
}
 
if(isset($_POST['submit'])){
    $status = $_POST['status'];
    $id = $_POST['id'];
}
if($status != '' && $id != ''){
    $query = mysqli_query($dbcon, "UPDATE tecnikal_zakaz SET status = '$status' WHERE id = $id");
}else{
    $query = false;
}
if ($query) {
    echo "<div align=center>Данные успешно обновлены!</div><br>";
} else {
    echo "<div align=center>Невозможно сохранение данных в БД!</div><br>";
}
mysqli_close($dbcon);
?>

@Kameramann · 29.08.2018, 10:46 **[ТС]**

эт на седьмой версии написали

@gbsoftware · 29.08.2018, 10:49

Если вы имеете ввиду версию PHP, то нет

@Para bellum · 29.08.2018, 10:53

Сообщение от Kameramann

эт на седьмой версии написали

mysqli не только в 7 версии.

@Kameramann · 29.08.2018, 10:58 **[ТС]**

Warning: mysqli_query() expects parameter 1 to be mysqli, resource given in on line 317
Warning: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, null given in on line 318

@gbsoftware · 29.08.2018, 11:01

ну я же для примера код дал, я так понимаю это при выводе формы, вы базу предварительно подключили?

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
$dbcon = mysqli_connect("localhost", "root", "", "users");
$query = mysqli_query($dbcon, "SELECT * FROM tecnikal_zakaz");
while ($record = mysqli_fetch_assoc($query)):
?>
<form  method='POST' action="../connect.php">
    <label>Заявка № <?php echo $record['id']; ?></label>
    <select  type="select" name="status">
        <option value="" selected="selected" disabled="">Выберите значение</option>
        <option VALUE="В обработке">В обработке</option>
        <option VALUE="Завершено">Завершено</option>
    </select>
    <input type="hidden" value="<?php echo $record['id']; ?>" name="id">
    </div>
    <div class="modal-footer">
        <div class="btn-animate">
            <input  class="btn-signin" type="submit" name="submit" value="ДОБАВИТЬ" >
        </div>
</form>
<?php
endwhile;
?>

@Kameramann · 29.08.2018, 11:22 **[ТС]**

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

@gbsoftware · 29.08.2018, 11:24

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

@Kameramann · 29.08.2018, 11:40 **[ТС]**

так как вывести текущую запись? а не все записи

Добавлено через 1 минуту
нужно выбрать нужную запись и редактировать там определенное значение, а не все записи выводить

@gbsoftware · 29.08.2018, 11:41

объясните что вы имеете ввиду под текущей записью? выбрать нужную запись откуда? по каким критериям? а в идеале объясните что вы пытаетесь сделать и как это должно работать

@Phantom-84 · 29.08.2018, 12:28

gbsoftware, видимо, у автора форма находится или должна находиться на странице конкретной записи.

Kameramann, на странице конкретной записи должен быть доступен ее id. Передавайте его в POST-обработчик в скрытом поле, как писали выше, или непосредственно в адресе обработчика:

PHP/HTML
1
<form  method='POST' action="../connect.php?id=<?= $id ?>">

Или вы не можете сделать отдельные страницы для записей?

Добавлено через 3 минуты
Избавиться от использования устаревшего расширения для работы с базами данных нужно обязательно. Сделать защиту от SQL-инъекций тоже.

Добавлено через 3 минуты
И с логикой разберитесь, например:

PHP
1
2
3
4
5
   if (isset($_POST['id'])) { $id = $_POST['id']; if ($id == '') { unset($id);} } 
  $query = mysql_query("UPDATE tecnikal_zakaz
   SET status = '".$status."',
   
   WHERE id='$id'");

Есть масса шансов, что $id не попадет в область видимости при вызове функции.

Добавлено через 10 минут
Зачем что-то писать в ответ на POST-запрос, когда результат можно и нужно показать наглядно? Делайте перенаправление "на себя" методом GET. Дополнительные оповещения могут быть полезны, особенно в случае возникновения ошибки, но только как приложение к показу актуального состояния страницы.

@Kameramann · 29.08.2018, 13:47 **[ТС]**

$id откуда брать?

Добавлено через 7 минут
ну или как вывести в форме нужный id и передать его?

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
<form  method='POST' action="../connect.php">
           <select  type="select" name="status">
           <option value="" selected="selected" disabled="">Выберите значение</option>
        <option VALUE="В обработке">В обработке</option>
        <option VALUE="Завершено">Завершено</option>
     </select>
       <?php
 $dbcon = mysql_connect("localhost", "root", ""); 
    mysql_select_db("users", $dbcon);
  if (!$dbcon)
  {
    echo "<p>Произошла ошибка при подсоединении к MySQL!</p>".mysql_error(); exit();
    } else {
    if (!mysql_select_db("users", $dbcon))
    {
    echo("<p>Выбранной базы данных не существует! коннект</p>");
    }
  }
$sql = mysql_query("SELECT `id` FROM `tecnikal_zakaz` ", $dbcon);
     while ($result = mysql_fetch_array($sql)) {
            echo("  <a>".$result['id']."</a>");
   }
   
    mysql_close($dbcon);
?>
     </div>
          <div class="modal-footer">
          <div class="btn-animate">
            <input  class="btn-signin" type="submit" name="submit" value="ДОБАВИТЬ" >
          </div></form>

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@Para bellum
	29.08.2018, 09:46
	Не по теме: Kameramann, и переходите, пока не поздно, на PDO или MySQLi. Расширение mysql было актуально лет семь назад. Пять лет назад было признано устаревшим. 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 09:48 [ТС]
	а как брать id текущей записи? Обновляет но только если явно указать что надо обновить 78 запись. А надо чтобы он сам обновлял открытую запись Добавлено через 52 секунды на 7 версию я перешел бы, если кодов и примеров было бы достаточно 0

@gbsoftware 120 / 116 / 77 Регистрация: 19.08.2013 Сообщений: 430
	29.08.2018, 10:01
	Что значит "открытую запись"? Примеров по PDO или MySQLi более чем предостаточно, как собственно и по 7 версии PHP к каковой они не имеют отношения. 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 10:03 [ТС]
	Он должен брать сам id записи и обновить ее, а не явно указывать номер записи 0

@gbsoftware 120 / 116 / 77 Регистрация: 19.08.2013 Сообщений: 430
	29.08.2018, 10:06
	откуда он должен брать это id? вы форму на основании базы строите? записи в базе откуда берутся? 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 10:11 [ТС]
	записи выводятся из бд Форма должна менять значение в поле в записи бд 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 10:46 [ТС]
	эт на седьмой версии написали 0

@gbsoftware 120 / 116 / 77 Регистрация: 19.08.2013 Сообщений: 430
	29.08.2018, 10:49
	Если вы имеете ввиду версию PHP, то нет 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 10:58 [ТС]
	Warning: mysqli_query() expects parameter 1 to be mysqli, resource given in on line 317 Warning: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, null given in on line 318 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 11:22 [ТС]
	во первых не работает, во вторых в форме выводятся кнопки добавить соответствующие числу записей 0

@gbsoftware 120 / 116 / 77 Регистрация: 19.08.2013 Сообщений: 430
	29.08.2018, 11:24
	это пример, я не знаю ни структуры вашей базы, ни того что вы пытаетесь сделать, соответственно не могу выдавать вам 100% рабочий код 0

@Kameramann 1 / 1 / 0 Регистрация: 06.12.2016 Сообщений: 55
	29.08.2018, 11:40 [ТС]
	так как вывести текущую запись? а не все записи Добавлено через 1 минуту нужно выбрать нужную запись и редактировать там определенное значение, а не все записи выводить 0

@gbsoftware 120 / 116 / 77 Регистрация: 19.08.2013 Сообщений: 430
	29.08.2018, 11:41
	объясните что вы имеете ввиду под текущей записью? выбрать нужную запись откуда? по каким критериям? а в идеале объясните что вы пытаетесь сделать и как это должно работать 0