Пётр Еремейкин. ДЗ по react #11
Conversation
There was a problem hiding this comment.
Ты молодец, спасибо за решение. Мне понравилось, что ты использовал materialUI, наследовался от PureComponent, думая о производительности =).Поделил на компоненты, которые можно было описать в отдельных модулях. Использовал Fragment и ref callback о которых мы не говорили на лекции. Мои комментарии, касаются не react части по большей части и носят информационный характер.
| }; | ||
|
|
||
| doFetch = (reqId) => { | ||
| let self = this; |
There was a problem hiding this comment.
это лишнее, используй синтаксический bind, если нужно привязать контекст.
| })); | ||
| }) | ||
| .catch((error) => { | ||
| self.setState(state => ({ |
There was a problem hiding this comment.
вот тут у тебя как раз будет this, выше делай тоже синтаксический bind.
| @@ -0,0 +1,40 @@ | |||
| import React, {PureComponent, Component} from "react"; | |||
|
|
|||
| export default class Logs extends Component { | |||
There was a problem hiding this comment.
Можно просто Function Component
|
|
||
| render() { | ||
| return ( | ||
| <Fragment> |
There was a problem hiding this comment.
Fragment в данном месте лишний.
| return ( | ||
| <Fragment> | ||
| <div id="toolbox" className="toolbox"> | ||
| <Input className={"searchInput"} inputRef={input => (this.input = input)}/> |
There was a problem hiding this comment.
Для информации можно, использовать React.createRef.
No description provided.