Each feed type should be a separate class, instead of checking all the time what type of feed it is: if($type == 'RSS 1') ... elseif($type == 'RSS 2') elseif($type == 'Atom') This is not the OOP way.
Each feed type should be a separate class, instead of checking all the time what type of feed it is:
if($type == 'RSS 1')
...
elseif($type == 'RSS 2')
elseif($type == 'Atom')
This is not the OOP way.