Dziwne zachowanie funkcji count() Opcje

[Panicz74]

Witam,

Mam taki problem i już mi się pomysły kończą. Chcę użyć funkcji count() w wyszukiwarce aby w razie znalezienia wyników je pokazała, a w razie ich braku pokazała echo "Nic nie znaleziono". Niby proste ale jak używam count() to w przypadku znalezienia wielu rekordów pokazuje tylko jeden, pierwszy. W przypadku braku rekordów pokazuje co należy. W ogóle to dziwne co pokazuje print_r(count($row)): 12. Przecież aktualnie mógł znaleźć max 2 rekordy bo tak by to filtrowało.

Baza składa się z 5 tabel, gdzie jedna z nich zawiera relacje w 4 pozostałymi. Czyli wiele do jednej.

Linie odpowiedzialne za count:

[PHP] pobierz, plaintext 
foreach($stmt as $row)
				  count($row);
				  print_r(count($row));
				  if(count($row) >= 1)
				 {
				  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].': '.$row['date'].'</li>';				  
				  $stmt->closeCursor();
				  echo '</ul>';
				 }
				  else
				  {
					  echo "Nic nie znaleziono";
					  $stmt->closeCursor();
				  }
				  //print_r($_GET);
				  }
[PHP] pobierz, plaintext 

Cały kod:

[PHP] pobierz, plaintext 
if(isset($_GET['nazwa']))
{
		if($_GET['nazwa'] !== "Branża" || $_GET['umowa'] || $_GET['lokalizacja'])
		{
			$mapping = array(
			'nazwa' => 'nazwa_branzy',
			'lokalizacja' => 'lokalizacja',
			'umowa' => 'rodzaj_umowy'
			);
 
			$where = array(); 
			$params = array(); 
			$wh = array();
 
			foreach($_GET as $key => $value)
			{
				if(isset($mapping[$key]) && '' !== $value)
				{
					$where[] = $mapping[$key].' = :'.$mapping[$key];
					$params[':'.$mapping[$key]] = $value;
				}
			}
				if(!empty($where))
				{
					$wh = 'WHERE '.implode(' AND ', $where);
					print_r($wh);
				}
				else {echo "Nie zaznaczono żadnego pola"; die;}
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
					$sql =   'SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
								  LEFT JOIN users u ON u.id_usera = o.id_usera 
								  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
								  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
								  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok '.$wh;
				  $stmt = $pdo->prepare($sql);
				  if(!empty($_GET['nazwa'])) {$stmt -> bindValue(':nazwa_branzy', $_GET['nazwa'], PDO::PARAM_STR);}
				  if(!empty($_GET['umowa'])) {$stmt -> bindValue(':rodzaj_umowy', $_GET['umowa'], PDO::PARAM_STR);}
				  if(!empty($_GET['lokalizacja'])) {$stmt -> bindValue(':lokalizacja', $_GET['lokalizacja'], PDO::PARAM_STR);}
				  //if(empty($_GET['nazwa']) && empty($_GET['nazwa']) && empty($_GET['nazwa'])) {echo "Nie wybrano filtrów"; die;}
				  $stmt -> execute($params);
				  //if($where = '') echo "Nic nie znaleziono";
				  echo '<ul>';
				  foreach($stmt as $row)
				  count($row);
				  print_r(count($row));
				  if(count($row) >= 1)
				 {
				  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].': '.$row['date'].'</li>';				  
				  $stmt->closeCursor();
				  echo '</ul>';
				 }
				  else
				  {
					  echo "Nic nie znaleziono";
					  $stmt->closeCursor();
				  }
				  //print_r($_GET);
				  }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }			
		}
}
[PHP] pobierz, plaintext 

[Riggs]

Ten count powinien być przed pętlą foreach bo jak $stmt będzie puste to pętla i tak się nie wykona.
Zrzuć jeszcze otrzymany SQL.

[lukaskolista]

Tak na logikę: counta robisz na zmiennej $row, która przechowuje dane pojedynczego wiersza. Wniosek: liczysz kolumny wiersza, których zawsze jest 12 (klucze asocjscyjne i numeryczne czyli po 2 klucze z tą samąwartością w wyniku) a nie ilość rekordów w wyniku.

Ten post edytował lukaskolista 29.01.2016, 07:25:33

[Panicz74]

A istnieje jakiś sposób żeby zamiast po kolumnach iterować po wierszach? Bo tak się nad tym zastanawiam i nie mogę wpaść na to jak...

Co ciekawe, nawet jak dodam jedynie var_dump(count($row)) tak:

[PHP] pobierz, plaintext 
foreach($stmt as $row)
				  var_dump(count($row));		
				  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].': '.$row['date'].'</li>';				  
				  $stmt->closeCursor();
				  echo '</ul>';
[PHP] pobierz, plaintext 

to też pokazuje tylko jeden wynik chociaż powinien znaleźć dwa. W przypadku istnienia dwóch ogłoszeń o podanych kryteriach var_dump daje: int(12)int(12), w przypadku jednego daje int(12).

Otrzymany SQL przy wybraniu jednego pola:

[SQL] pobierz, plaintext 
SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o LEFT JOIN users u ON u.id_usera = o.id_usera LEFT JOIN branza b ON b.id_branzy = o.id_branzy LEFT JOIN umowa i ON i.id_umowy = o.id_umowy LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok WHERE nazwa_branzy = :nazwa_branzy
[SQL] pobierz, plaintext 

Ten post edytował Panicz74 30.01.2016, 02:15:42

[lukaskolista]

Jak się klamer nie używa to pokazuje tylko 1 (ostatni) Zapomniałeś użyć { ... }

[Panicz74]

No tak... Teraz działa:

[PHP] pobierz, plaintext 
				  foreach($stmt as $row)
				  {
				  //print_r($row);
				  //$result = count($row);
				  //echo $result;
				  echo '<li>'.$row['nazwa_branzy'].' - '.$row['rodzaj_umowy'].' - '.$row['login'].' - '.$row['tresc'].' - '.$row['lokalizacja'].' - '.$row['date'].'</li>'; 				  
				  }
				  if(empty($row)) {echo "Nie znaleziono ofert o podanych kryteriach :(";}
				  $stmt->closeCursor();
				  echo '</ul>';
[PHP] pobierz, plaintext 

Dlaczego potrafię zrozumieć coś takiego jak tworzenie i bindowanie tablicy przez execute() a łapię się na takich pierdołach? To już nie pierwszy raz. Też mieliście takie problemy kiedyś

[lukaskolista]

Tak, jak się nie formatuje porządnie kodu, to tak jest