Домашняя работа №7 - React form

redexp commented 3 years ago

Я так подумал, что вас не так много и я уже могу лично проверять ваши домашки. Это будет быстрее и эфективнее. Когда вы будете считать, что ваша домашка готова к проверке, то просто напишите в этом issue.

IrshakAnna-Mariia commented 3 years ago

Доброго дня. Домашку зробила.

redexp commented 3 years ago

неплохо, смутило только то что ты вместо cell использовала phones[index] https://github.com/IrshakAnna-Mariia/GeekHub-2020/blob/674cf6e55df2bb648ec13290ee4477120a3ee908/react-form/src/UserForm.js#L129-L133 вот так будет лучше

{phones.map((cell, index) => (
  <div 
    className="input-group mb-3"
    key={cell.number}
  >...

использовать this.state в этом коде принципиально неправильно https://github.com/IrshakAnna-Mariia/GeekHub-2020/blob/674cf6e55df2bb648ec13290ee4477120a3ee908/react-form/src/UserForm.js#L16-L20

IrshakAnna-Mariia commented 3 years ago

Виправила

IrshakAnna-Mariia / GeekHub-2020

Домашняя работа №7 - React form #6