PHP weg halen van de comma in een meerdere if loops.

[jandeman023]

code:

<?php
for ($i = 0; $i < $count_contacts; $i++) {
                                if ($contacts1 == "1") {
                                    echo "Main, ";
                                }
                                if ($contacts2 == "1") {
                                    echo "Financial, ";
                                }
                                if ($contacts3 == "1") {
                                    echo "Accounts, ";
                                }
                                if ($contacts4 == "1") {
                                    echo "Permissions, ";
                                }
                                if ($contacts5 == "1") {
                                    echo "Always CC, ";
                                }
                                if ($contacts6 == "1") {
                                    echo "Backup alerts, ";
                                }
}
                                ?>

bovenstaand mijn code.

Het kan zijn dat contacts# een 1 of een 0 is. Als contacts# 1 is dan heeft hij de naam weer. als contacts# 0 is dan heeft hij het niet weer.
Hij maakt er een lijst van die ik graag zou willen onderschijden met comma's. zoals bijvoorbeeld:

Main, Accounts, Permissions

Het is alleen nu wel zo dat hij dit echo't:
Main, Accounts, Permissions,

Dus ik zou graag die laatste comma weg willen. Hoe doe ik dit?

rtrim() werkt niet omdat het geen 1 string is..

[Merethil]

jandeman023 schreef op donderdag 1 juni 2017 @ 11:16:

code:

<?php
for ($i = 0; $i < $count_contacts; $i++) {
                                if ($contacts1 == "1") {
                                    echo "Main, ";
                                }
                                if ($contacts2 == "1") {
                                    echo "Financial, ";
                                }
                                if ($contacts3 == "1") {
                                    echo "Accounts, ";
                                }
                                if ($contacts4 == "1") {
                                    echo "Permissions, ";
                                }
                                if ($contacts5 == "1") {
                                    echo "Always CC, ";
                                }
                                if ($contacts6 == "1") {
                                    echo "Backup alerts, ";
                                }
}
                                ?>

bovenstaand mijn code.

Het kan zijn dat contacts# een 1 of een 0 is. Als contacts# 1 is dan heeft hij de naam weer. als contacts# 0 is dan heeft hij het niet weer.
Hij maakt er een lijst van die ik graag zou willen onderschijden met comma's. zoals bijvoorbeeld:

Main, Accounts, Permissions

Het is alleen nu wel zo dat hij dit echo't:
Main, Accounts, Permissions,

Dus ik zou graag die laatste comma weg willen. Hoe doe ik dit?

rtrim() werkt niet omdat het geen 1 string is..

Alleen de komma toevoegen aan het einde van de String als het niet de laatste in de loop is?
Ik zou trouwens er gewoon één String van maken, en helemaal aan het einde éénmaal echo aanroepen als je het al gewoonweg wil echo-en.

[ Voor 3% gewijzigd door Merethil op 01-06-2017 11:19 ]

[LordSinclair]

Alles wat je echo't wordt al weggeschreven als output. Je zal de gewenste output dus even moeten bewaren in een variabele en deze bewerken voordat je deze echo't

[jandeman023]

LordSinclair schreef op donderdag 1 juni 2017 @ 11:24:
Alles wat je echo't wordt al weggeschreven als output. Je zal de gewenste output dus even moeten bewaren in een variabele en deze bewerken voordat je deze echo't

Hoe stop ik dan al die variaberen in 1 string zonder dat hij gingen dubbel er in gaat zetten?

[Merethil]

jandeman023 schreef op donderdag 1 juni 2017 @ 11:38:
[...]


Hoe stop ik dan al die variaberen in 1 string zonder dat hij gingen dubbel er in gaat zetten?

Doet hij nu al dubbele tonen dan? Want zo niet, dan kan je in je IF-statements gewoon de String concatenaten en aan het einde (na de loop) echo-en.

[phpn3rd]

Onderstaande lijkt me een oplossing?

PHP:

<?php
$str = "";
for ($i = 0; $i < $count_contacts; $i++) {
    if ($i > 0)
        $str .= ", ";
    if ($contacts1 == "1") {
        $str .= "Main";
    }
    if ($contacts2 == "1") {
        $str .= "Financial";
    }
    if ($contacts3 == "1") {
        $str .= "Accounts";
    }
    if ($contacts4 == "1") {
        $str .= "Permissions";
    }
    if ($contacts5 == "1") {
        $str .= "Always CC";
    }
    if ($contacts6 == "1") {
        $str .= "Backup alerts";
    }
}
echo $str;
?>

[jandeman023]

Merethil schreef op donderdag 1 juni 2017 @ 11:40:
[...]


