Как загрузить файл и записать его имя в бд?

Question

[viktorross]

здравствуйте, подскажите пожалуйста, я пытаюсь из шаблона передать и записать значения input, но мне нужно дополнительно загрузить фото и подставить id, подскажите как загрузить фото и добавить его в бд? и не пойму почему вместо $id в user_id ничего не записывает, либо 0

function add_ban() {

$id_us=users::userIdget($id);


		global $db;
		$this->clean=array();
		$this->clean['title'] = escape($_POST['title']);
		$this->clean['location'] = escape($_POST['location']);
		$this->clean['max_clicks'] = escape($_POST['max_clicks']);
		$this->clean['max_impressions'] = escape($_POST['max_impressions']);
		$this->clean['link'] = escape($_POST['link']);

	$path = 'images/baners/';

		if (!@copy($_FILES['image']['tmp_name'], $path . $_FILES['image']['name']))

		$this->clean['position'] = escape($_POST['position']);
		$db->query('insert into `class_banners` set `title` = "'.$this->clean['title'].'",

		`location` = "'.$this->clean['location'].'", `max_clicks` = "'.$this->clean['max_clicks'].'",

		 `max_impressions` = "'.$this->clean['max_impressions'].'", `link` = "'.$this->clean['link'].'",

		 `position` = "'.$this->clean['position'].'", `user_id` = "'.$id_us.'",  `position` = "'.$this->clean['position'].'", `filename` = "'.$this->clean['image'].'";');
		return 1;

	}

саму фотку добавляю обычно в форме

<input type="file" style="width: 350px;" name="image" id="image" />

Comments

[Дмитрий]

Дырка не важна?

Answer 1

[Алексей]

$id_us=users::userIdget($id);
А откуда внутри функции add_ban $id берётся? Его нет

Comments

[viktorross]

wagoodoogoo она раньше идёт, я так ее по всему файлу вызываю чтобы id подставлять в бд, но именно в этой функции она почему-то не вставляет значение

[Алексей]

Внутри функции $id не виден, надо передать в функцию аргументом add_ban($id) , или global $id; на худой конец внутри функции написать.

[viktorross]

wagoodoogoo, хорошо, а как фотку загрузить и передать в бд?

[Дмитрий]

viktorross, передавать надо имя и путь изображения.
Не используйте copy(), замените на move_uploaded_file().
Не используйте "@", этим Вы делаете только хуже в данном случае.

[Алексей]

viktorross, https://www.php.net/manual/ru/features.file-upload...

[Дмитрий]

viktorross, откуда у Вас берётся $this->clean['image']?

[viktorross]

Дмитрий, я что-то пока ковырял это , скрипт вообще перестал работать, проверил уже тысячу раз все, все правильно, но не работает

посмотрите пожалуйста, может вы что-то поймете

function add_ban() {

		global $db;

		$this->clean=array();
		$this->clean['title'] = escape($_POST['title']);
		$this->clean['location'] = escape($_POST['location']);
		$this->clean['max_clicks'] = escape($_POST['max_clicks']);
		$this->clean['max_impressions'] = escape($_POST['max_impressions']);
		$this->clean['link'] = escape($_POST['link']);
		$this->clean['position'] = escape($_POST['position']);

		$db->query('insert into `class_banners` set `title` = "'.$this->clean['title'].'",`location` = "'.$this->clean['location'].'",`max_clicks` = "'.$this->clean['max_clicks'].'",`max_impressions` = "'.$this->clean['max_impressions'].'",`link` = "'.$this->clean['link'].'",`position` = "'.$this->clean['position'].'";');

		return 1;

	}

функция пока без id и фото, т.к и так не понятно что сломалось

вызов

$ctyp=new users();
if(isset($_POST['now_ads'])){
	if(!$ctyp->add_ban()) {
	} else {
		header ('Location: ads.php');
		exit(0);
	}
}

<form name="bg" method="post" action="">

<input type="text" name="title" value="" size=5 maxlength="20" />
<br/>
<br/>
<input type="text" name="location" value="" size=5 maxlength="20" />
<br/>
<br/>
<input type="text" name="max_clicks" value="" size=5 maxlength="20" />
<br/>
<br/>
<input type="text" name="max_impressions" value="" size=5 maxlength="20" />
<br/>
<br/>
<input type="text" name="link" value="" size=5 maxlength="20" />
<br/>
<br/>
<select name="position" value="">
<option value="header">Header</option>
</select>
<br/>
<br/>
<input type="submit" class="button positive" name="now_ads" id="now_ads" value="{$lng.general.submit}" />

</form>

правильно же все вроде бы

Answer 2

[ThunderCat]

1) $id видимо определен выше вызова функции, но как уже вам указали - область видимости внутри функции - локальная, все что не определено внутри тела функции для нее не существует, исключение - глобальные переменные.
2) Научитесь пользоваться дебаггером, например xdebug, или хотя бы используйте var_dump().
3) sql запросы СНАЧАЛА формируете в виде строки, затем передаете в вызов, тогда можно будет их вывести и посмотреть что с ними не так, подозреваю что в варианте который вы "поправили" проблема в отсутствии поля user_id которое скорее всего обязательное. Можно проверить напрямую введя этот запрос в консоль мускуля или через phpmyadmin.
4) Код кошмарный, во-первых зачем-то засунут в функцию, хотя этот код вряд ли будет использоваться дважды, во-вторых функция использует внутри себя глобальные переменные, что плохо. Почитайте про чистые функции. Про метод "проверки" переменных вообще нет слов, сплошь междометия...

