[smalczyk1]

#php #naukaprogramowania

Cześć.Jeżeli dostaje dane i chce stworzy swoje tablice zależnie od typu to robić to tak na switchu i powtarzać te pętle i przypisywać odpowiednie wartości do pól w array?Tablice będą różniły się nazwami pól .

[mariecziek]

@smalczyk1: No jeżeli masz jakiś algorytm przypisania wartości, to możesz to robić na switchu, ale po tym co tu załaczyłeś, to ma wrażenie, że najlepiej to zrobić w funkcji rekurencyjnej, bo te zagnieżdżone foreache wyglądają fatalnie

[Jare_K]

@smalczyk1: tak możemy zgadywać. Byłoby łatwiej gdybyś podał przykłady przed i po dla dwóch typów.

[smalczyk1]

@Jare_K: @mariecziek: ok faktycznie stworze jakiś przykład i wrzucę na sandboxa

[smalczyk1]

@Jare_K: @mariecziek: oczywiście struktura będzie bardziej złożona.http://sandbox.onlinephpfunctions.com/code/7635b25d5429c78500909b677c0c039cfd130291

[Jare_K]

@smalczyk1: obstawiam, że switch ($type) też musi mieć jakąś zewnętrzną pętlę, w której iterujesz po typach.
Może odwróć podejście i zrób sobie pętle po obiektach struktury a wewnątrz pętli zrób te switch ($type) i specyficzne instrukcje dla określonych typów, np.:
foreach (...) {
//wspólne działania
switch ($type)
case ...:
// specyficzne działania dla typu
}
Powinno być sporo mniej pętli.
Ale ogólnie to słabo rozumiem twój przykład z sandboxa, więc nic

[smalczyk1]

@Jare_K: Ogólnie dostaję z api kilka typów movie,book,song.Wszystkie mają taką samą strukturę json.Jednak każdy utwór zależnie od typu ma trafiać do innych tabel w bazie.Każda tabela ma już inne nazwy kolumn dla każdego typu utworu.Chcę stworzyć odpowiednie array dla typu,które będą zawierać wartości które chcę wrzucić do bazy.

[Jare_K]

Nic nie stoi na przeszkodzie, żeby zrobić tak jak podałem. Będziesz miał kilka pętli po liście obiektów i zbudujesz sobie na koniec każdej iteracji jedno zapytanie do bazy zapisujące w odpowiedniej tabeli, zamiast robić tych zapytań wiele, tyle ile typów porozrzucane po różnych pętlach. W razie konieczności zmian nie będziesz musiał zmieniać wielu miejsc, tylko jedno.

[smalczyk1]

@Jare_K: A może JsonMapper() zmapować najpierw to do obiektu Utwór i potem zależnie od typu tworzyć arrary ?
Book
$insertDataSong['title']=Utówr->title
Movie
$insertDataMovie['orginal-title']=Utwór->title
Song
$insertDataSong['song-title']=Utwór->title

[Jare_K]

@smalczyk1: Skoro to i tak ma lecieć wprost do bazy, to obiekty (tablice) pośrednie są chyba w ogóle niepotrzebne?
Możesz mapować wprost do zmiennych PDO, np.:
$sth->bindValue(':originalTitle', Utwór->title, PDO::PARAM_STR);
i zależnie od typu w switchu mapować wybrane własności obiektów.
Chyba, że masz ORMa jakiegoś.

[smalczyk1]

@Jare_K: Ok dzięki za ukierunkowanie.Z tym JsonMapper() to myślałem że sam mi odnajdzie wartości po kluczu.Tak to muszę wszystkie obiekty odwzorować w klasach a trochę tego będzie.Tak więc zostanę przy pętlach

versions=>0=>object=>authors
'relationships' =>links=>authors
'relationships' =>links=>magazine

ijeszcze więcej zagnieżdżeń

[Jare_K]

@smalczyk1: pewnie, jak masz gotową opcję na mapowanie obiektów/tablic, to nie ma co ręcznie się bawić tylko skorzystać z gotowych rozwiązań.

[smalczyk1]

@Jare_K: Sam będę mapował w tych pętlach.Ja myślałem że ten JsonMapper mi wybierze tylko te wartości z array które są w klasie Movie i że sam filtruje wszystkie zagnieżdżenia array.

To tylko tak mogę to robić mam tablicę utworów a każdy utwór to mulit wymiarowa tablica.Każdy utwór pojedynczo wpada do metody createDataByType()
http://sandbox.onlinephpfunctions.com/code/fad83bd48fc677e7571b57202f6ad28b103baf58

[smalczyk1]

@Jare_K: Ewentualnie teraz mi przyszedł pomysł stworzyć jeden array ze wszystkimi polami które występują w tych 3 tabelach.Przypisać je odpowiednio i potem przy insercie pobierać po kolumnach je wybierać które insertować

http://sandbox.onlinephpfunctions.com/code/b55d64628625e7382e5bd882024504b50fbe0a94

[Jare_K]

@smalczyk1: no to już jest o wiele bardziej eleganckie rozwiązanie. Dobry kierunek ( ͡° ͜ʖ ͡°)
Pętle z warunkiem w funkcji createDataByType można lepiej ogarnąć jakimś arrayfilter lub arraywalk.
To:

if($i==0){
 $values="('" . implode("', '", $value) . "')";
}else{
 $values=$values.",('" . implode("', '", $value) . "')";
}

możesz zastąpić:

przed pętlą:
$values = [];
w pętli:
$values[] = "('" . implode("', '", $value) . "')";