Doet hij nu al dubbele tonen dan? Want zo niet, dan kan je in je IF-statements gewoon de String concatenaten en aan het einde (na de loop) echo-en.

zoals dit:

code:

<?php
                                if ($contacts_lct_main_contact == "1") {
                                    $contacts_array_1 = "Main, ";
                                }
                                if ($contacts_lct_financial_contact == "1") {
                                    $contacts_array_2 = "Financial, ";
                                }
                                if ($contacts_lct_account_request == "1") {
                                    $contacts_array_3 = "Accounts, ";
                                }
                                if ($contacts_lct_permissions_request == "1") {
                                    $contacts_array_4 = "Permissions, ";
                                }
                                if ($contacts_lct_always_cc_incidents == "1") {
                                    $contacts_array_5 = "Always CC, ";
                                }
                                if ($contacts_lct_backup_alerts == "1") {
                                    $contacts_array_6 = "Backup alerts, ";
                                }
                                $contacts_array_compleate = $contacts_array_1 . $contacts_array_2 . $contacts_array_3 . $contacts_array_4 . $contacts_array_5 . $contacts_array_6;
                                echo $contacts_array_compleate;
                                ?>

werkt alleen niet...

Dit is de output van mijn eerste code:
person1: Main, Always CC,
person2: Financial,
person3: Main, Accounts, Permissions,
person4: Always CC,

Dit word dan de output als ik de bovenstaande code gebruik:
person1: Main, Always CC,
person2: Main, Financial, Always CC,
person3: Main, Financial, Accounts, Permissions, Always CC,
person4: Main, Financial, Accounts, Permissions, Always CC,

Dus hij neemt het bovenstaande mee... en dat hord niet.

[orf]

jandeman023 schreef op donderdag 1 juni 2017 @ 11:16:

code:

<?php
for ($i = 0; $i < $count_contacts; $i++) {
                                if ($contacts1 == "1") {
                                    echo "Main, ";
                                }
                                if ($contacts2 == "1") {
                                    echo "Financial, ";
                                }
                                if ($contacts3 == "1") {
                                    echo "Accounts, ";
                                }
                                if ($contacts4 == "1") {
                                    echo "Permissions, ";
                                }
                                if ($contacts5 == "1") {
                                    echo "Always CC, ";
                                }
                                if ($contacts6 == "1") {
                                    echo "Backup alerts, ";
                                }
}
                                ?>

bovenstaand mijn code.

Het kan zijn dat contacts# een 1 of een 0 is. Als contacts# 1 is dan heeft hij de naam weer. als contacts# 0 is dan heeft hij het niet weer.
Hij maakt er een lijst van die ik graag zou willen onderschijden met comma's. zoals bijvoorbeeld:

Main, Accounts, Permissions

Het is alleen nu wel zo dat hij dit echo't:
Main, Accounts, Permissions,

Dus ik zou graag die laatste comma weg willen. Hoe doe ik dit?

rtrim() werkt niet omdat het geen 1 string is..

Ik ben niet zo van de complete voorkauwen, maar in dit geval is dit wel het duidelijkst:

PHP:

<?php

$messages = [];

for ($i = 0; $i < $count_contacts; $i++) {
    if ($contacts1 == "1") {
        $messages[] = "Main";
    }
    if ($contacts2 == "1") {
        $messages[] = "Financial";
    }
    if ($contacts3 == "1") {
        $messages[] = "Accounts";
    }
    if ($contacts4 == "1") {
        $messages[] = "Permissions";
    }
    if ($contacts5 == "1") {
        $messages[] = "Always CC";
    }
    if ($contacts6 == "1") {
        $messages[] = "Backup alerts";
    }
}

echo implode(', ', $messages);

Makkelijker is om een array te vullen en die in één keer te printen met komma's ertussen.

Je kunt implode() opzoeken in de documentatie om te zien wat dit doet.

[Merethil]

jandeman023 schreef op donderdag 1 juni 2017 @ 11:44:
[...]


zoals dit:

code:

<?php
                                if ($contacts_lct_main_contact == "1") {
                                    $contacts_array_1 = "Main, ";
                                }
                                if ($contacts_lct_financial_contact == "1") {
                                    $contacts_array_2 = "Financial, ";
                                }
                                if ($contacts_lct_account_request == "1") {
                                    $contacts_array_3 = "Accounts, ";
                                }
                                if ($contacts_lct_permissions_request == "1") {
                                    $contacts_array_4 = "Permissions, ";
                                }
                                if ($contacts_lct_always_cc_incidents == "1") {
                                    $contacts_array_5 = "Always CC, ";
                                }
                                if ($contacts_lct_backup_alerts == "1") {
                                    $contacts_array_6 = "Backup alerts, ";
                                }
                                $contacts_array_compleate = $contacts_array_1 . $contacts_array_2 . $contacts_array_3 . $contacts_array_4 . $contacts_array_5 . $contacts_array_6;
                                echo $contacts_array_compleate;
                                ?>

