Witam,

Mam problem ktory idealnie nadaje sie na przedszkole i nie mogę nigdzie znaleść rozwiązania.

Mam funkcje która pobiera sobie dane:

[PHP] pobierz, plaintext 
function pobierz_liste_bydla($user_id) {
    $lacz = lacz_bd();
 
    $wynik = $lacz->query("select * from bydlo where user_id = '".$user_id."'") or die(mysqli_error($lacz));
    if(!$wynik) {
        return false;
    }
 
    //tworzymy tablice url
  $tablica_bydla =  $wynik->fetch_object();
 
    return $lista_bydla;
}
[PHP] pobierz, plaintext 

I funkcje która ma je wyświetlać:

[PHP] pobierz, plaintext 
function wyswietl_liste_bydla($tablica_bydla) {
?>
 
  <br />
 
  <form name="tabela_zak" action="usun.php" method="post">
 
  <table width="300" cellpadding="2" cellspacing="0">
 
  <?php
  $kolor = "#cccccc";
 
  echo "<tr bgcolor=\"".$kolor."\"><td><strong></strong></td>";
 
  echo "<td><strong>Usuń?</strong></td><td><strong>Usuń?</strong></td></tr>";
  $ile_bydla = "60";
  if ($ile_bydla > 0 ) {
        echo "<table style='border-collapse: collapse;'>
						<tr>
							<th style='width: 20px;'>id (sort)</th>
							<th style='width: 150px;'>Imię</th>
							<th style='width: 200px;'>Numer</th>
							<th style='width: 80px;'>Rasa</th>
                            <th style='width: 200px;'>Data urodzenia</th>
							<th style='width: 200px;'>Numer Matki</th>
							<th style='width: 80px;'>Płeć</th>
							<th style='width: 80px;'>operacja</th>
						</tr>";
    for($i=0; $i<$ile_bydla; $i++) {
        echo "<tr style='height: 30px;'>	
									<th class='tab_2'>".$tablica_bydla->id."</th>
									<th class='tab_2'>".$tablica_bydla->nazwa."</th>
									<th class='tab_2'>".$tablica_bydla->numer."</th>
                                    <th class='tab_2'>".$tablica_bydla->rasa."</th>
                                    <th class='tab_2'>".$tablica_bydla->data_urodzenia."</th>
                                    <th class='tab_2'>".$tablica_bydla->numer_matki."</th>
                                    <th class='tab_2'>".$tablica_bydla->plec."</th>
									<th class='tab_2'>
										<a href=''><img src='images/edit.png' alt='edytuj' title=skonfiguruj'/></a>
										<a href=''><img src='images/settingsadd.png' alt='edytuj usługi dodatkowe' title='skonfiguruj dodatkowe opcje monitoringu'/></a>
										<a href=''><img src='images/delete.png' alt='usun' title='usun'/></a>
									</th>
								</tr>";
    }
    }
    }
[PHP] pobierz, plaintext 

Napisałem to samo w jednym pliku i tutaj działa:

[PHP] pobierz, plaintext 
$zapytanie ="select * from bydlo where user_id=".$user_id."";
$wynik=$bd->query($zapytanie);
$ile=$wynik->num_rows;
if ($ile > 0 ) {
        echo "<table style='border-collapse: collapse;'>
						<tr>
							<th style='width: 20px;'>id (sort)</th>
							<th style='width: 150px;'>Imię</th>
							<th style='width: 200px;'>Numer</th>
							<th style='width: 80px;'>Rasa</th>
                            <th style='width: 200px;'>Data urodzenia</th>
							<th style='width: 200px;'>Numer Matki</th>
							<th style='width: 80px;'>Płeć</th>
							<th style='width: 80px;'>operacja</th>
						</tr>";
    for($i=0; $i<$ile; $i++) {
        $wiersz = $wynik->fetch_object();
        echo "<tr style='height: 30px;'>	
									<th class='tab_2'>".$wiersz->id."</th>
									<th class='tab_2'>".$wiersz->nazwa."</th>
									<th class='tab_2'>".$wiersz->numer."</th>
                                    <th class='tab_2'>".$wiersz->rasa."</th>
                                    <th class='tab_2'>".$wiersz->data_urodzenia."</th>
                                    <th class='tab_2'>".$wiersz->numer_matki."</th>
                                    <th class='tab_2'>".$wiersz->plec."</th>
									<th class='tab_2'>
										<a href='?page=admin_panel&act=edit&id=2'><img src='images/edit.png' alt='edytuj' title=skonfiguruj serwer'/></a>
										<a href='?page=edit_other&id=2'><img src='images/settingsadd.png' alt='edytuj usługi dodatkowe' title='skonfiguruj dodatkowe opcje monitoringu'/></a>
										<a href='?page=del_proof&id=2'><img src='images/delete.png' alt='usun' title='usun serwer'/></a>
									</th>
								</tr>";
    }
echo "</table>";
}
[PHP] pobierz, plaintext 

Widze gdzie jest różnica tzn że w 2 przykładzie dane z obiektu są pobierane w petli a tutaj je wysyłam stąd w 2 przykładzie mam ładnie pokazane wszystkie przykłady a w pierwszym mam pętle pierwszego rekordu ( bo tylko go przekazuje).


Pytania sa 2:

1) Jak przenieść for.... do funkcji pobierz_liste...... by w widoku czyli wyswietl_liste.... mógł sobie tylko podać nazwy $tablica_bydla->numer itp.?
2) Jak z funkcji pobierającej dane przekazać liczbe znalezionych rekordów? mam zrobić tam dodatkową funkcje liczącą czy poprostu dodać drugi argument do wyswietl_liste_bydla i jemu przekazać ilość?

Dzięki za pomoc.

Ad 1. No normalnie ... wstaw po prostu tą pętle do tej funkcji - samo fetch_object pobiera tylko jeden wiersz wyniku, dlatego musisz zrobić pętle za pomocą właściwości num_rows obiektu wyniku - całość tego robisz po stronie modelu i zwracasz dane.

Ad 2. Nie musisz przekazywać żadnej ilości, po prostu poszczególne znalezione rekordy dodajesz do tablicy zanim je przekażesz do widoku, a w nim jeżeli chcesz wyświetlić wszystko robisz pętle i tyle.


Ogólnie to powinno działać tak:

1. Odpowiednia metoda kontrolera zgłasza widokowi żądanie wyświetlenia odpowiednich rekordów przekazując mu obiekt modelu, oraz id żądania.
2. Widok pobiera sobie z modelu surowe dane, wczytuje odpowiedni szablon z elementami dynamicznymi, czyli ze wstawkami zmiennych i łączy wszystko w całość i wypluwa wszystko na wyjście.

OK, czyli pobierz dodałem:

[PHP] pobierz, plaintext 
    $ile=$wynik->num_rows;
        for($i=0; $i<$ile; $i++) {
        $tablica_bydla =  $wynik->fetch_object();
        }
    return $tablica_bydla;
}
[PHP] pobierz, plaintext 

Według tego skrypt wyświetlania powinien działać niezmieniony, jednak caly czas pokazuje mi pętle pierwszego rekordu - jak to rozgryść?
Możesz podać kod jak to powinno wyglądać? Po stronie widoku też mam dodać pętle?

Wyniki musisz wrzucić do tablicy, czyli tak:

[PHP] pobierz, plaintext 
$tablica_bydla[] =  $wynik->fetch_object();
[PHP] pobierz, plaintext 

I taką tablicę przekazujesz do widoku i tam również robisz pętle po niej jeżeli chcesz wyświetlić wszystkie dane z poszczególnych rekordów - można to oczywiście trochę modyfikować wczytując layout pojedynczego wyniku w wszystko składać w klasie widoku, ale to już inna bajka.

I nadal jestem w tym samym miejscu, bo obiekt mam przekazany, ale pokazuje mi tylko jeden rekord ( teraz już nie pierwszy a ostatni - mogę prosić o napisanie funkcji która wyświetli pokolei te rekordy? Bo to jest głownie problemem że nie wiem jak je obrobić i wyświetlić od strony widoku.

Stary... Zwracasz $tablica_bydla (jeden obiekt) tak?
I zobacz w swojej petli for tylko nadpisujesz swoj obiekt, ktory pozniej zwracasz...

Przerob tak:

[PHP] pobierz, plaintext 
$ile=$wynik->num_rows;
$tablica_bydla = array();
        for($i=0; $i<$ile; $i++) {
        $tablica_bydla[] =  $wynik->fetch_object();
        }
    return $tablica_bydla;
[PHP] pobierz, plaintext 

i w widoku robisz:

[PHP] pobierz, plaintext 
foreach($tablica_bydla as $bydle) {
    echo '<pre>Bydle[' . $bydle->id . ']: ' . print_r($bydle, 1) . '</pre>'; //oczywiscie ubierz to w html...
}
[PHP] pobierz, plaintext 

Olej tabelki. Zobacz co moze css3 -> flex box. szybko, latwo.

Dzięki ci bardzo - działa właśnie tak jak chciałem!

Pozwole sobie odświeżyć temat - jak z takiego zapytania pobrać wynik i jedną zmienną dać do innego zapytania?

Jak w widoku mam pętle foreach i wynik $bydle->id to chciałbym na podstawie tego wyniku zrobić zapytanie - ktoś pomoże mi to ugryść?

[PHP] pobierz, plaintext 
foreach($tablica_bydla as $bydle) {
//tu chce pobrać $bydle->id i sprawdzić w innej tabeli jaka jest nazwa dla tego id
    echo '<pre>Bydle[' . $bydle->id . ']: ' . print_r($bydle, 1) . '</pre>'; //oczywiscie ubierz to w html...
}
[PHP] pobierz, plaintext 

w strukturalnym poprostu robiłem po foreach zapytanie ale nie wiem jak to zrobić by nie było to w widoku.

//EDIT

Zrobiłem:

[PHP] pobierz, plaintext 
    foreach($tablica_zdarzen as $bydle) {
        require_once('funkcje/funkcje_obora.php');
        $lacz = lacz_bd(); 
    $zapytanie = $lacz->query("select * from bydlo where id = '".$bydle->bydlo_id."'") or die(mysqli_error($lacz));
$wiersz = $zapytanie->fetch_object();
[PHP] pobierz, plaintext 

nazwe wywołuje jako $wiersz->nazwa, ale takie coś mieć w widoku to wstyd

czy przerobienie tego na zasadzie:

[PHP] pobierz, plaintext 
    foreach($tablica_zdarzen as $bydle) {
        if ($nazwa_bydla = pobierz_nazwe_bydla(".$bydle->bydlo_id.")) {
    wyswietl_nazwe_bydla($nazwa_bydla);
}
[PHP] pobierz, plaintext 

będzie bardziej zgodne z oddzielaniem modelu/kontrolera od widoku?
jak to ugryść by miało ręce i nogi?