Comments

[viktorross]

ThunderCat , спасибо большое, поправил, вроде бы заработало, почему-то после удаления user_id из бд, потом добавил, но id все равно не записывает никак, пробовал по разному, скажите что в этих глобалах такого? дело в том, что скрипт написан давно, и он весь на них построен, т.е файлов так 300 по 100 кб, все это дело менять очень сложно будет, если так и оставлять дальше, что будет? И какую проверку сделать формам и как правильно?

[ThunderCat]

viktorross,

почему-то после удаления user_id из бд

ээ, зачем? Просто можно было поставить его необязательным.

скажите что в этих глобалах такого?

Ничего в них, просто чистая/хорошая функция должна выдавать всегда одинаковый результат при любых входных параметрах, а когда она использует кучу глобальных переменных результат слабо предсказуем. Вроде я сказал что читать по этому поводу.

И какую проверку сделать формам и как правильно?

Валидация в php тема обширная, есть как нативные средства, так и более навороченные библиотеки.

если так и оставлять дальше, что будет?

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

но id все равно не записывает никак,

Так он туда не попадает, вам же уже объяснили, переменная $id НЕ ВИДНА ВНУТРИ ФУНКЦИИ. Если хотите чтобы она была видна - либо до использования добавьте ее в $this->id например, или же передайте ее в аргумент функции.

[viktorross]

ThunderCat, спасибо большое за ответ)

вот смотрите, я добавил

$this->id=$id;

затем добавил это

, `user_id` = "'.$this['id'].'"

после этого новая запись в бд перестает создаваться

и так не хочет

if(!$id) $id=$this->id;
`user_id` = '.$id.'

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

$this->id->clean['user_id'] = escape($_POST['user_id']);
`user_id` = "'.$this->clean['user_id'].'"

[viktorross]

ThunderCat, все разобрался, у меня же смарти, в файле где идет вызов функции забыл передать id пользователя

$ctyp=new users();
if(isset($_POST['post_bann'])){
	if(!$ctyp->add_ban($crt_usr)) {
	} else {
		header ('Location: ads.php');
		exit(0);
	}
}

add_ban($crt_usr) было add_ban()

а дальше сделал так

function add_ban($id='') {

и так

 `user_id` = "'.$id.'"'

теперь то что нужно

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