PR para correção, não precisa mergear. #2
Conversation
|
|
||
| export class Mensagem extends React.Component{ | ||
|
|
||
| state={ |
There was a problem hiding this comment.
Sempre se preocupem com a indentação do código. Esse state estaria bem mais claro se estivesse escrito dessa forma:
state = {
mensagens: [
{
remetente: "",
conteudo: ""
}
],
valorInputUsuario: "",
valorInputMensagem: ""
}
Eles estão fazendo a mesma coisa, mas de forma bem mais indentada e clara.
| usuario:this.state.valorInputUsuario, | ||
| mensagemUsuario:this.state.valorInputMensagem | ||
| } | ||
| const mensagemAtualizada=[...this.state.mensagem,mensagemNova |
There was a problem hiding this comment.
Perfeita a cópia do array feita aqui. Apenas tomem cuidado com a indentação. O array está sendo fechado na linha de baixo, poderia estar tudo na mesma linha.
| } | ||
|
|
||
| render(){ | ||
| const listaMensagem=this.state.mensagem.map((msg) =>{ |
There was a problem hiding this comment.
Legal o uso do map. É isso mesmo, muito bem feita a lógica!
|
Fala, queridos! Fiz umas observações ao longo do código de vcs, peço que deem um olhada. No geral, está muito bem feito. A lógica de uso do estado e criação das mensagens está feita de forma clara e vcs aplicaram muito bem os conhecimentos passados. Um único feedback que eu dou pra vcs é o de prestarem atenção com a indentação e a organização do código de vcs. Isso é mto importante para a boa leitura do que vcs estão fazendo, principalmente quando os projetos forem maior. Mas, no geral, vcs estão de parabéns! Gostei muito da organização do PR e de terem deixado o link do surge. Continuem assim! |
PR para correção, não precisa mergear.