Forum PHP.pl > Operacja na tablach

Pomoc - Szukaj - Użytkownicy - Kalendarz

pela222

10.07.2020, 09:43:30

Tutaj moja tabela wyjściowa.

[PHP] pobierz, plaintext 
<?php
$policies = 
  array (
    'TerLifIns' => 
    array (
      0 => 
      array (
        'company' => 'NIA',
        'type' => 'Term Life Insurance',
        'no' => '67845678',
        'issueDate' => '2016-09-10',
        'used' => 'yes',
        'group' => 'Group',
        'owner' => 'OWNER',
        'premium' => '55',
        'paymentFrequency' => 'miesięcznie',
        'coverBasis' => 'Dual Life',
        'lifeInsured' => 'Chester Tester, Mary Testerowa',
        'indexation' => 'Yes',
        'term' => '15',
        'seriousIllness' => 'Standalone',
        'conversion' => 'Yes',
        'lifeCoverMark' => 170,
        'seriousIllnessMark' => 70,
        'hospitalCoverMark' => 161,
        'accidentCoverMark' => 201,
        'surgeryMark' => 1,
        'fractureMark' => 1,
        'lifecoverBasisMark' => 0,
        'primary' => 
        array (
          'lifeCover' => '85,000',
          'seriousIllness' => '35,000',
          'cancerCover' => '0',
          'hospitalCover' => '80',
          'accidentCover' => '100',
          'fracture' => 'Yes',
          'surgery' => 'Yes',
        ),
        'secondary' => 
        array (
          'lifeCover' => '85,001',
          'seriousIllness' => '35,001',
          'cancerCover' => '0',
          'hospitalCover' => '81',
          'accidentCover' => '101',
          'fracture' => 'Yes',
          'surgery' => 'No',
        ),
        'DOC' => 
        array (
          'sictypedis' => 'Opis sictypedis dla NIA.',
          'lifedis' => 'Opis life dla NIA.',
          'sicdis' => 'Opis sic dla NIA.',
          'hospitaldis' => 'Opis hospital dla NIA.',
          'accidentdis' => 'Opis acc dla NIA.',
          'surgerydis' => 'Opis surg dla NIA.',
          'fracturedis' => 'Opis fra dla NIA.',
          'indexationdis' => 'Opis index dla NIA.',
          'conversiondis' => 'Opis conver dla NIA.',
          'companydis' => 
          array (
            0 => 'Opis 0 dla NIA.',
            1 => 'Opis 1 dla NIA.',
            2 => 'Opis 2 dla NIA.',
          ),
          'companyaddress' => 
          array (
            'name' => 'NIA ',
            'street' => 'street NIA',
            'area' => 'area NIA',
            'county' => 'county NIA',
            'zipcode' => 'zip NIA',
          ),
          'otherdefault' => 'x',
        ),
      ),
      1 => 
      array (
        'company' => 'IL',
        'type' => 'Term Life Insurance',
        'no' => '12345678',
        'issueDate' => '2016-09-10',
        'used' => 'yes',
        'group' => 'Group',
        'owner' => 'OWNER',
        'premium' => '25',
        'paymentFrequency' => 'miesięcznie',
        'coverBasis' => 'Dual Life',
        'lifeInsured' => 'Chester Tester, Mary Testerowa',
        'indexation' => 'Yes',
        'term' => '23',
        'seriousIllness' => 'Accelerated',
        'conversion' => 'Yes',
        'lifeCoverMark' => 400,
        'seriousIllnessMark' => 200,
        'hospitalCoverMark' => 201,
        'accidentCoverMark' => 241,
        'surgeryMark' => 0,
        'fractureMark' => 0,
        'lifecoverBasisMark' => 0,
        'primary' => 
        array (
          'lifeCover' => '200,000',
          'seriousIllness' => '100,000',
          'cancerCover' => '0',
          'hospitalCover' => '100',
          'accidentCover' => '120',
          'fracture' => '',
          'surgery' => '',
        ),
        'secondary' => 
        array (
          'lifeCover' => '200,001',
          'seriousIllness' => '100,001',
          'cancerCover' => '0',
          'hospitalCover' => '101',
          'accidentCover' => '121',
          'fracture' => '',
          'surgery' => '',
        ),
        'DOC' => 
        array (
          'sictypedis' => 'Opis sictypedis dla IL.',
          'lifedis' => 'Opis life dla IL.',
          'sicdis' => 'Opis sic dla IL.',
          'hospitaldis' => 'Opis hospital dla IL.',
          'accidentdis' => 'Opis acc dla IL.',
          'surgerydis' => 'Opis surg dla IL.',
          'fracturedis' => 'Opis fra dla IL.',
          'indexationdis' => 'Opis index dla IL.',
          'conversiondis' => 'Opis conver dla IL.',
          'companydis' => 
          array (
            0 => 'Opis 0 dla IL.',
            1 => 'Opis 1 dla IL.',
            2 => 'Opis 2 dla IL.',
          ),
          'companyaddress' => 
          array (
            'name' => 'IL ',
            'street' => 'street IL',
            'area' => 'area IL',
            'county' => 'county IL',
            'zipcode' => 'zip IL',
          ),
          'otherdefault' => 'x',
        ),
      ),
    ),
    'MorIns' => 
    array (
    ),
    'WOL' => 
    array (
    ),
    'IncIns' => 
    array (
    ),
    'AccSicAndHos' => 
    array (
    ),
    'MedCas' => 
    array (
    ),
    'PerAcc' => 
    array (
    ),
    'SovIncAssPla' => 
    array (
    )
);
// echo ('<pre>');
// print_r($policies);
// echo ('</pre>');
?>		
[PHP] pobierz, plaintext

