abonnement bol.com Unibet Coolblue
  vrijdag 17 juli 2009 @ 04:51:39 #176
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71054826
Als de te includen file $nummers (ja met s) wilt lezen, moet het zeker als volgt?

1$nummers = implode(", ", str_split($nummer, 10));


Wat kunnen de gevolgen zijn van een .= als er niets boven zit?
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 05:32:46 #177
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71054986
Okay, heb nu dit (wel de nummers en zoektermen gefingeerd):

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<?php
    $nummer 
"";

if (
    
stripos($melding"UTRECHT") !== false
)
{
    
$nummer .= "0611111111";
}

if (
    
stripos($melding"ROTTERDAM") !== false
)
{
    
$nummer .= "0622222222";
}

if (!empty(
$nummer)) {

    
$nummers implode(", "str_split($nummer10));
    
$onderwerp "SMS ONDERWERP";
    include(
"sendsms.php");
}
?>


Volgens mij kan het geen kwaad dat de lege $nummer buiten de IF's staat. Hij staat wel achter een ELSE (niet te zien op deze code). Heb 'm als test gevuld en toen stond hij mooi in de array!
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 06:19:15 #178
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71055048
Mag ik de include trouwens een IF terugzetten en kan de included sendsms.php dan het $nummer, $onderwerp etc. een paar IF's dieper ophalen?

Ik weet dat het andersom wel mag, dat een dieper gelegen IF een $ uit een hoger gelegen IF kan ophalen, maar ik doe het nu net andersom.

Of sla ik nu op door? Heb het wel getest en het werkt, maar weet niet of het officieel mag.

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
<?php
        $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($nummer10));
            include(
"sendsms.php");
        }
?>


[ Bericht 0% gewijzigd door F4T4L_3RR0R op 17-07-2009 07:11:01 ]
Een ballonvaarder die door de mand valt is nooit grappig...
pi_71055984
Als het werkt, dan werkt het

Heeft niks met mogen te maken. Of het netjes / handig / whatever is is een tweede.
Maar dat lijkt me voor jou op dit moment nog niet heel erg aan de orde, zorg eerst maar eens dat het script het gewoon doet
  vrijdag 17 juli 2009 @ 09:05:27 #180
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71056268
Het script doet het perfect.
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 09:10:44 #181
187069 slacker_nl
Sicko pur sang
pi_71056398
Ik snap dat $nummers .= "1234"; niet echt.

Waarom niet gewoon:

1
2
3
4
5
6
7
8
<?php
$nummers 
= array();

#array_push()
$nummers[] = "1234";

$nummers join(", "$nummers);
?>
In theory there is no difference between theory and practice. In practice there is.
  vrijdag 17 juli 2009 @ 09:20:08 #182
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71056587
Dat zijn al 4 regels code tegenover 1 regel in mijn script.
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 11:13:27 #183
75592 GlowMouse
l'état, c'est moi
pi_71059765
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.
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.
eee7a201261dfdad9fdfe74277d27e68890cf0a220f41425870f2ca26e0521b0
  FOK!-Schrikkelbaas vrijdag 17 juli 2009 @ 11:38:52 #184
1972 Swetsenegger
Egocentrische Narcist
pi_71060633
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
<?php
        $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");
        }
?>

Waaruit wel weer blijkt dat er vele wegen naar Rome leiden.
pi_71060704
je laatste if klopt niet meer
pi_71060849
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).
Waarom vervang je dan niet het versturen van de SMS met een platte debug tekst?
'De sms met tekst "bladiebladiebla" werd verstuurd naar de volgende nummers: ....'
  FOK!-Schrikkelbaas vrijdag 17 juli 2009 @ 11:55:11 #187
1972 Swetsenegger
Egocentrische Narcist
pi_71061228
quote:
Op vrijdag 17 juli 2009 11:40 schreef Xcalibur het volgende:
je laatste if klopt niet meer
nou ja, die 's' vind ie zelf wel.... toch.
  vrijdag 17 juli 2009 @ 12:11:22 #188
