der __construct is schon mal voll kommen falsch, der UserService ( ja service passt da eher als User, koennte man mit der tabelle verwechseln) sollte lieber eine private methode haben um an die db zu kommen, bzw. an den DbService und keine neue db instanz und kein globaler scheiß, sondern auf eine factory die sich darum kuemmert die db instanz zu erstellen, ka wie du das genau machst, ob plain sql oder doctrine oder so schoene klassen um querys zuerzeugen, aber das waere wenigstens bissel besserer weg
|