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

Question

[Seream220]

Ребята может кто помочь
Пытаюсь с делать сайт и выходит ряд проблем.
Хочу к примеру вывести картинки на сайт с базы делаю всё так

$sql_img = $db->super_query("SELECT * FROM imgtop");
foreach($sql_img as $q)
	$img .= '<li  style="background-image: url({theme}/img/'.$sql_img['name'].');" class="overlay"></li> ';	
$tpl->set('{img}', $img);

В базе есть 4 картинки в таблице name указанны названия картинок. Картинку он выводит так:

<li  style="background-image: url(/system_sain/lk_system/system_tpl/komp/img/slider_1.jpg);" class="overlay"></li> 
<li  style="background-image: url(/system_sain/lk_system/system_tpl/komp/img/slider_1.jpg);" class="overlay"></li>

Почему то он сам вывод сразу 2 картинки одну и туже.
Как можно исправить это что бы выводил он одну картинку а не 2 сразу одно и туже?
Как можно сделать что бы он выводил с mysql все картинки которые там есть и они показывались?

Comments

[Максим Федоров]

вы делаете цикл:
foreach($sql_img as $q)
то есть далее в теле цикла, то есть в каждой итерации у вас есть перемнная $q
но вы выводите элемент из переменной $sql_img

$sql_img['name'] скорее всего (не факт, что глубже верно все у вас) заменить нужно $q['name']

[Seream220]

Максим Федоров, При замене как вы говорили он выводит вообще так.

<li  style="background-image: url(/system_sain/lk_system/system_tpl/komp/img/1);" class="overlay"></li> 
<li  style="background-image: url(/system_sain/lk_system/system_tpl/komp/img/s);" class="overlay"></li>

Вообще другое

[Максим Федоров]

Seream220, ну надо смотреть что у вас в массиве, по котормоу вы делаете цикл

[Seream220]

Максим Федоров, Можно как то с вами связаться в голосом? Скайп, вацап?

[Seream220]

Максим Федоров, А так за это же вы говорите?

function super_query($query, $multi = false, $cache_prefix = false, $system_cache = false){
		
		//Если включен кеш, то проверяем на его существование
		if($cache_prefix){
		
			if($system_cache)
			
				$data = system_cache($cache_prefix);
				
			else
			
				$data = mozg_cache($cache_prefix);
			
		}
		
		//Если есть ответ с кеша
		if($data){
			
			$unSerData = unserialize($data);
			
			if($unSerData)
			
				return $unSerData;
				
			else
			
				return array();
			
		} else {

			if(!$multi) {

				$this->query($query);
				$data = $this->get_row();
				$this->free();	

				//Если включен кеш, то создаём его
				if($cache_prefix){
				
					$cache_rows = serialize($data);
					
					mozg_create_cache($cache_prefix, $cache_rows);
					
				}
				
				return $data;
				
			} else {
				$this->query($query);
				
				$rows = array();
				while($row = $this->get_row()) {
					$rows[] = $row;
				}

				$this->free();			

				//Если включен кеш, то создаём его
				if($cache_prefix){
				
					$cache_rows = serialize($rows);
					
					if($system_cache)
					
						creat_system_cache($cache_prefix, $cache_rows);
						
					else
					
						mozg_create_cache($cache_prefix, $cache_rows);
				}
				
				return $rows;

			}
		
		}
	}

Answer 1

[Ningaro]

Выборка не правильно делается.
Советую ознакомиться - php.net/manual/ru/mysqli-result.fetch-row.php

Comments

[Seream220]

Пробую разными способами. На любой другой способ выдаёт мне сразу 500 ошибку сайт.
Можешь с этого написать пример?

[Ningaro]

Seream220,

$sql_img = $db->super_query("SELECT * FROM imgtop", true);

Секундочку, а так работает?

[Seream220]

Ningaro, Да так работает. Выводит все записи с базы. Но вот только не подставляет название картинок

Answer 2

[Uno]

foreach($sql_img as $q){
$img .= ...$sql_img['name']...

Comments

[Uno]

Пояснение:
Трай ту юз $q вместо $sql_img

[Seream220]

Noizefan zzz, Может лучше подскажешь как мне сделать что бы всё работало?

[Uno]

Seream220,

Пояснение:
Трай ту юз $q вместо $sql_img

[Seream220]

Noizefan zzz, Вот почему вы только и пишут как сделать. Но не смотрят что другим пишут. Все такие умные что говорят. Но такой вариант вообще не работает!!!!!