Na podstawie tej tabli tworze kolejną jak poniżej.

[PHP] pobierz, plaintext 
function getRecomendationTerLifInsData($policies) {
        $recomend = array();
 
            //listuj wszystkie discription dla wszystkich typów i wszystkich polisy
            foreach ($policies as $type => &$policieType) {
                if($type == 'TerLifIns') {
                    $count = count($policieType);
                    foreach ($policieType as &$policie) {
                    for($n=0; $n<$count; $n++){    
                        if($policie['lifeCoverMark']){$recomend[] = array('Kwota dla primary to €'.$policies['TerLifIns'][$n]['primary']['lifeCover'].', natomiast kwota dla drugiego ubezpieczonego to €'.$policies['TerLifIns'][$n]['secondary']['lifeCover'].'');}
                        if($policie['seriousIllnessMark']){$recomend[] = array('SIC dla pierwszego to €'.$policies['TerLifIns'][$n]['primary']['seriousIllness'].', natomiast suma SIC dla drugiego to €'.$policies['TerLifIns'][$n]['secondary']['seriousIllness'].' na okres '.$policies['TerLifIns'][$n]['term'].' lat.'.$policies['TerLifIns'][$n]['DOC']['sictypedis'].'');}
                        if($policie['hospitalCoverMark']){$recomend[] = array('Szpital dla pierwszego to €'.$policies['TerLifIns'][$n]['primary']['hospitalCover'].' za dzień pobytu dla pierwszego, a dla drugego €'.$policies['TerLifIns'][$n]['secondary']['hospitalCover'].'');}
                        if($policie['accidentCoverMark']){$recomend[] = array('Acc dla pierwszego to €'.$policies['TerLifIns'][$n]['primary']['accidentCover'].' a dla drugiego Acc to €'.$policies['TerLifIns'][$n]['secondary']['accidentCover'].'');}
                        if($policie['surgeryMark']){$recomend[] = array('Sur opis. '.$policies['TerLifIns'][$n]['DOC']['surgerydis'].'');}
                    }
                        // $recomend[] = array('----');
                       // foreach ($policie['DOC'] as $value) {
                           // if(!is_array($value)){
                               // $recomend[] = array($value);   
                           // }
                       // }
                       // foreach ($policie['DOC']['companydis'] as $value) {
                           // $recomend[] = array($value);
                       // }
                       // $recomend[] = array('----'); // spacer for statement of suitability
                    }
                }    
            }
        return $recomend;
    }
 
