Php error issue in while loop

Question

I've a SMS Api which send sms(not email) to my client with php. Now I'm trying to send sms from .txt file where all contact number exist:

My .txt file

shibbir, 8801678877639
babu, 8801534552503

So my while loop is look like this:

$file_handle = fopen("$file", "r");
while ( !feof( $file_handle ) ) {
    $line_of_text = fgetcsv($file_handle, 1024);
    $line_of_text[1] ."<BR>";       
    $obj = new Sender("myservername","myport","username","password","$sender", "$msg", 
"$line_of_text","2","1"); 
    $obj->Submit();     
    echo "<div class='success'>Successfully sent your message to " . $line_of_text[1] . " Thank 
    You.</div>";
}
fclose( $file_handle );

But I get following error message:

Notice: Array to string conversion in D:\Software Installed\xampp\htdocs\evan\toplevel
\bulksms.php on line 108

Notice: Undefined offset: 1 in D:\Software Installed\xampp\htdocs\evan\toplevel\bulksms.php on 
line 106

Notice: Undefined offset: 1 in D:\Software Installed\xampp\htdocs\evan\toplevel\bulksms.php on 
line 107

My final goal is, User can send sms by uploading .txt file where all contact number exit. So once it's submit the form then sms must be send to those contact number's.

Can you guys tell me why i get error message ?
And it's show 2 Success confirmation but i want it's must be show 1 success confirmation.

Thanks.

These are not errors. They're notices. Also they point to specific line in code. How this code looks like? Which line is it? — Jakub Matczak
– Jakub Matczak, Commented Sep 17, 2013 at 7:56
@MansoorkhanCherupuzha Output is : array(2) { [0]=> string(7) "shibbir" [1]=> string(14) " 8801671133639" } AND array(2) { [0]=> string(4) "babu" [1]=> string(14) " 8801534552503" } — Alex
– Alex, Commented Sep 17, 2013 at 7:59
fgetcsv returns an array, you cant then just add the <br> to it like you have — user557846
– user557846, Commented Sep 17, 2013 at 8:00
Is it working if you remove the statement $line_of_text[1] ."<BR>"; from the code? — Mansoorkhan Cherupuzha
– Mansoorkhan Cherupuzha, Commented Sep 17, 2013 at 8:04

Mansoorkhan Cherupuzha · Accepted Answer · 2013-09-17 08:40:56Z

1

Try this.

$file_handle = fopen("$file", "r");
$success_number = array();
    while ( !feof( $file_handle ) ) {
        $line_of_text = fgetcsv($file_handle, 1024);
        $success_number[]=$line_of_text[1]; //call this statement if msg sent successfully.
        $line_of_text  = implode(',',$line_of_text ); // i think we need to pass string to the api.
        $obj = new Sender("myservername","myport","username","password","$sender", "$msg", 
    "$line_of_text","2","1"); 
        $obj->Submit();     
    }
    echo "<div class='success'>Successfully sent your message to ".implode(',',$success_number) ."<BR> Thank You.</div>";
    fclose( $file_handle );

MY TEST

my.txt

shibbir, 8801678877639
babu, 8801534552503

my.php (removed the call to api)

    <?php
    $file = "my.txt";
    $file_handle = fopen($file, "r");
    $success_number = array();
    while ( !feof( $file_handle ) ) {
        $line_of_text = fgetcsv($file_handle, 1024);    
        $success_number[]=$line_of_text[1]; //call this statement if msg sent successfully.
    }
    fclose( $file_handle );
    echo "<div class='success'>Successfully sent your message to " . implode(',',$success_number) ."<BR> Thank You.</div>";
?>

OUTPUT

    Successfully sent your message to 8801678877639, 8801534552503
Thank You.

edited Sep 17, 2013 at 8:40

answered Sep 17, 2013 at 8:05

Mansoorkhan Cherupuzha

1,7611 gold badge24 silver badges45 bronze badges

Sign up to request clarification or add additional context in comments.

14 Comments

Shankar Narayana Damodaran Over a year ago

Again, this will print the message 2 times which the OP does not want.

Mansoorkhan Cherupuzha Over a year ago

@ShankarDamodaran, yes it will print two times since their is two lines of text in the file

Mansoorkhan Cherupuzha Over a year ago

@Alex, let me test this in my local host.

Alex Over a year ago

@MansoorkhanCherupuzha, Now I don't get any error message but it's not sending SMS. You know if i want to send multiple sms then i've put comma(,) after one number like: 8801671144528, 8801547744854. So now is it the problem ? If so what can i do in my $Obj-> ?

Alex Over a year ago

@MansoorkhanCherupuzha, GREAT, JUST GREAT. It's showing me..Successfully sent your message to 8801675588639,8801535884503,, Thank You. I saw the double comma after last number.

|

Gar · Accepted Answer · 2013-09-17 08:00:15Z

1

try changing your code into this

$file_handle = fopen("$file", "r");
while (!feof($file_handle) )
{
$line_of_text = fgetcsv($file_handle, 1024);
$line_of_text[1]=$line_of_text[1] ."<BR>";       
$obj = new Sender("myservername","myport","username","password","$sender", "$msg", 
$line_of_text[1],"2","1"); 
$obj->Submit();     
echo "<div class='success'>Successfully sent your message to " . $line_of_text[1] . " Thank 
You.</div>";
}
fclose($file_handle);

answered Sep 17, 2013 at 8:00

Gar

86213 silver badges20 bronze badges

Comments

Shankar Narayana Damodaran · Accepted Answer · 2013-09-17 08:01:00Z

1

1) Can you guys tell me why i get error message ?
2) And it's show 2 Success confirmation but i want it's must be show 1 success confirmation.

They are not errors.
Move the echo DIV tag out of your while loop.

answered Sep 17, 2013 at 8:01

Shankar Narayana Damodaran

68.6k43 gold badges102 silver badges129 bronze badges

Comments

Charaf JRA · Accepted Answer · 2013-09-17 08:03:31Z

0

Change this line :

$line_of_text[1] ."<BR>";

It should be :

$line_of_text[1] .= "<BR>";

and also your message is inside your loop , so it will be echoed more than once.

This task can be done easily using file() and explode()

answered Sep 17, 2013 at 8:03

Charaf JRA

8,3551 gold badge36 silver badges44 bronze badges

Collectives™ on Stack Overflow

Php error issue in while loop

4 Answers 4

14 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

14 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related