getkep / kep Goto Github PK

Acabei de me deparando com um post apresentando o Keep. Sempre é bom conhecer novas ferramentas, parabéns pela iniciativa.

Essa issue é uma "dica".

Não peguei 100% do conceito do FW, porém algo me incomodou bastante, o code-style.
Vocês possuem um pouco de inconsistência na nomenclatura de classes e namespaces.

namespace GetKep\Kep\route;

use GetKep\Kep\route\RouteFactory as Factory;

class Route extends Group { 

Seu namespace route esta minusculo, enquanto todo o resto esta com camel-case.

<?php
    class encryption {

Já esta esta sem namespace e a classe esta em minusculo.
Alem de ferir o code-style o não uso de namespace pode gerar possíveis colisões no futuro alem de tornar virtualmente impossível uma modularização do sistema.

Sobre seeds.
Seria bem mais produtivo o uso de traits no lugar dessa abordagem, alem disso o termo seed é muito atrelado a Database Seed Laravel e Cake

Uma diga "geral" que impactaria em todo o projeto é a adoção de um sistema de DI
Uma lib que vocês podem usar PHP-DI

É isso, espero que alguma dica seja útil e boa sorte com a empreitada.