187069 slacker_nl
Sicko pur sang
pi_71061822
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.
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
In theory there is no difference between theory and practice. In practice there is.
  FOK!-Schrikkelbaas vrijdag 17 juli 2009 @ 12:40:06 #189
1972 Swetsenegger
Egocentrische Narcist
pi_71062777
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.
  vrijdag 17 juli 2009 @ 13:05:47 #190
187069 slacker_nl
Sicko pur sang
pi_71063749
Ja, ben traag vandaag, zag je code en deed posten, had de replies niet gezien.
In theory there is no difference between theory and practice. In practice there is.
  FOK!-Schrikkelbaas vrijdag 17 juli 2009 @ 14:23:23 #191
1972 Swetsenegger
Egocentrische Narcist
pi_71066909
quote:
Op vrijdag 17 juli 2009 14:23 schreef Swetsenegger het volgende:
Had ik al GD Library gezegd?
Ik vind ImageMagick toch beter .
pi_71069684
gewoon svg eruit gooien
  FOK!-Schrikkelbaas vrijdag 17 juli 2009 @ 16:18:12 #194
1972 Swetsenegger
Egocentrische Narcist
pi_71070901
quote:
Op vrijdag 17 juli 2009 14:27 schreef HuHu het volgende:

[..]

Ik vind ImageMagick toch beter .
Die staat niet standaard geinstalleerd bij de meeste shared hosters
pi_71071547
Owja, dat is ook zo stom inderdaad . Ik ben te verwend geraakt met servers in eigen beheer.
  vrijdag 17 juli 2009 @ 18:35:32 #196
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71074559
Mag:

1
2
3
4
5
6
<?php
        
if (!empty($nummer)) {
           
$nummers join(", "$nummer);
            include(
"sendsms.php");
        }
?>


Dit ook als de nummers in de IF's het volgende bevatten:

1
2
3
4
5
6
<?php
$nummer 
"0611111111";
$nummer "0622222222";
$nummer "0633333333";
$nummer "0644444444";
?>


En de included file leest dan $nummers uit de " join" .

Edit:

Werkt niet:

1Warning: join(): Invalid arguments passed in /home/remon01/domains/***l/public_html/phpsms/email.php on line 171


[ Bericht 4% gewijzigd door F4T4L_3RR0R op 17-07-2009 18:41:49 ]
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 18:37:20 #197
75592 GlowMouse
l'état, c'est moi
pi_71074596
quote:
Op donderdag 16 juli 2009 22:26 schreef GlowMouse het volgende:
Waarom probeer je het niet gewoon uit?
eee7a201261dfdad9fdfe74277d27e68890cf0a220f41425870f2ca26e0521b0
  vrijdag 17 juli 2009 @ 18:42:06 #198
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71074703
Zie edit vorige post.

Edit:

Ik lees dat "join" een alias van " implode" is met dezelfde functionaliteiten.

De nummers staan niet in een array, dus zal ik ze toch op nummerlengte in een array moeten splitten met "str_split". Wat uiteindelijk op precies hetzelfde uitkomt als hoe ik het vanmorgen had.

Edit2:

Had de bovenste regel gemist...

Weer testen brb.

[ Bericht 29% gewijzigd door F4T4L_3RR0R op 17-07-2009 18:55:55 ]
Een ballonvaarder die door de mand valt is nooit grappig...
  vrijdag 17 juli 2009 @ 19:10:48 #199
110933 F4T4L_3RR0R
R.I.P. webicon ;(
pi_71075340
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:

1$nummer = implode(", ", $nummer);


Of

1$nummers = implode(", ", $nummer);


Gebruik? Is het ene veiliger of beter dan het andere? Het werkt in ieder geval allebei.

[ Bericht 7% gewijzigd door F4T4L_3RR0R op 17-07-2009 19:57:55 ]
Een ballonvaarder die door de mand valt is nooit grappig...
pi_71077736
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.
Ik zou die laatste gebruiken, omdat het anders wat onoverzichtelijk wordt wat '$nummer' precies is.
Trotse poster van het 37000000ste bericht ^O^
abonnement bol.com Unibet Coolblue
Forum Opties
Forumhop:
Hop naar:
(afkorting, bv 'KLB')