werkt alleen niet...

Dit is de output van mijn eerste code:
person1: Main, Always CC,
person2: Financial,
person3: Main, Accounts, Permissions,
person4: Always CC,

Dit word dan de output als ik de bovenstaande code gebruik:
person1: Main, Always CC,
person2: Main, Financial, Always CC,
person3: Main, Financial, Accounts, Permissions, Always CC,
person4: Main, Financial, Accounts, Permissions, Always CC,

Dus hij neemt het bovenstaande mee... en dat hord niet.

Dit is ook totaal iets anders dan je eerst liet zien... Kijk naar de oplossing van @orf, als je die gebruikt zit 't wel goed. Echter, als je iets laat zien in je topic start en dat komt totaal niet overeen met wat je nu eigenlijk echt aan het doen bent is het best lastig helpen he!?

[jandeman023]

Merethil schreef op donderdag 1 juni 2017 @ 11:47:
[...]


Dit is ook totaal iets anders dan je eerst liet zien... Kijk naar de oplossing van @orf, als je die gebruikt zit 't wel goed. Echter, als je iets laat zien in je topic start en dat komt totaal niet overeen met wat je nu eigenlijk echt aan het doen bent is het best lastig helpen he!?

@Merethil excuses. ja inderdaad niet zo handig van mij. sorry daarvoor.

de output van @orf :

person1: Main, Always CC,
person2: Main, Financial, Always CC,
person3: Main, Financial, Accounts, Permissions, Always CC,
person4: Main, Financial, Accounts, Permissions, Always CC,

Zoals ik het dan graag zou willen hebben:
person1: Main, Always CC
person2: Financial
person3: Main, Accounts, Permissions
person4: Always CC

[ Voor 32% gewijzigd door jandeman023 op 01-06-2017 11:50 ]

[Merethil]

jandeman023 schreef op donderdag 1 juni 2017 @ 11:49:
@Merethil excuses. ja inderdaad niet zo handig van mij. sorry daarvoor.

de output van @orf :

person1: Main, Always CC,
person2: Main, Financial, Always CC,
person3: Main, Financial, Accounts, Permissions, Always CC,
person4: Main, Financial, Accounts, Permissions, Always CC,

Zoals ik het dan graag zou willen hebben:
person1: Main, Always CC
person2: Financial
person3: Main, Accounts, Permissions
person4: Always CC

Laat eens zien waar je al je contacts vandaan tovert dan, want blijkbaar doe je meer dan alleen dit loopje laat zien. De oplossing van @orf zou namelijk prima moeten werken als alles wat je doet staat in je post.

[LordSinclair]

Die "PersonX:" komt uit het niets, niks in je code, die je geeft, genereert dat.. Dus dan wordt het glazenbol kijken om te kijken wat het probleem is.

[jandeman023]

LordSinclair schreef op donderdag 1 juni 2017 @ 11:51:
Die "PersonX:" komt uit het niets, niks in je code, die je geeft, genereert dat.. Dus dan wordt het glazenbol kijken om te kijken wat het probleem is.

Uit een json requst:

code:

[
  {
    "LocationContacts": [
      {
        "gbr_id": 001,
        "gbr_Naam": "person1",
        "lct_main_contact": 1,
        "lct_financial_contact": 0,
        "lct_account_request": 0,
        "lct_permissions_request": 0,
        "lct_always_cc_incidents": 1,
        "lct_backup_alerts": 0
      },
      {
        "gbr_id": 002,
        "gbr_Naam": "person2",
        "lct_main_contact": 0,
        "lct_financial_contact": 1,
        "lct_account_request": 0,
        "lct_permissions_request": 0,
        "lct_always_cc_incidents": 0,
        "lct_backup_alerts": 0
      },
      {
        "gbr_id": 003,
        "gbr_Naam": "person3",
        "lct_main_contact": 1,
        "lct_financial_contact": 0,
        "lct_account_request": 1,
        "lct_permissions_request": 1,
        "lct_always_cc_incidents": 0,
        "lct_backup_alerts": 0
      },
      {
        "gbr_id": 004,
        "gbr_Naam": "person4",
        "lct_main_contact": 0,
        "lct_financial_contact": 0,
        "lct_account_request": 0,
        "lct_permissions_request": 0,
        "lct_always_cc_incidents": 1,
        "lct_backup_alerts": 0
      }
    ]
  }
]

