g1ovanaqueiroz / highscore

Fórum de apredizagem de música

Hi there!

I am a computer scientist.

I had the chance to participate in some amazing communities like @guardians-DSC @elasComputacao @ras-ufcg you might like to check them out 💜

I'd love to talk about teaching, functional programming, agile methodologies, and music.

📫 You can reach me at:

highscore's Issues

criar constante para peças

Olá florzinha de cacto, então, notei que vc está setando todos os valores das peças quando vai chamar o component. Isso dá um trabalhão... que tal criar uma constante SCORE contendo o json de todas as peças e exporta-la junto ao component? ficaria bem mais legal e facilitado <3

Estados "curtido" e "não curtido" ambíguos em Comentario.js

Olá Giovana, tudo bem?

Gostei bastante do seu projeto, achei muito interessante! 🎶 ✨

Fui vasculhar os componentes e lá em components/Comentario.js percebi o seguinte cenário:

this.state = {
    curtido: false,
    nao_curtido: false
}

Acredito que você não precisa desses dois estados curtido e nao_curtido, basta substituir os usos de nao_curtido para !curtido.

Assim, curtido pode assumir 3 valores:

• undefined: quando o usuário nem curtiu, nem "descurtiu" o comentário.
• false: quando o usuário "descurtiu" o comentário.
• true: quando o usuário curtiu o comentário.

Para isso, você pode iniciar o estado de curtido da seguinte forma:

this.state = {
    curtido: undefined
}

PS: Lembre-se que !curtido vai retornar verdadeiro quando ele for undefined, então quando quiser checar se o usuário ainda não curtiu nem "descurtiu" o comentário, pode usar if (curtido === undefined) { ... } que funciona numa boa!

Que achas? 😄

Boas praticas para nome de componentes d separação em pasta

Esse artigo fala um pouco sobre boas práticas de nomeação e organização de com componentes em um proj react.

Acho que seria interessante centralizar todos os assets em uma única pasta. Todas as imagens que forem ser usadas na aplicação ficariam em um único diretório.

Converter componentes de classe em componentes funcionais

Oi, @giovanaBritoOliveira! 🧙‍♂️ Queria sugerir converter teus componentes atuais (baseados em classes) em componentes funcionais. Aqui tem uma introdução bem didática do React sobre como utilizá-los.

Além de alinhar teu projeto à disciplina, vai reduzir a complexidade do desenvolvimento em várias frentes, como dispensando o uso dos .bind(), por exemplo. 🧜‍♂️

Criar um package.json na base do projeto

Poderia colocar um arquivo package.json na base do projeto e assim quando desse 'npm run' rodária tanto o back como o front em um único comando.
Tu pode achar um exemplo nesse repo aqui.

Destructuring para props

https://github.com/giovanaBritoOliveira/HighScore/blob/86a034cb9109117819b4d55ff8a4be0f11974001/frontend/src/components/Peca/Descricao/Descricao.js#L5

Oi Giovana, quero deixar uma sugestão, pois vi que você usou props em alguns lugares sem tirar vantagem da desestruturação. Acho pode deixar teu código mais legível. Nesse link aqui explica melhor: link

Sugestão para usar camelCase

Oi, @giovanaBritoOliveira ! 🎃

Gostaria de sugerir que você considere usar camelCase invés de snake_case para as props dos componentes. Dessa forma, o teu código fica padronizado quanto à nomenclatura de propriedades do React.

Por exemplo nesse componente a propriedade className (que é coisa do JSX mesmo) da div usa camelCase enquanto as propriedades de Comentario usam snake_case.

https://github.com/giovanaBritoOliveira/HighScore/blob/5bdf51afd5e06ebbe737a835d2b4d1e8bfe41786/frontend/high_score/src/App.js#L9-L11

É isso! Bom trabalho. 💯

Mover funções para dentro do componente

Oi, @giovanaBritoOliveira .

Acredito que essas funções fariam mais sentido dentro do componente Comentario, uma vez que elas usam informação (curtido) que reside dentro dele.

https://github.com/giovanaBritoOliveira/HighScore/blob/8b10a8931e158efc199cddaa0425f80ce83c66a4/frontend/src/components/Comentario/Comentario.js#L21-L34

pasta de imagens dissociada

no caso, adiciona uma pasta de imagens a cada parte do projeto. No real, as duas partes poderao esta sendo executadas em locais de diretorios diferentes e acabar gerando um conflito de busca. Alem de gerar colisao no uso do mesmo arquivo por dois processos diferentes 👍