PHP生成器出错

Question

What is the best way to inform who use my generator function if something errors occurs, instead of writing weird return or raising exception like this piece of code

function csv_file_generator($csvFilename, $delimiter = ";", $enclousure = '"') {
    if(($csvHandler = fopen($csvFilename, 'rb')) === false) {
        return;
    }

    while (($row = fgetcsv($csvHandler, 0, $delimiter, $enclousure)) !== false) {
        yield $row;
    }

    if (feof($csvHandler) === false) {
        return;
    }

    if (fclose($csvHandler) === false) {
        return;
    }

    return; /* Exit Generator */
}

Answer 1

<?php
class CsvFileGenerator {
    protected $fp;
    protected $delimiter;
    protected $enclousure;
    public function __construct($filename, $delimiter = ";", $enclousure = '"'){
        $this->delimiter=$delimiter;
        $this->enclousure=$enclousure;
        if(!file_exists($filename)){
            throw new Exception("file [$filename] dont exists");
        }
        if(!is_readable($filename)){
            throw new Exception("file [$filename] is not readable");
        }
        $this->fp =  fopen($filename, 'rb');
        if($this->fp === false){
            throw new Exception("cant open [$filename]");
        }
    }
    public function getGenerator(){
        while (($row = fgetcsv($this->fp, 0, $this->delimiter, $this->enclousure)) !== false) {
            yield $row;
        }
    }
    public function __destruct() {
        if($this->fp){
            fclose($this->fp);
        }
    }
}

foreach( (new CsvFileGenerator('mycsvfile.csv'))->getGenerator() as $line){
    #do some
}

One way to rome. :-)