code:

$contacts1 = $json_data[0]['LocationContacts'][$i]['contacts1'];
$contacts2 = $json_data[0]['LocationContacts'][$i]['contacts2'];
$contacts3 = $json_data[0]['LocationContacts'][$i]['contacts3'];
$contacts4 = $json_data[0]['LocationContacts'][$i]['contacts4'];
$contacts5 = $json_data[0]['LocationContacts'][$i]['contacts5'];
$contacts6 = $json_data[0]['LocationContacts'][$i]['contacts6'];

[Harrie_]

Ik zou überhaupt alles in een array zetten, dus $contacts[1], $contacts[2] etc. dan kun je ook gelijk die array tellen en daar overheen loopen en vervolgens imploden (zoals hierboven is genoemd)... Maar goed in je huidige code zou je zoiets kunnen doen:

code:

<?php
for ($i = 0; $i < $count_contacts; $i++) {
    // $output maken (leeg)
    $output = ""
    if ($contacts1 == "1") {
        // toevoegen aan $output
        $output .= "Main, ";
    }
    if ($contacts2 == "1") {
        // toevoegen aan $output
        $output .= "Financial, ";
    }
    if ($contacts3 == "1") {
        // toevoegen aan $output
        $output .= "Accounts, ";
    }
    if ($contacts4 == "1") {
        // toevoegen aan $output
        $output .= "Permissions, ";
    }
    if ($contacts5 == "1") {
        // toevoegen aan $output
        $output .= "Always CC, ";
    }
    if ($contacts6 == "1") {
        // toevoegen aan $output
        $output .= "Backup alerts, ";
    }
    // na het toevoegen van bovenstaande strings checken
    // of er wel iets in $output terecht is gekomen,
    // zoja: ', ' rechts weghalen
    if (strlen($output) >= 1) {
         $output = substr($output, 0, -2);
    }
}
echo $output;
?>

Succes!

[ Voor 36% gewijzigd door Harrie_ op 02-06-2017 08:40 . Reden: toch een klein foutje in de code... ]

[Rushleader]

Probeer dit eens

PHP:

<?php

$keys = array(
    'lct_main_contact' => 'Main', 
    'lct_financial_contact' => 'Financial', 
    'lct_account_request' => 'Accounts',
    'lct_permissions_request' => 'Permissions',
    'lct_always_cc_incidents' => 'Always CC', 
    'lct_backup_alerts' => 'Backup alerts'
);

for ($i = 0; $i < $count_contacts; $i++) {
    $message = array();
    $contact = $json_data[0]['LocationContacts'][$i];

    foreach ($keys as $key => $label) {
        if ($contact[$key] == 1) {
            $message[] = $label;
        }
    }

    echo $contact['gbr_Naam'] . ': ' implode(',', $message) . PHP_EOL;
}

Zo kijk je per persoon wat de daadwerkelijke acties zijn ipv globaal er doorheen gaan

Edit: Als eens gekeken om de JSON objecten naar een stdClass te muteren en daar methods op te callen die je nodig hebt ? Lijkt me efficiënter dan constant door alles heen te lussen

[ Voor 10% gewijzigd door Rushleader op 01-06-2017 12:21 ]

[Harrie_]

[b]Rushleader schreef op donderdag 1 juni 2017 @ 12:17:
[...]
Edit: Als eens gekeken om de JSON objecten naar een stdClass te muteren en daar methods op te callen die je nodig hebt ? Lijkt me efficiënter dan constant door alles heen te lussen

No offense, maar dit is zo'n typisch gedrag altijd op Tweakers.
Kijk eens naar de code van TS en vervolgens naar je eigen opmerking...

[Rushleader]

McLatey schreef op donderdag 1 juni 2017 @ 12:27:
[...]


No offense, maar dit is zo'n typisch gedrag altijd op Tweakers.
Kijk eens naar de code van TS en vervolgens naar je eigen opmerking...

offtopic:
Sorry, maar ik vind dit ook een typische reactie. Waarom zou een verbetering op code (en mogelijk algemene programmeer kennis) nou een slecht iets zijn ?

Puur voor de initiële vraag zou jouw antwoord al werken (als hij niet direct zou outputten).
Anders zou je nog de output via ob kunnen afvangen en daarna aanpassen

[ Voor 0% gewijzigd door Rushleader op 01-06-2017 12:37 . Reden: Typos xD ]

[jandeman023]

