1 |
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 | $nummer = ""; if ( stripos($melding, "UTRECHT") !== false ) { $nummer .= "0611111111"; } if ( stripos($melding, "ROTTERDAM") !== false ) { $nummer .= "0622222222"; } if (!empty($nummer)) { $nummers = implode(", ", str_split($nummer, 10)); $onderwerp = "SMS ONDERWERP"; include("sendsms.php"); } ?> |
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 | $nummer = ""; if (stripos($body, "0701234") !== false) { $onderwerp = "ONDERWERP EEN"; if ( stripos($melding, "AMSTERDAM") !== false ) { $nummer .= "0611111111"; } } else { $onderwerp = "ONDERWERP TWEE"; if ( stripos($melding, "ROTTERDAM") !== false ) { $nummer .= "0622222222"; } if ( stripos($melding, "UTRECHT") !== false ) { $nummer .= "0633333333"; } } if (!empty($nummer)) { $nummers = implode(", ", str_split($nummer, 10)); include("sendsms.php"); } ?> |
1 2 3 4 5 6 7 8 | $nummers = array(); #array_push() $nummers[] = "1234"; $nummers = join(", ", $nummers); ?> |
Ik tel geen extra regel Maar als je zo graag weinig regels wilt hebben, waarom zet je dan niet alles op één regel? En verdiep je eens in de ternary operator, daar kun je echt veel ruimte mee besparen.quote:Op vrijdag 17 juli 2009 09:20 schreef F4T4L_3RR0R het volgende:
Dat zijn al 4 regels code tegenover 1 regel in mijn script.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 | $nummers = array(); if (stripos($body, "0701234") !== false) { $onderwerp = "ONDERWERP EEN"; if ( stripos($melding, "AMSTERDAM") !== false ) { $nummers[] = "0611111111"; } } else { $onderwerp = "ONDERWERP TWEE"; if ( stripos($melding, "ROTTERDAM") !== false ) { $nummers[] = "0622222222"; } if ( stripos($melding, "UTRECHT") !== false ) { $nummers[] = "0633333333"; } } if (!empty($nummer)) { $nummers = join(", ", $nummers); include("sendsms.php"); } ?> |
Waarom vervang je dan niet het versturen van de SMS met een platte debug tekst?quote:Op donderdag 16 juli 2009 22:38 schreef F4T4L_3RR0R het volgende:
[..]
Dat require/require_once wil ik niet teveel mee testen, kostte me al stuk of 50 smsjes vanavond... (ja ik kan ook een testscript maken wat aangeroepen moet worden, maar dan gaat daar weer alle tijd in zitten).
Owja? Die array_push ipv strings concatten om vervolgens geen split te hoeven doen op elk 10 karakter (wat dus problemen gaat geven als je plots ook met buitenlandse nummers zou gaan werken enzo). Dat heb ik nu niet, aangezien ik gewoon direct een array gebruik en daar de nummers inprop.quote:Op vrijdag 17 juli 2009 09:20 schreef F4T4L_3RR0R het volgende:
Dat zijn al 4 regels code tegenover 1 regel in mijn script.
quote:Op vrijdag 17 juli 2009 12:11 schreef slacker_nl het volgende:
[..]
Owja? Die array_push ipv strings concatten om vervolgens geen split te hoeven doen op elk 10 karakter (wat dus problemen gaat geven als je plots ook met buitenlandse nummers zou gaan werken enzo). Dat heb ik nu niet, aangezien ik gewoon direct een array gebruik en daar de nummers inprop.
@swets:
if (!empty($nummers)) ipv $nummer
quote:Op vrijdag 17 juli 2009 11:55 schreef Swetsenegger het volgende:
[..]
nou ja, die 's' vind ie zelf wel.... toch.
Die staat niet standaard geinstalleerd bij de meeste shared hostersquote:
1 2 3 4 5 6 | if (!empty($nummer)) { $nummers = join(", ", $nummer); include("sendsms.php"); } ?> |
1 2 3 4 5 6 | $nummer = "0611111111"; $nummer = "0622222222"; $nummer = "0633333333"; $nummer = "0644444444"; ?> |
1 |
quote:Op donderdag 16 juli 2009 22:26 schreef GlowMouse het volgende:
Waarom probeer je het niet gewoon uit?
1 |
1 |
Ik zou die laatste gebruiken, omdat het anders wat onoverzichtelijk wordt wat '$nummer' precies is.quote:Op vrijdag 17 juli 2009 19:10 schreef F4T4L_3RR0R het volgende:
Jaaaaaa, werkt. (veel beter ook!)
Ik heb trouwens "implode" gebruikt in plaats van "join". Dit omdat ik meerdere keren "implode" in het script gebruik en het anders voor verwarring kan zorgen.
Maak het trouwens uit als ik:
[ code verwijderd ]
Of
[ code verwijderd ]
Gebruik? Is het ene veiliger of beter dan het andere? Het werkt in ieder geval allebei.
Forum Opties | |
---|---|
Forumhop: | |
Hop naar: |