[Clarence]

En metod bör vara ansvarig för bara att kolla om användaren är admin (eller för den delen andra rollen i samma metod). Om du sedan ska skriva ut saker beroende av roll, bör det i ditt fall ligga i en annan metod. Från funktionen du har skrivit, så returnernar den content, så du kan lägga till och förändra $content genom att anropa den med $content = admin(..., $content); Men i din funktion modifierar du inte alls content och har ens ingen return om man inte är admin. Resultatet av din kod är helt enkelt otroligt otydlig och väldigt jobbig för andra än programmeraren, eller också för denna efter ett år, att läsa.

Tar man det ett steg längre och strukturerar upp det lite så har du en klass som är ansvarig för autensifiering och därmed rollbestämmande. Denna anropas från controller-nivån och utifrån resultat så har du en annan klass som ansvarar för att hämta mer information beroende på roll (och för det mesta fler klasser för varje typ av information). Sedan skickas all data till ett vy-lager som skriver ut det eller iaf skapar dokumentinnehållet (ofta vill man skriva ut det centralt för att få bra kontroll på headers, felhantering, loggning etc etc).

MVC kallas den generella approachen om det skapar något intresse. Det ger nästan per automatik gör att du får mer lättläslig och straight forward kod överallt, vilket minskar risken för buggar, ökar testbarhet och läsbarhet osv.