Rushleader schreef op donderdag 1 juni 2017 @ 12:17:
Probeer dit eens

PHP:

<?php

$keys = array(
    'lct_main_contact' => 'Main', 
    'lct_financial_contact' => 'Financial', 
    'lct_account_request' => 'Accounts',
    'lct_permissions_request' => 'Permissions',
    'lct_always_cc_incidents' => 'Always CC', 
    'lct_backup_alerts' => 'Backup alerts'
);

for ($i = 0; $i < $count_contacts; $i++) {
    $message = array();
    $contact = $json_data[0]['LocationContacts'][$i];

    foreach ($keys as $key => $label) {
        if ($contact[$key] == 1) {
            $message[] = $label;
        }
    }

    echo $contact['gbr_Naam'] . ': ' implode(',', $message) . PHP_EOL;
}

Zo kijk je per persoon wat de daadwerkelijke acties zijn ipv globaal er doorheen gaan

Edit: Als eens gekeken om de JSON objecten naar een stdClass te muteren en daar methods op te callen die je nodig hebt ? Lijkt me efficiënter dan constant door alles heen te lussen

Eorm bedankt! Heeft mijn probleem verholpen.

Nee ik heb nog nooit gekeken om JSON objecten naar een stdClass te muteren. Ik ga er meteen even naar googlen.
Nogmaals, enorm bedankt!

[Harrie_]

Rushleader schreef op donderdag 1 juni 2017 @ 12:34:
[...]

Sorry, maar ik vind dit ook een typische reactie. Waarom zou een verbetering op code (en mogelijk algemene programmeer kennis) nou een slecht iets zijn ?

Tuurlijk is dat niet slecht! Ik maakte er een opmerking over omdat de complexiteit van hetgeen je noemt niet in verhouding staat tot het kennisniveau van TS (verwacht kennisniveau afgaand op zijn eigen script). Het is een beetje alsof iemand net een staartdeling onder de knie heeft en je dan termen als kwadratische vergelijking en logaritme naar zijn hoofd gaat gooien.

Dus verbetering op code / algemene programmeerkennis is altijd goed, maar het advies mag wel iets meer in verhouding staan tot het huidige niveau. TS heeft kennelijk arrays nog niets een onder de knie en dan vind ik het advies om 'JSON objecten naar een stdClass te muteren en daar methods op te callen' ook niet echt constructief...

En inderdaad... ik had die 'echo $output;' buiten de loop moeten knallen...

[Rushleader]

McLatey schreef op donderdag 1 juni 2017 @ 13:41:
[...]


Tuurlijk is dat niet slecht! Ik maakte er een opmerking over omdat de complexiteit van hetgeen je noemt niet in verhouding staat tot het kennisniveau van TS (verwacht kennisniveau afgaand op zijn eigen script). Het is een beetje alsof iemand net een staartdeling onder de knie heeft en je dan termen als kwadratische vergelijking en logaritme naar zijn hoofd gaat gooien.

Ik vond de complexiteit ervan niet zo heel erg hier aan gezien hij dus in de basis van zijn code JSON structs gebruikt en dit (m.i.) een basis kunde is. Dat is het aan het om de keuze te maken. Maar je hebt wel gelijk dat het misschien nog wat ver is

Dus verbetering op code / algemene programmeerkennis is altijd goed, maar het advies mag wel iets meer in verhouding staan tot het huidige niveau. TS heeft kennelijk arrays nog niets een onder de knie en dan vind ik het advies om 'JSON objecten naar een stdClass te muteren en daar methods op te callen' ook niet echt constructief...

En inderdaad... ik had die 'echo $output;' buiten de loop moeten knallen...

Hehe alles is op te lossen Daarom de ob optie

[Flipull]

Harrie_ schreef op donderdag 1 juni 2017 @ 13:41:
[...]
Tuurlijk is dat niet slecht! Ik maakte er een opmerking over omdat de complexiteit van hetgeen je noemt niet in verhouding staat tot het kennisniveau van TS

Ik denk wel eens bij sommige topicstarters, dat de complexiteit ergens anders zit. Het is alsof het modelleren van gedrag (van logica en informatie) niet gedaan word, maar meteen in code gewerkt word.

Uit praktijk heb ik ontdekt, dat UML heel erg handig is voor modelleren. En dat ik wat vaker op moest letten, tijdens de school-les Dus TS, schrijf de werking uit van wat je wilt, en breek het op in elementaire handelingen. En laat die mee-toetsen met de code die fouten geeft.

Zo is helpen ook een stuk makkelijker, sneller, en productiever voor de TS (ongeacht welk topic).