Pfthb

I have a function that is essentially going through a CSV file, and finding rows that match up with a user's settings in their profile. Anyway...

The issue is that I'm checking through the CSV file for each car (row), and then returning the value if the car matches the user's setting. The issue is, if a user has a setting for the following:

• Honda Civic 1993 through 2007


• Jeep Liberty 2002 through 2018

And the CSV has records of:

• Honda Civic 1996


• Honda Civic 2004


• Jeep Liberty 2010

... currently the if statement will only return the first iteration of the Honda Civic, before it then returns the Jeep Liberty. So it never knows to move to the next / subsequent matches of the if statement.

 private function find_vehicle($cars,$make,$model,$yr_start,$yr_end)
 $tmp = explode("

How can I adjust the foreach and/or if statement, so that it will not stop at the first version of a Make/Model/Year... but that it will continue to return each match as a new $car?

Otherwise, currently the output is:

• Honda Civic 1996


• Jeep Liberty 2010

But I need it to also return the other matching Honda Civic 2004

You can turn your function into a generator (see PHP Generators Overview https://secure.php.net/manual/en/language.generators.overview.php):

private function find_vehicle($cars,$make,$model,$yr_start,$yr_end) 
 $tmp = explode("

and call it like:

foreach (find_vehicle($cars, $make, $model, $yr_start, $yr_end) as $car) 
 // Do something with $car

The idea would be to build an array for the matching cars, and return this list at the end of the function (some comments in the code)...

private function find_vehicle($cars,$make,$model,$yr_start,$yr_end)", $model);
 $model = $tmp[1];
 // List of matching records
 $matching = ;
 // echo 'Looking for: ' . $make . ' '. $model . ' between: '.$yr_start .'-'.$yr_end . '<br>';
 foreach ( $cars as $car ) 
 if ($make == $car[1] && $model == $car[2])
 if ($car[4] >= $yr_start && $car[4] <= $yr_end)
 // Add new entry to matching list (rather than return the first one)
 $matching = $car;
 
 
 
 return $matching;

This uses $matching to keep a list, adding to it using

$matching = $car;

At the end it passes this back, it will be an empty list if nothing is found

return $matching;

As you can also see I've commented out the echo - it's useful for debugging, but try and avoid methods that both process data and produce output unless you have to.