Cześć,
Tabela typu produktu wygląda mniej więcej tak:
ID | type | i tak dalej
1 | dvd | xx
2 | book | yy
3 | furniture | zz
4 | furniture | aa
5 | furniture | bb

Wyświetlam z bazy te dane i wrzucam w DIVy. Chcę zrobić tak aby dla wartości które się nie powtarzają każda lądowała do oddzielnego DIVa (dvd, book) a te które występują więcej niż raz żeby lądowały do jednego wspólnego diva (furniture). Chcę osiągnąć takie coś:

[HTML] pobierz, plaintext 
<div id="book">1 - xx</div>
<div id="dvd">2 - yy</div>
<div id="furniture">3 - zz | 4 - aa | 5 - bb</div>
[HTML] pobierz, plaintext 

Kombinuję z IFami i zmiennymi pomocniczymi, gdzie zapisuję do zmiennej type (dvd, book, furniture) aktualnie iterowanego w pętli wiersza i porównuję w następnej iteracji czy są takie same. Jeśli tak, to wiadomo że powinna wylądować do już istniejącego DIVa. Ale problem w tym, że przecież taki DIV trzeba najpierw otworzyć a potem zamknąć. I tutaj już się moja metoda nie sprawda.
Naprowadźcie pls.
Dzięki.

Powinienies zaczac od prawidlowego przygotowania danych, czyli pogrupowania, a potem wyswietlanie to bajka
Tu masz opisane przykladowe grupowanie
http://nospor.pl/grupowanie-wynikow.html

Ok, Twój samouczek jest dla mnie jasny i chętnie bym z niego skorzystał, ale niestety muszę to zrobić obiektowo. I tutaj pojawia się dla mnie trudność.

Tak wygląda wycinek mojej klasy, w której generalnie listuję większość rzeczy:

[PHP] pobierz, plaintext 
namespace nameOne;
 
class ProductAdd
{
 
 
    public function listFullProperties()
    {
        $properties = $this->getFullProperties();
        return $properties;
    }
    private function getFullProperties()
    {
        $db = new DB();
        $db->query("SELECT * FROM property ORDER BY type ASC");
        $results = $db->resultSet();
        return $results;
    }
 
}
[PHP] pobierz, plaintext 

Tutaj jest metoda resultSet(); z klasy DB:

[PHP] pobierz, plaintext 
class DB
{
    private $dbh;
    private $stmt;
 
    public function __construct()
    {
        $this->dbh = new \PDO("mysql:host=" . DB_HOST . ";dbname=" . DB_NAME, DB_USER, DB_PASS);
    }
 
    public function resultSet()
    {
        $this->execute();
        return $this->stmt->fetchAll(\PDO::FETCH_ASSOC);
    }
 
}
[PHP] pobierz, plaintext 

Klasę DB zmodyfikowałem na własny użytek na podstawie kursu OOP który przed świętami robiłem. Tak więc sam tego nie wymyśliłem, ale rozumiem co tam się dzieje. Niestety dla mnie, zamiast starego poczciwego while ($row = mysql_fetch_array($res)) mamy tutaj metody i obiekty, z którymi jeszcze nie jestem jakoś ekstra zaznajomiony. a PDO to dla mnie na razie niemalże czarna magia - wiem że jest, potrafię w prostym przykładzie użyć i to tyle.
Jak powyższe zmodyfikować, aby uzyskać pogrupowane wyniki, tak jak to opisałeś?

Tutaj

private function getFullProperties()
{
$db = new DB();
$db->query("SELECT * FROM property ORDER BY type ASC");
$results = $db->resultSet();
return $results;
}

w $results masz teraz tablice swoja. No to dopisz teraz metode w tej klasie ktora pogrupuje te tablice. To nei ma znaczenia czy to jest klasa czy nie. Metoda to metoda

zamiast mojego

