Undefined variable в чем проблема?

Question

[xormakor]

В общем, делаю редактирование в админке. При нажатии на кнопку "редактировать" нужно переходить в окно с редактированием, собственно, реализовано вот так:

<form action="actions/city/delete_city.php" method="post">
            <label for="name">
                Редактирование
            </label>
            <button>Редактировать</button>
        </form>

После он переходит в delete_city.php, но уже выдает ошибку, сам файл выглядит так:

<?php
require_once $_SERVER['DOCUMENT_ROOT'] . "/www/functions/city.php";
deleteCity($id);
exit();

в city.php он выбирает функцию deleteCity, которая выглядит следующим образом:

function deleteCity($id) {
    global $mysqli;

    try {
        $city = $mysqli->query("SELECT * FROM city WHERE id = {$id}")->fetch_assoc();

        if(empty($city)) {
            return json_encode("Город, который вы хотите удалить не существует!");
        }
        
        return $mysqli->query("DELETE FROM city WHERE id = {$city['id']}");

    } catch(Exception $ex) {
        return $ex;
    }
}

выдает ошибку: Warning: Undefined variable $id in C:\OSPanel\domains\site\actions\city\delete_city.php on line 3

Comments

[Dmitry Bay]

ну так вам и пишут что $id нет.
вы не инициализировали $id.

Выглядеть должно как то так -

$id = $_GET['id'] ?? null;

if ($id) {deleteCity($id);}

[Сергей delphinpro]

<?php
require_once $_SERVER['DOCUMENT_ROOT'] . "/www/functions/city.php";
deleteCity($id); // Откуда эта переменная $id взялась?
exit();

[xormakor]

Dmitry Bay, $id берется уже как раз из функции, файл обращается к ней, она так и называется

[Дмитрий]

xormakor,

<form action="actions/city/delete_city.php?id=<?=$_GET['id']>" method="post">

возможно порешает вашу проблему

[Дмитрий]

xormakor, ну а так этому коду лет 10, и тут куча косяков)
оно может быть и работало на пхп5 с register_globals=On

[xormakor]

Дмитрий, все сломалось

[xormakor]

Дмитрий, делаю для курсового проекта и обучаюсь вместе с этим, так что да, код может быть староват. Но хочется чтобы он работал хотя бы)

[Дмитрий]

xormakor, тут несколько проблем, формы лучше писать формами с сабмитами, а не бутонами
вы используете ооп стиль mysqli, но не пользуетесь подготовленными выражениями, ваша форма должна передавать ид, первая ссылка на редактирование с ид элемента, вторая на удаление с ид элемента, используется global для прокидывания базы в функцию, используются инклюды

Answer 1

[MKKM MMM]

ну вы же не передаете $id от куда он должен знать что в какую кнопку вы нажали