I have a controller that does:

```
$objImporter->import('filename');

```

Then a class that:

```
public function import ($filename){

if(this->notExcelFile){
    return "not an excel file";
}
else carry on
}

```

Instead of having lots of returns (I only like one exit from a function, at the bottom) I usually throw and exception and then return that in the catch.

Is this the best way to do this? I started life as a VB developer and wonder if there is a better OO way to do this?

Mick