PR de correção. Não preicisa mergeear. #5
Conversation
Push inicial, criacao do app react e branch
Iniciando o modelo messenger
incremente projeto
| const dadosMensagem = [ | ||
| { | ||
| usuario: "", | ||
| mensagem: "" | ||
| } | ||
| ] |
There was a problem hiding this comment.
Bem legal a ideia de isolar o state em uma constante fora do componente. Isso é bem legal para reaproveitamento de componentes. Parabéns!
| this.setState({ | ||
| usuario: event.target.value, | ||
| mensagem: event.target.value, | ||
| }); |
There was a problem hiding this comment.
Aqui, faltou vcs criarem no estado, além do valor dadoMensagem, que está guardando o array de mensagens, dois valores de estado que guardariam os inputs controlados. O nome poderia ser, por exemplo, inputUsuario e inputMensagem. Além disso, para cada input seria necessário uma função onChange. E no setState dessa função, vcs mudariam esses dois valores:
onChangeMensagem = (event) => {
this.setState({
inputMensagem: event.target.value,
});
}
onChangeUsuario = (event) => {
this.setState({
inputUsuario: event.target.value,
});
}
E lá nos respectivos inputs de usuário e mensagem, o value seria esses dois valores do estado:
<input
placeholder={"Usuario"}
value={this.state.inputUsuario}
onChange={this.onChangeUsuario}
/>
<input
placeholder={"Mensagem"}
value={this.state.inputMensagem}
onChange={this.onChangeMensagem}
/>
| <input | ||
| placeholder={"Usuário"} | ||
| value={this.state.usuario} | ||
| onChange={this.onChangeTextos} | ||
| /> | ||
|
|
||
| <input | ||
| placeholder={"Mensagem"} | ||
| value={this.state.mensagem} | ||
| onChange={this.onChangeTextos} |
There was a problem hiding this comment.
Deem uma olhada no último comentário que eu fiz.
|
Oi, Queridos! Fiz alguns comentários no código, peço que vcs deem uma olhada. No geral, vcs estão de parabéns. Conseguiram aplicar o conteúdo que foi passado durante a semana e estão com um código bem legal. Deem uma olhada sobre manipulação de array, estado inicial e input controlado. Principalmente, as aulas de quarta e quinta da semana passada e um vídeo que o Darvas gravou sobre o assunto e mandou no vimeo para vocês. |
PR de correção. Não preicisa mergeear.