Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Style e dicas gerais #22

Open
vinicius73 opened this issue May 14, 2016 · 1 comment
Open

Code Style e dicas gerais #22

vinicius73 opened this issue May 14, 2016 · 1 comment

Comments

@vinicius73
Copy link

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.

@matuzalemsteles
Copy link
Member

@vinicius73 muito obrigado pela sua visão sobre o Kep.

O conceito da FW é simples, trabalhar com API's apenas, para poder ser consumida por qualquer tipo de aplicação, Vue, Angular, Mobile, IoT etc... claro que ainda preciso melhorar algumas coisas e adicionar mais regras ao sistema, mas é algo a se trabalhar.

Sobre o code-style é uma coisa que pretendo corrigir e melhorar.

Sobre Seeds, não sabia do uso do traits, dei uma leve pesquisada e com certeza é uma feature poderosa do PHP, desconhecia, obrigado por apresentar.

O uso do Dependency Injection pretendo colocar ele sim no Kep, já vinha falando sobre isto em meu twitter... mas obrigado pela referencia do PHP-DI.

Obrigado, deste já!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants