Explore our sites

Stack Exchange
tour help careers 2.0
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 3 down vote favorite
1

I am looping over an array,

if I spot an error in the array I am adding a key called error.

however I am getting a whole pile of 'undefined index 'error' warnings.

How can I do this without generating those warnings?

Code as requested

$csv = array();
if (($handle = fopen($filePath, "r")) !== FALSE) {
  while (($csv[] = fgetcsv($handle)) !== FALSE);
  fclose($handle);
}

foreach ($csv as &$row) {
  if (count($row) > $maxCols)
    $maxCols = count($row);

  if (count($row) == 0) {
    $errors++;
    $row['error'] = 'Empty Column!';
    continue;
  }
  //more code
}

Example of what $row would look like in the foreach()

Array (
[0] => 1
[1] => 12
[2] => 64273566141
[3] => bakery
[4] => 2009-12-08 09:07:39
[5] => 2009
[6] => 2009-12-08 09:08:35
[7] =>
[8] => 0 )

See? Full Size

share|improve this question
1  
can you please show some code? –  kjy112 Feb 17 '11 at 1:27
 
and no, I cant add an empty index when I create the array as it comes from fgetcsv –  Hailwood Feb 17 '11 at 1:27
 
sorry, have added code now. –  Hailwood Feb 17 '11 at 1:30
 
isset or array_key_exists are the usual solutions to this error, but code please, cause given what you are describing you shouldn't be getting this error unless i've misunderstood what you are asking –  tobyodavies Feb 17 '11 at 1:30
1  
that code shouldn't produce an error... I think the error is more likely to be where you are checking for the error later (i.e. expand the //more code comment please) –  tobyodavies Feb 17 '11 at 1:33
show 7 more comments

1 Answer

up vote 2 down vote accepted

You're using == instead of =.

Using the comparison operator is attempting to fetch the potentially undefined array index. If $row['error'] has not been set previously, this will trigger an E_NOTICE undefined index error.

share|improve this answer
 
nope, Im adding the error index to all 'rows' of the csv array that have errors. Please see my code above –  Hailwood Feb 17 '11 at 1:34
 
@Hailwood Edited my answer given the new information in your question –  Phil Feb 17 '11 at 1:41
1  
And now I feel like an idiot :/ Thanks lol –  Hailwood Feb 17 '11 at 1:52
add comment

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.