Ошибка с выводом переменной?

Question

[dimavfox]

if(isset($_GET['edit']))
	{
		$id = $_GET['edit'];
		$get = mysqli_query($db, "SELECT * FROM users WHERE id = '$id'");
		$str = mysqli_fetch_array($get);
	}
if(isset($_GET['delete']))
	{
		$id = $_GET['delete'];
		mysqli_query($db, "DELETE FROM users WHERE id='$id' ");
		header("location: users.php");
	}

	if(isset($_POST['update'])){
		$name = $_POST['username'];
		$email = $_POST['email'];
		$password = $_POST['password'];
		$btc_add = $_POST['btc_add'];
		$eth_add = $_POST['eth_add'];
		$bch_add = $_POST['bch_add'];
		$btc = $_POST['btc'];
		$eth = $_POST['eth'];
		$bch = $_POST['bch'];
		$btc_dep = $_POST['btc_dep'];
		$eth_dep = $_POST['eth_dep'];
		$bch_dep = $_POST['bch_dep'];

		mysqli_query($db, "UPDATE users (name, email, password, btc_add, eth_add, bch_add, btc, eth, bch, btc_dep, eth_dep, bch_dep) VALUES ('$name', '$email', '$password', '$btc_add', '$eth_add', '$bch_add', '$btc', '$eth', '$bch', '$btc_dep', '$eth_dep', '$bch_dep') ");
	}

<input type="text" name="username" id="" value="<?php echo $str['name']; ?>" />

Выдает Notice: Undefined variable: str in

Сначала обьявил $str = "";, потом $str = " ";, и потом $str="name";
Не помогает дает ошибку по массиву

Как исправить?

Comments

[noeer]

У вас же переменная $str создается по условию if(isset($_GET['edit']))
Если условие не выполняется, то переменная создана не будет.
К тому же, если выборка из БД вернет FALSE, то $str['name'] также не будет объявлена.
Проверяйте isset($str['name']) перед выводом.

[ThunderCat]

Выше noeer верно написал, и в данном контексте лучше использовать не isset($_GET['edit']), а !empty($_GET['edit']).

[dimavfox]

ThunderCat, !empty($_GET['edit']) не работает

[dimavfox]

noeer, не работает

[noeer]

В смысле не работает?
Если $_GET['edit'] не передан, или пользователь не найден в БД, то поле username заполняться и не будет.
Главное что ошибка в этом случае должна исчезнуть.

[dimavfox]

noeer,

<input type="text" name="username" id="" value="<?php if(!empty($str['name'])) echo $str['name']; ?>" />

Ошибка пропала, но в БД ничего не пишется

[noeer]

Вместо !empty проверяйте isset, т.к. $str['name'] может быть вообще не определен.
Или лучше так
if(isset($str['name']) && !empty($str['name']))

[dimavfox]

noeer, У меня есть таблица, в которую выводятся пользователи с их данными, и кнопки к каждой строке "Edit" and "Delete"
Я нажимаю на Edit, в форму ниже выводятся все данные пользователся, их можно редактировать и после нажимаю кнопку Update и вот эти все данные в форме должны записаться в БД

[ThunderCat]

dimavfox, это совет по коду вообще, у вас же конкретно вообще редкая хрень с кодом.

[dimavfox]

ThunderCat, какая?

[ThunderCat]

dimavfox, я написал подробно, но все равно думаю это бесполезно, если все остальное так же написано. Первый же заход более-менее хитрожопого юзера на ваш проект, и в лучшем случае вы потеряете все базы, в худшем вам ваши бтц накрутят так, что не расплатитесь за ближайшие 300 лет.

[dimavfox]

noeer, Так смотри, если при клике на Edit в форму выводятся значения юзера, то значит $_GET['edit'] существует, так ведь?

[ThunderCat]

noeer,

Вместо !empty проверяйте isset, т.к. $str['name'] может быть вообще не определен.

доки читайте, и глупости не несите.

[dimavfox]

ThunderCat, ага, спасибо за конструктив

[ThunderCat]

dimavfox,

и вот эти все данные в форме должны записаться в БД

у вас там еще и пароль в открытом виде хранится, ваще найс )

Answer 1

[ThunderCat]

1) $id = $_GET['edit']; здесь $id в итоге может быть вообще пустой, или с шикарным sql инжектом.
2)

$get = mysqli_query($db, "SELECT * FROM users WHERE id = '$id'");

можно только надеяться что выше есть объявление $db

3)$str = mysqli_fetch_array($get); неплохо бы проверить что запрос что-то вернул вообще
4) if(isset($_GET['edit'])) если это условие не выполняется, переменная $str вообще нигде не будет создана.

ниже даже смысла смотреть нет, все то же самое, в лучших традициях гуанокодинга.

Comments

[dimavfox]

Понятно, исправить как?

[ThunderCat]

dimavfox, pdo, prepared statements, объекты, читать про архитектуру MVC, взять какой-нибудь толковый фреймворк, типа ларавел или Yii, много чего короче. У вас каждая строчка почти смотрится как пример "как не надо делать", увы.