Naming
mathieutu opened this issue · comments
Mathieu TUDISCO commented
@ifnot je trouve que les noms de tes méthodes et classes sont pas fous.
Exemple: pour moi broadcastWith()
c'est une action, qui prend des arguments qui correspondent au with.
là tu retournes juste un array, c'est bizare.
idem pour les autres methodes.
Et pour ton trait, pour moi un trait c'est qqchose qui te rajoute des actions, comme un service que tu rajoute à ton model donc il doit avoir un nom d'action. isStore
pour moi c'est un truc qui retourne un booléen, comme tous les is*
ou has*
. Exemple pour mon package: JsonExportable
pour l'interface, JsonExporter
pour le trait