while ($row = mysql_fetch_array($res)){

ty lecisz FOREACH po $results i tyle. petla to petla. Jak to sam mowisz, zadne rocket sience

Hmm, no w sumie logiczne. Dzięki za pomoc, muszę teraz lecieć z córką na basen ale wieczorem przysiądę do tego.

OK, elegancko grupuje. Posłużyłem się przykładem 2. Jednakże mam teraz problem aby wyświetlić pozostałe kolumny z bazy. Przy jednej wartości (w przykładzie 'name-miasto' u mnie 'property.property') to jest ok. Ale z tabeli property mam do wyświetlenia jeszcze type, label i description.
Posługując się przykladem 2 nospora, w linijce 12 mamy $name = $row['NAME']; i to tutaj mogę dodać wartości pozostałych kolumn, jednakże jako string, a przeciez potrzebuję aby wartość z każdej kolumny była oddzielną zmienną, lub elementem tablicy.
Mozna zrobić tak w linijce 12 z przykładu 2:

[PHP] pobierz, plaintext 
$name = array();
$name[] = $row['NAME'];
$name[] = $row['DESCRIPTION'];
$name[] = $row['kolejnaKolumna'];
[PHP] pobierz, plaintext 

Ale to uniemożliwia wyświetlenie danych w 2 foreachach bo robi się tablica z kolejnym wymiarem.
Naprowadźcie pls.
Thx.

pokaz caly kod co teraz masz bo nie kumam problemu

No i skoro twoja struktura ma przechowywac wiecej info to powinienies raczej isc w kierunku przykladu 1 a nie 2. no i pamietaj, to tylko przyklady, przeciez ty mozesz dodawac tam weicej info

Kawałek klasy ProductAdd:

[PHP] pobierz, plaintext 
    private function getFullProperties()
    {
        $db = new DB();
        $db->query("SELECT * FROM property ORDER BY type ASC");
        $results = $db->resultSet();
        return $results;
    }
 
    public function groupFullProperties()
    {
        $properties = array();
        foreach ($this->getFullProperties() as $row) {
            $property = $row['property'];
 
            $newProperty = $row['type'];
            if (!isset($properties[$newProperty])) {
                $properties[$newProperty] = array();
            }
            $properties[$newProperty][] = $property;
 
        }
        return $properties;
    }
[PHP] pobierz, plaintext 

wyświetlanie pogrupowanych wartości:

[PHP] pobierz, plaintext 
 
                    foreach ($properties->groupFullProperties() as $type => $property) {
                        echo '<div style="border:1px solid red; margin:5px 0 0 5px;" class="TEMPdynamicFields" id="">' . $type . '';
                        foreach ($property as $oneProperty) {
                            echo '<div style="margin:5px 0 0 5px;">' . $oneProperty . '</div>';
 
                        }
                        echo '</div>';
                    }
[PHP] pobierz, plaintext 

Powyższe działa, ale...
W groupFullProperties() powinienem mieć jeszcze pozostałe wartości z wiersza. Uff, zmęczyłem się, idę na fajkę.




No właśnie jakośc nie mogę wpaść w którym miejscu w metodzie. Może dodatkowy poziom nikotyny pomoże...

A gdzie sa inne kolumny/dane co chciales dodac?

No w powyższym nie ma (bo to działa), natomiast pisałem, że próbowałem je dodać w taki sposób:

[PHP] pobierz, plaintext 
//linijka 13 z klasy ProductAdd
  $property = $row['property'].' '.$row['type'];
[PHP] pobierz, plaintext 

Ale wiadomo, teraz mam zwykły string, a nie o to chodzi.

Lub w taki sposób:

[PHP] pobierz, plaintext 
//od linijki 13 z klasy ProductAdd
  $property['property'] = $row['property'];
  $property['type'] = $row['type'];
 
//linijka 19
$properties[$newProperty]['property'] = $property['property'];
$properties[$newProperty]['type'] = $property['type'];
[PHP] pobierz, plaintext 

Obie metody złe

pokaz mi co dokladnie zawiera $row

print_r($row);

var_dump($row);


Zabieram się za analizę tego.

No to zamiast tylko nazwy property, wrzucaj tez wszustkie inne rzeczy dla tej property, czyli jako tablice a nie jako text

[PHP] pobierz, plaintext 
        foreach ($this->getFullProperties() as $row) {
            $property = $row['property'];
 
            $newProperty = $row['type'];
            if (!isset($properties[$newProperty])) {
                $properties[$newProperty] = array();
            }
            $properties[$newProperty][] = [
'property' => $property,
'label' => $row['label'],
'description' => $row['description']
 
];
 
        }
[PHP] pobierz, plaintext 

I juz. No tu naprawde nie ma zadnej filozofii.

A, ok.
ale to wtedy jeszcze wyświetlanie trzeba zmodyfikować:

[PHP] pobierz, plaintext 
                    foreach ($properties->groupFullProperties() as $type => $property) {
                        echo '<div style="border:1px solid red; margin:5px 0 0 5px;" class="TEMPdynamicFields" id="">' . $type . '';
                        foreach ($property as $oneProperty) {
                            foreach($oneProperty as $p)
                            echo '<div style="margin:5px 0 0 5px;">' . $p . '</div>';
 
                        }
                        echo '</div>';
                    }
[PHP] pobierz, plaintext 

No filozofii nie ma, ale jak tak się posiedzi nad tym to łeb puchnie i czasem ciężko wpaść na rozwiązanie

To straszne

Działa. Kurde dzięki nospor, naprawdę doceniam twoja pomoc.

Miałbym jeszcze jedno pytanie co do powyższego.
Przy 'type' == 'Furniture' chciałbym żeby 'description' było wyświetlane tylko raz, dla ostatniego row. Naprowadź pls.
Może na początku pętli $i=0; a w pętli $i++; i jeśli $i != 0 to nie wyświetlać description? Ale wtedy mi tylko pierwsze wyświetli a nie istatnie.

Pokaz kod wyswietlania po zmianach

Tak wygląda wyświetlanie:

[PHP] pobierz, plaintext 
                    foreach ($properties->groupFullProperties() as $type => $property) {
                        echo '<div style="border:1px solid red; margin:5px 0 0 5px;" class="dynamicFields" id="'.$type.'">' . $type . '';
                        foreach ($property as $oneProperty) {
                            foreach($oneProperty as $p)
                            echo '<div style="margin:5px 0 0 5px;">' . $p . '</div>';
 
                        }
                        echo '</div>';
                    }
[PHP] pobierz, plaintext 

Oczywiście to jeszcze wersja robocza bo np style= zostanie usunięte itd.

Ciagle mowisz o przejsciu na obiektowek, dobrych praktykach itp a jak patrze na twoj kod to mnie przerazenie bierze. Balagan, nazewnictwo zmiennych z 4 liter i wszedzie panujacy chaos. Zamiast brac sie za obiektowke naprawde warto by poprawic te mase drobnych rzeczy, ktore kola oczy.

np te nieszczesne nazwy zmiennych, ktore totalnie nic nie mowia ala
$newProperty = $row['type'];

pobierasz typ a nazywasz go newProperty. Co ma typ do newPRoperty? No nic

$type = $row['type'];
Prawda ze czytelniej?

Albo tu:
foreach ($properties->groupFullProperties() as $type => $property) {
Przeciez tam jest kilka properties a nie jedna property wiec tak ma sie nazywac
foreach ($properties->groupFullProperties() as $type => $properties) {

Moze ci sie to wydawac smieszne takei czepiaanie, ale przy wiekszym kodzie ty naprawde sie nie polapiesz co jest co, nie wpsonijac o innej biednej osobie majacej pracowac na twoim kodzie.

Sorki, musialem, moze dotrze

Wracajac do tematu: po co petla jeszcze na wyswietlanie description lang i property? Te wartosci znasz, petla jest totalnie zbedne i komplikuje zycie. Petle sie robi dla wartosci dynamicznych a nie stalych

[PHP] pobierz, plaintext 
                        foreach ($properties->groupFullProperties() as $type => $properties) {
                            echo '<div style="border:1px solid red; margin:5px 0 0 5px;" class="dynamicFields" id="'.$type.'">' . $type . '';
                            foreach ($properties as $propertyData) {
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['property'] . '</div>';
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['label'] . '</div>';
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['description'] . '</div>';
     
                            }
                            echo '</div>';
                        }
[PHP] pobierz, plaintext 

Czyz nie wyglada to czytelniej?


Teraz jak kod wyglada w miare ok, to mozna przejsc do problemu

[PHP] pobierz, plaintext 
                        foreach ($properties->groupFullProperties() as $type => $properties) {
                            echo '<div style="border:1px solid red; margin:5px 0 0 5px;" class="dynamicFields" id="'.$type.'">' . $type . '';
                            foreach ($properties as $index => $propertyData) {
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['property'] . '</div>';
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['label'] . '</div>';
if ($type !== 'Furniture' || $index === count($properties) - 1) {
                                echo '<div style="margin:5px 0 0 5px;">' . $propertyData['description'] . '</div>';
}
 
                            }
                            echo '</div>';
[PHP] pobierz, plaintext 

Wiem, nad tym muszę popracować. Czasem wymyślenie sensownej nazwy zmiennej zajmuje mi za dużo czasu, więc walę $p= i tyle. wiem, że to zły nawyk, muszę się go pozbyć.

Ale jak to? Kod ogólnie poformatowany, odstępy, wcięcia porobione. Co do mieszania php z html to wiem, że tak nie powinno się robić, ale w takiej sytuacji jak tutaj opisuję, to ta pętla powinna być w funkcji w pliku z klasą i tylko wklejone wywołanie metody wklejam to pliku z html?



Absolutnie nie wydaje mi się to śmieszne. Lata pisania kodu tylko dla siebie (nigdy komercyjnie lub za drobne kwoty dla znajomych) spowodowały u mnie wyrobienie złych nawyków, z których mogę sobie nawet nie zdawać sprawy, lub ciężko mi się ich pozbyć. Staram się zacząć stosować PSR-12 ale do sukcesu jeszcze droga.


A no racja. Działa elegancko. Teraz muszę to ubrać w formularz, inputy, style itd.
Następnie zabieram się za problem z grupowania z tego nowszego tematu - 3 tabele, w tym łącznikowa.

Dzięki wielkie!!!!!

Nazewnictwo z 4liter wproadza dla mnie wystarczajacy balagan
Brak typowania
Do tego w html uzywanie inline STYLE zamiast klass i CSS dopelnia robote.


Na stronie z grupowaniem, do ktorej odeslalem cie wczesniej, masz rozwiazanie z grupowaniem w bazie, przy pomocy GROUP BY i GROUP_CONCAT. Moze to byc bardziej przydatne. Wszystko zalezy co i jak bedzesz chcial wyswietlac


Nie. Klasy nie powinny generowac zadnego kodu html.Do kodu html powinno uzywac sie widoku. Moze to byc zwykly plik .phtml czy tam nawet php a nie zadna klasa. Chodzi o rozdzielenie logiki od wyswietlania.
W mojej stopce masz link do Widoku takiego naprawde banalnego. Jakby cie interesowal cos powazniejszego to np. Twig

Nazewnictwo zmiennych - ok, przyjąłem i się z tym nie kłócę To trzeba usprawnić

Brak typowania - rozwiń trochę myśl pls.

Tak robię tylko roboczo, jak potrzebuję zobaczyć na szybko co się wyświetla, to daję style="border:1px solid red; i widzę. Potem usuwał style= z html i wszystkie style wędrują do css/style.css


Ok, to rozumiem - mam oddzielne pliki do klas i oddzielne do widoku (czasem *.html a czasem *.php jak ten omawiany powyżej).
W moim zadaniu mam narzucone OOP i powiedziane, że programowanie proceduralne może być stosowane jedynie do zainicjowania klas PHP, a reszta logiki powinna być umieszczona wewnątrz metod klas. Stąd moje wcześniejsze pytanie o to czy ten foreach powinien powędrować do klasy. Jak rozumiesz w takim razie powyższy wytłuszczony fragment? Czy nie doczepią mi się, że zastosowałem programowanie proceduralne w pliku widoku?


Wybacz pytanie, ale to artykuł z 2013. Czy te praktyki nadal obowiązuję w niezmiennej formie?

Kazda metoda ma parametry i cos zwraca. parametry maja miec ustawiony typ, tak samo jak metoda ma miec ustawione co zwraca np twoja metoda:

public function groupFullProperties()
{
...
}

Ona zwraca tablice wiec masz to powiedziec w definicji

public function groupFullProperties(): array
{
...
}

Jakbys mial metode z parametrami to tez masz powiedziec jakiego typu jest dany parametr.
public function someMethod(int $number, string $name): array
{
...
}


Do tego byles odsylany pare tematow temu


Na poczatek powinno wystarczyc


No jesli ktos ci kaze widok generowac w klasie to tak srednio na jeza ma pojecie. Wiec zakladam ze twoj zleceniodawca ma jednak pojecie i robienie widokow w oddzielnych plikach .php bedzie dla niego ok

Pytanie jak najbardziej zasadne. Klase te pisalem na php5 wiec jest daleka od poprawnych klas pisanych teraz w php8. Nie mniej jednak idea sie nie zmienila za bardzo a mi bardziej chodzilo o pokazanie idei a nie klasy

Super. Dzięki