$output = getRecomendationTerLifInsData($policies);
echo ('<pre>');
print_r($output);
echo ('</pre>');
[PHP] pobierz, plaintext

Problem z jakim nie wiem jak sobie poradzić to niepotrzebne wyświetlenie jednej z pętli.

Nie wiem czy jest to widoczne ale chodzi o niepotrzebne wywołanie tej pętli

[PHP] pobierz, plaintext 
foreach ($policieType as &$policie) {
                    for($n=0; $n<$count; $n++){  
[PHP] pobierz, plaintext

Nie wiem jak to prawidłowo zapisać.
Dziękuję tradycyjnie za wyrozumiałość i pomoc.

nospor

10.07.2020, 09:56:45

Troche nie na temat ale o malo mozg mi oczami nie wyszedl....

Wiele: policies
Jedna:
nie: zaden policzek
tylko: policy

Tomplus

10.07.2020, 09:57:50

Skoro doskonale wiesz co ma trafić do funkcji, to zmiast całej tablicy $policies wrzuć $policies['TerLifIns']

albo:

[PHP] pobierz, plaintext 
$policies = $policies['TerLifIns'];
[PHP] pobierz, plaintext

pela222

10.07.2020, 11:05:19

Cytat(nospor @ 10.07.2020, 09:56:45 )

Troche nie na temat ale o malo mozg mi oczami nie wyszedl....

Wiele: policies
Jedna:
nie: zaden policzek
tylko: policy

Zakładam że żart... no ale nie załapałem...
Fakt że jestem nieogarnięty mamy ustalony

Jak będę w domu to zastanowię się na odpowiedzią @Tomplus i czy rozwiązuje ona mój problem.

Thx

nospor

10.07.2020, 11:10:13

Liczba pojedyncza dla policies to policy a nie jak ty piszesz policie. Takie tam podstawy angielskiego

pela222

10.07.2020, 12:04:01

Cytat(nospor @ 10.07.2020, 11:10:13 )

Liczba pojedyncza dla policies to policy a nie jak ty piszesz policie. Takie tam podstawy angielskiego

Dzięki za merytoryczna podpowiedź, ale pytanie zadałem w wątku php a nie nauka angielskiego w weekend

Chyba że ten błąd wpływa na wykonanie kodu

a to przepraszam wtedy

nospor

10.07.2020, 12:09:17

Wiem o co pytales. Ale jakbys byl dobrym programista to bys nie marudzil za poprawki nawet nie dotyczace twojego problemu

A poprawka przeze mnie zaproponowana jest bardzo wazna. Wplywa korzystnie na oczy czytajacych twoje posty

pela222

10.07.2020, 12:18:11

Cytat(nospor @ 10.07.2020, 12:09:17 )

Wiem o co pytales. Ale jakbys byl dobrym programista to bys nie marudzil za poprawki nawet nie dotyczace twojego problemu

A poprawka przeze mnie zaproponowana jest bardzo wazna. Wplywa korzystnie na oczy czytajacych twoje posty

Ale je wiem, że ty wiesz bo wielokrotnie mi i innym pomogłeś.
Problem w tym, że ja nawet programistą nie jestem, a co dopiero dobrym

.
Bawię się czasem kodem i robię / modyfikuję projektu, który ktoś mi zrobił jakiś czas temu. Ten błąd językowy widziałem już dawno temu, ale ponieważ przewija się w kilku miejscach to "bałem się" ruszać.
Moje wypociny też często wynikają z modyfikacji kodu...świeże spojrzenie kogoś kto tak ja ty czy kolega Tomplus poruszają się w kodzie sprawnie jest bardzo ... bardzo pomocne i nie pisze tego z przekorą, tylko jesteście jedynym miejscem, gdzie mogę zadać pytanie.
Pozdrawiam.
PS. Starczy tego off-topic.

nospor

10.07.2020, 13:54:28

Cytat

ale ponieważ przewija się w kilku miejscach to "bałem się" ruszać.

Zmienne tworzone w konstrukcji FOREACH sa zazwyczaj tylko zmiennymi lokalnymi i nie maja wplywu na reszte kodu. Mozesz wiec zmieniac raczej spokojnie

Tomplus

10.07.2020, 21:24:06

Potwierdzam co mówi @nospor, spokojnie możesz zmieniać nazwy zmiennych, tablic gdy są wewnątrz funkcji czy też metod.

Baa... gdy są za duże funkcje np. jak u Ciebie pętla, w pętli ifami poganiane to warto rozdzielić je na mniejsze funkcje.

Pyton_000

13.07.2020, 09:44:28

Przepraszam, ale nie mogę patrzeć na taki kod

[PHP] pobierz, plaintext 
        function getRecomendationTerLifInsData($policies) {
            if(!\array_key_exists('TerLifIns', $policies)) {
                return [];
            }
            $recommend = [];
 
            foreach ($policies['TerLifIns'] as $policy) {
                if($policy['lifeCoverMark']) {
                    $recommend[] = 'Kwota dla primary to €'.$policy['primary']['lifeCover'].', natomiast kwota dla drugiego ubezpieczonego to €'.$policy['secondary']['lifeCover'];
                }
 
                if($policy['seriousIllnessMark']){
                    $recommend[] = 'SIC dla pierwszego to €'.$policy['primary']['seriousIllness'].', natomiast suma SIC dla drugiego to €'.$policy['secondary']['seriousIllness'].' na okres '.$policy['term'].' lat.'.$policy['DOC']['sictypedis'];
                }
                if($policy['hospitalCoverMark']){
                    $recommend[] = 'Szpital dla pierwszego to €'.$policy['primary']['hospitalCover'].' za dzień pobytu dla pierwszego, a dla drugego €'.$policy['secondary']['hospitalCover'];
                }
                if($policy['accidentCoverMark']){
                    $recommend[] = 'Acc dla pierwszego to €'.$policy['primary']['accidentCover'].' a dla drugiego Acc to €'.$policy['secondary']['accidentCover'];
                }
                if($policy['surgeryMark']){
                    $recommend[] = 'Sur opis. '.$policy['DOC']['surgerydis'];
                }
            }
            return $recommend;
        }
[PHP] pobierz, plaintext

pela222

13.07.2020, 09:53:06

Cytat(Tomplus @ 10.07.2020, 09:57:50 )

Skoro doskonale wiesz co ma trafić do funkcji, to zmiast całej tablicy $policies wrzuć $policies['TerLifIns']

albo:

[PHP] pobierz, plaintext 
$policies = $policies['TerLifIns'];
[PHP] pobierz, plaintext

Cześć, dopiero dzisiaj udało się usiąść do tematu.
Prawda jest taka, że to co wrzuciłem to było i tak jakieś tam uproszczenie, tego co rzeczywiście jest u mnie (uciąłem jeden poziom tablicy w oryginalnym poście nr.1).
Nie zmienia to faktu że jak zrobiłem:

[PHP] pobierz, plaintext 
private function getRecomendationTerLifInsData($documentData['policies']['TerLifIns']) {
        $recomend = array();
                    $count = count($documentData['policies']['TerLifIns']);
...
 
        return $recomend;
    }
[PHP] pobierz, plaintext

IDE pokazuje mi błąd składni: unexpected [ after variable '$documentData'

nospor

13.07.2020, 10:36:07

private function getRecomendationTerLifInsData($documentData['policies']['TerLifIns']) {
W definicji funckji okresla sie nazwe parametru a nie jakas sciezke w tablicy

pela222

13.07.2020, 12:47:40

Cytat(nospor @ 13.07.2020, 10:36:07 )

private function getRecomendationTerLifInsData($documentData['policies']['TerLifIns']) {
W definicji funckji okresla sie nazwe parametru a nie jakas sciezke w tablicy

Ja się cały czas uczę, codziennie się uczę - (cytując pewnego Pana z telewizji

)

[PHP] pobierz, plaintext 
    //Rekomendacja Term Life
    private function getRecomendationTerLifInsData($documentData) {
        $TerLifIns = $documentData['policies']['TerLifIns'];
        $recomend = array();
        $count = count($TerLifIns);
            for($n=0; $n<$count; $n++){
                if($TerLifIns[$n]['coverBasis'] == 'Dual Life' || $TerLifIns[$n]['coverBasis'] == 'Joint Life') {   
                    if($TerLifIns[$n]['lifeCoverMark']){
                        $recomend[] = array('Utraty życia, gdzie suma ubezpieczenia jaką rekomenduje dla pierwszego ubezpieczonego ('.$documentData['primary']['firstName'].' '.$documentData['primary']['lastName'].') to ?'.$TerLifIns[$n]['primary']['lifeCover'].', natomiast suma ubezpieczenia jaką rekomenduje dla drugiego ubezpieczonego ('.$documentData['secondary']['firstName'].' '.$documentData['secondary']['lastName'].') to ?'.$TerLifIns[$n]['secondary']['lifeCover'].' na okres '.$TerLifIns[$n]['term'].' lat.');
                    }
                    if($TerLifIns[$n]['seriousIllnessMark']){
                        $recomend[] = array('Poważnej choroby, gdzie rekomendowana kwota ubezpieczenia dla pierwszego ubezpieczonego ('.$documentData['primary']['firstName'].' '.$documentData['primary']['lastName'].') to ?'.$documentData['policies']['TerLifIns']['0']['primary']['seriousIllness'].', natomiast suma ubezpieczenia jaką rekomenduje dla drugiego ubezpieczonego ('.$documentData['secondary']['firstName'].' '.$documentData['secondary']['lastName'].') to ?'.$documentData['policies']['TerLifIns']['0']['secondary']['seriousIllness'].' na okres '.$documentData['policies']['TerLifIns']['0']['term'].' lat.'.$documentData['policies']['TerLifIns']['0']['DOC']['sictypedis'].'');
                    }
                    if($TerLifIns[$n]['hospitalCoverMark']){
                        $recomend[] = array('Pobytu w szpitalu, kwota ubezpieczenia to ?'.$documentData['policies']['TerLifIns']['0']['primary']['hospitalCover'].' za dzień pobytu dla '.$documentData['primary']['firstName'].' oraz ?'.$documentData['policies']['TerLifIns']['0']['secondary']['hospitalCover'].' za dzień pobytu dla '.$documentData['secondary']['firstName'].'.'.$documentData['policies']['TerLifIns']['0']['DOC']['hospitaldis'].'');
                    }
                    if($TerLifIns[$n]['accidentCoverMark']){
                        $recomend[] = array('Niezdolności do pracy w skutek wypadku, kwota ubezpieczenia to ?'.$documentData['policies']['TerLifIns']['0']['primary']['accidentCover'].' za tydzień niezdolności dla '.$documentData['primary']['firstName'].' oraz ?'.$documentData['policies']['TerLifIns']['0']['secondary']['accidentCover'].' za tydzień niezdolności dla '.$documentData['secondary']['firstName'].'. '.$documentData['policies']['TerLifIns']['0']['DOC']['accidentdis'].'');
                    }
                    if($TerLifIns[$n]['surgeryMark']){
                        $recomend[] = array('Ubezpieczenie zabiegów chirurgicznych dla '.$documentData['primary']['firstName'].' oraz dla '.$documentData['secondary']['firstName'].'. '.$documentData['policies']['TerLifIns']['0']['DOC']['surgerydis'].'');
                    }
 
                    foreach ($TerLifIns[$n]['DOC'] as $value) {
                        if(!is_array($value)){
                            $recomend[] = array($value);   
                        }
                    }
                    foreach ($TerLifIns[$n]['DOC']['companydis'] as $value) {
                        if(!is_array($value)){
                            $recomend[] = array($value);   
                        }
                    }
                    $recomend[] = array('----'); // spacer for statement of suitability
 
                }
                else {
                    if($TerLifIns[$n]['lifeCoverMark']){
                        $recomend[] = array('Utraty życia, gdzie rekomendowana suma ubezpieczenia to ?'.$TerLifIns[$n]['primary']['lifeCover'].' na okres '.$TerLifIns[$n]['term'].' lat.');
                    }
                    if($TerLifIns[$n]['seriousIllnessMark']){
                        $recomend[] = array('Poważnej choroby, gdzie rekomendowana kwota ubezpieczenia to ?'.$TerLifIns[$n]['primary']['seriousIllness'].' na okres '.$TerLifIns[$n]['term'].' lat.');
                    }
                    if($TerLifIns[$n]['hospitalCoverMark']){
                        $recomend[] = array('Pobytu w szpitalu, kwota ubezpieczenia to ?'.$TerLifIns[$n]['primary']['hospitalCover'].' za dzień pobytu.');
                    }
                    if($TerLifIns[$n]['accidentCoverMark']){
                        $recomend[] = array('Niezdolności do pracy w skutek wypadku, kwota ubezpieczenia to ?'.$TerLifIns[$n]['primary']['accidentCover'].' za tydzień niezdolności. '.$TerLifIns[$n]['DOC']['accidentdis'].'');
                    }
 
                    foreach ($TerLifIns[$n]['DOC'] as $value) {
                        if(!is_array($value)){
                            $recomend[] = array($value);   
                        }
                    }
                    foreach ($TerLifIns[$n]['DOC']['companydis'] as $value) {
                        if(!is_array($value)){
                            $recomend[] = array($value);   
                        }
                    }
                    $recomend[] = array('----'); // spacer for statement of suitability
                }
            }
        return $recomend;
    }
[PHP] pobierz, plaintext

Czy to miało by sens? W sensie działa, ale jak macie jakieś uwagi, które pozwolą mi na lepsze zrobienie tego tutaj po to, żeby pójść krok do przodu w mojej nauce, to będę wdzięczny.

PS. Czy jest taka forma zadawania pytań na forum, że zakładam wątek dla projektu, w którym mógłbym Was prosić o podpowiedzi. Czy po prostu przy następnym pytaniu / problemie powinienem zakładać nowy wątek? Tak jak pisałem wcześniej. Mam napisaną API które używam w mojej pracy. Projekt używam sam i sam go rozwijam / modyfikuję. Przy okazji uczę się / staram się uczyć programowania php na nim...Pewnie nie najlepszy sposób bo duże braki podstaw powodują że zadaje frustrujące pytania. Z drugiej strony mam tego świadomość i jak jest czas to wracam / czytam podstawy. Dziękuję za informację.

Tomplus

14.07.2020, 22:04:51

Skoro codziennie się uczysz, to skorzystaj z rad:
https://devenv.pl/wp-content/uploads/2018/1...4%87-DEVENV.pdf

Cytat

15) Kod powinien być samo dokumentujący się – rób małe funkcje skupione na jednym zadaniu

Możesz mi wyjaśnić po co tworzysz tablicę $recommend[] a następnie jeszcze do niej dodajesz tablice?
i wygląda to tak:

[PHP] pobierz, plaintext 
$recomment = [
  0 => ['string'],
  1 => ['string'],
  2 => ['string']
];
[PHP] pobierz, plaintext

Nie lepiej równiez używać sprintf() ?

[PHP] pobierz, plaintext 
$recomend[] = sprintf(
       'Utraty życia, gdzie suma ubezpieczenia jaką rekomenduje dla pierwszego ubezpieczonego (%s %s) to ?%s, natomiast suma ubezpieczenia jaką rekomenduje dla drugiego ubezpieczonego (%s %s) to ?%s na okres %d lat.',
       $documentData['primary']['firstName'],
       $documentData['primary']['lastName'],
       $TerLifIns[$n]['primary']['lifeCover'],
       $documentData['secondary']['firstName'],
       $documentData['secondary']['lastName'],
       $TerLifIns[$n]['secondary']['lifeCover'],
       $TerLifIns[$n]['term']
);
[PHP] pobierz, plaintext

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.