[Nikas]

Hi, I have another question.

It's about validating a csv file when uploaded. I tried this code, but doesn't work perfectly, so wanna check out where did I gone wrong or is there a better method.

PHP Code:

foreach($temp as $n){
        $temp1= explode("\r\n",$n);//
        $temp2= explode(",",$temp1[0]);
        // check there is no NULL values in the first three column for every row.
        for ($i=0;$i<=$temp2[2];$i++){
            if ($temp2[$i] == NULL) { // if any of the first three column is NULL
                echo $message3; // error message
                echo "<meta http-equiv='refresh' content='0;URL=../'>";
            }
        } 

The above is my current validation code. There is still some more to it, those seem to be alright. My main concern is the above codes. What I need is to check every row in the csv file and on each row, the first 3 column to see if there is any NULL value. If there is, then either reject the whole file OR upload only those that are without NULL values. (Not sure which is better).

If possible, I would also like to check exactly which ROW and COLUMN is missing and prompt the user about the particular missing ROW and COLUMN.
TIA.

[UnrealEd]

i created a new thread of this post, as it has nothing to do with your old thread

[Nikas]

Sure, thanks for it.

[job0107]

If $temp contains the entire contents of the CSV file, and all records were seperated by \r\n,
then the foreach loop would put one record into $n each time. So you wouldn't need to explode
$n on \r\n because it only contains one record.

PHP Code:

<?php
foreach($temp as $n)
{
 $temp1= explode(",",$n);
 // check there is no NULL values in the first three column for every row.
 for ($i=0;$i<3;$i++)
 {
  if ($temp1[$i] == NULL)
  {
   // if any of the first three column is NULL
   echo $message3; // error message
   break;
   // I'm not sure what you are trying to do here //
   //echo "<meta http-equiv='refresh' content='0;URL=../'>";
   /////////////////////////////////////////////////
   }
  }
 }
?>

[Nikas]

Thanks for the reply.

Yes, $temp contains the entire contents of the CSV file. Here's the code.

PHP Code:

$temp = file($path.$MyFile_name); 

Actually, I don't really understand what is \r, can you explain to me?

After echoing message3, I'm doing some other validation. Should it fall under the for loop ? or outside the for loop better?

PHP Code:

if (strlen($temp2[1]) == '11'){ // if mobile no length is 11
            if ($temp2[3]){                                            //¸¸×é´æÔÚ, if parent department
                $key = array_search($temp2[3],$g_type); // search uploaded parent department name with DB g_type (department name)
                if(isset($key)){                                    //¸¸×éµÄID²¢ÇÒ´æÔÚ, if exist
                    //È¡µÃÌí¼Ó×éµÄ id ºÅ
                    if($t_id = $g[$g_id[$key]][$temp2[2]]){        //×Ó×é´æÔÚ
                        $p_id = $g_id[$key];
                        $isTrue = true;
                    }
                }
            } else{
                $key = array_search($temp2[2],$g_type);
                if(isset($key)){
                    $p_id = ($g_uid[$key] == '0')?$g_id[$key]:$g_uid[$key];
                    $t_id = $g_id[$key];
                    $isTrue = true;
                }else{
                    $isTrue = false;
                }
            } 

[Sxooter]

why not use fgetcsv() ?

[Nikas]

Isn't fgetcsv() also getting all the content in the csv file?

Right now, I need to validate the data, NOT getting the data.

[job0107]

Actually, I don't really understand what is \r, can you explain to me?

\r is the symbol for a carriage return and the \n is the symbol for new line.
Some word processors use the \r and some use the \n for a new line.
So sometimes we use both \r\n just in case.




Here, check this out. What do you think?

I stored the contents of the file into a multi-dimensional array for later use.
Now you won't have to go back to the file. Everything is in the array $temp1.
I also tagged the end of each record with a 0 or 1 depending on the
value of $isTrue. You don't have to use it but it might come in handy later.

PHP Code:

<?php
$rc = 0;
$handle = fopen($path.$MyFile_name, "r");
while(($temp = fgetcsv($handle)) !== FALSE)
{
 $temp1[$rc] = explode(",",$temp);
 // check there is no NULL values in the first three column for every row.
 for($i = 0;$i < 3;$i++)
 {
  if($temp1[$rc][$i] == NULL)
  {
   // if any of the first three column is NULL
   echo $message3; // error message
   break;
   }
  }
 if(strlen($temp1[$rc][1]) == '11')  // if mobile # length is 11
 {
  if($temp1[$rc][3])   //  if parent department
  {
   $key = array_search($temp1[$rc][3],$g_type); // search uploaded parent department name with DB g_type (department name)
   if(isset($key))
   {
    if($t_id = $g[$g_id[$key]][$temp1[$rc][2]])
    {
     $p_id = $g_id[$key];
     $isTrue = true;
     $temp1[$rc][count($temp1[$rc])] = 1;
     }
    }
   }
  else
  {
   $key = array_search($temp1[$rc][2],$g_type);
   if(isset($key))
   {
    $p_id = ($g_uid[$key] == '0')?$g_id[$key]:$g_uid[$key];
    $t_id = $g_id[$key];
    $isTrue = true;
    $temp1[$rc][count($temp1[$rc])] = 1;
    }
   else
   {
    $isTrue = false;
    $temp1[$rc][count($temp1[$rc])] = 0;
    }
   }
  }
 $rc++;
 }
fclose($handle);
?>

Or maybe like this. I'm not sure. You pick.

PHP Code:

<?php
$rc = 0;
$handle = fopen($path.$MyFile_name, "r");
while (($temp = fgetcsv($handle)) !== FALSE)
{
 $temp1[$rc] = explode(",",$temp);
 // check there is no NULL values in the first three column for every row.
 for ($i = 0;$i < 3;$i++)
 {
  if ($temp1[$rc][$i] == NULL)
  {
   // if any of the first three column is NULL
   echo $message3; // error message
   break;
   }
  if(strlen($temp1[$rc][1]) == '11')  // if mobile # length is 11
  {
   if($temp1[$rc][3])   //  if parent department
   {
    $key = array_search($temp1[$rc][3],$g_type); // search uploaded parent department name with DB g_type (department name)
    if(isset($key))
    {
     if($t_id = $g[$g_id[$key]][$temp1[$rc][2]])
     {
      $p_id = $g_id[$key];
      $isTrue = true;
      $temp1[$rc][count($temp1[$rc])] = 1;
      }
     }
    }
   else
   {
    $key = array_search($temp1[$rc][2],$g_type);
    if(isset($key))
    {
     $p_id = ($g_uid[$key] == '0')?$g_id[$key]:$g_uid[$key];
     $t_id = $g_id[$key];
     $isTrue = true;
     $temp1[$rc][count($temp1[$rc])] = 1;
     }
    else
    {
     $isTrue = false;
     $temp1[$rc][count($temp1[$rc])] = 0;
     }
    }
   }
  }
 $rc++;
 }
fclose($handle);
?>

[Nikas]

Thank You 0107, just saw your reply as I was out of town this weekend.

Will try out your suggestion tomorrow, getting late here now.

[Sxooter]

Originally Posted by Nikas

Isn't fgetcsv() also getting all the content in the csv file?

Right now, I need to validate the data, NOT getting the data.

Not exactly. fgetcsv gets a single line at a time from a file and decodes it from csv to an array.

If your csv file should have 7 fields in it, you can do something like:

PHP Code:

$fp = fopen("file.csv","r");
while (!feof($fp)){
    $row = fgetcsv($fp,10000);
    if (count($row)!=7) {
       die ("Bad file");
    } else {
        do something
    }
}