Hi

I'm trying to get the following script to 1st, pick out all the enties out of a particular file with Gwent in the address, then all of those entries within Newport.  It starts printing when it reached the first Newport/Gwent, but then print everything after it.  Any ideas gratefully received.

<?php
$fh = fopen("/usr/home/testdata.txt","r");
$count =0;
while (list($place,$name,$phone,$addy,$email) = fgetcsv($fh,10000))  {

ereg('Gwent',$addy,$got);
  if($got){
   if ($place=="Newport")  {
      print "$accom, $addy, $got[0]<br>\n";
      $count++;
   }
 }
}
print $count;
?>      

Thanks

Chris

Have you trimmed some code out of the example? Where does $accom come from??  I'm on the road - so only able to have a brief look today.  I also wonder ... I think there's somthing missing or that I haven't spotted.  I would tend to cut /paste/ modify the code to get more metrics for the problem.   I looked at the fgetcsv call - if it fails to match, it probably doesn't overwrite each of the list items but merely returns a "false" leaving old values in the variables including $place.   That could be a pointer, except that in the sample code the problem appears to be within the loop.

Sorry, $accom should be $name, other than that all the code is there.

I've tested the script without the if ($got) { ....  loop and it does print out all places with Newport in $place, so I know this bit works, also it confirms that $got contains Gwent.

I just can't seem to then filter down for places with Gwent in the Address($addy).  Although I appreciate the script as it stands does it in reverse order.

Ah - I've spotted something.

ereg returns a true / false value if it matches.  If it matches, it saves its "interesting bits" into the third parameter variable, $got in this case.  You want to write
       $isit = ereg( .....
then test $isit rather than $got.

The problem you've hit is that if a match fails, $got isn't changed  - thus the data leaks through from sucessful matches to failed matches the way it's written at the moment.


Cool, thanks.

Its working perfectly now.

---