[Resolved] Undefined Offset Error

cbrknight

I am importing data from csv into a temporary table...

The insert is working properly... all files are being imported into the users_working database....

however.... I am getting an undefined offset when I try to get a count of the files that were imported... any ideas what I have done wrong?

// get csv contents and build multi-value list insert query
while ($data = fgetcsv ($fp, 1000, ",")) {

$num = count ($data);
$first = $data[0];
$last= $data[1];
$valid = $data[7];

$sql1 = "INSERT INTO users_working (fName, lName, validation, datestamp, status) VALUES ('$first', '$last', '$valid', '$datestamp', '$status')";
$c0++;

// check file was not empty then insert data into work table
if ($c0 > 0){
$r1 = mysql_query($sql1, $importer)or die("<br>Error: ". mysql_error());
$c1 = mysql_affected_rows();
// get count of records inserted, part of hte string returned
$ary = explode(" ", $r1);
$c1 = $ary[1];
}
else{
echo ("Input file was empty");
}
}//end loop

tsinka

Hi,

INSERT queries don't return a result set.
Just remove the lines

$ary = explode(" ", $r1);
$c1 = $ary[1];

The line

$c1 = mysql_affected_rows();

already sets $c1 to the number of inserted rows.

EDIT: No, preset $c1 to 0 outside the loop and change that line to

$c1++;
to count the total number of rows inserted. Each query just inserts one row so you just need to increase $c1 by one.

Thomas

cbrknight

I tried that before ... and just tried it again....

It only increments the counter by one.

tsinka

So, that part of the code looks like this now ?

if ($c0 > 0){
$r1 = mysql_query($sql1, $importer)or die("<br>Error: ". mysql_error());
$c1++;
}

Alternatively, just use $c0 as counter since it already gets incremented by one each time.

Thomas

Roger_Ramjet

If you use an INSERT ... VALUES statement with multiple value lists or INSERT ... SELECT, the statement returns an information string in this format:

Records: 100 Duplicates: 0 Warnings: 0

Now, I gave cbrknight the code that he is using here, the idea being that he could check the information string before continuing with the next operation.

It was my mistake. I have since found that the information in the manual is incomplete. The action query does return information, but this has to be called up using mysql_info().

" Some queries (LOAD DATA INFILE ..., INSERT INTO ... SELECT ..., UPDATE) return additional information. The result is returned by mysql_info(). See the description for mysql_info() for the format of the string that it returns. mysql_info() returns a NULL pointer if there is no additional information."

Apologies, cbrknight. Just use the usual rows affected to check query execution.

cbrknight

Well I did quite a bit of modification of the code... but I finally got it to do what I want. Thanks to everyone for all their help. Wouldnt have been so thick headed usually but I am fighting a cold right now. I am sure all of you know how hard it is to trace code with a cold.

:queasy: