reactjs / reactjs/react.dev

Challenge Solution in "Updating Arrays in State" Page Uses Mutation

オープン
#5,672 コメント 5 件 リアクション 1 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

feedback
主要言語
JavaScript
スター
11.8k
フォーク
7.9k
平均マージ
1日 11時間
マージ済み PR(30日)
11

説明

For the solution to the third challenge in the Updating Arrays in State page, it uses the following function:

function handleAddTodo(title) {
  setTodos([
    ...todos,
    {
      id: nextId++,
      title: title,
      done: false
    }
  ]);
}

This changes nextId, a variable outside the component's scope, which I believe is a mutation. It should instead use the length of todos to set the next id.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

リンク先の「Updating Arrays in State」ページから始めて、3つ目のチャレンジの解決策、特に handleAddTodo 関数とその nextId の使用方法を確認してください。コンポーネントの外部にある変数を変更しないように例を更新し、その後もチャレンジの解決策が期待される todo ID を生成することを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, react
領域
documentation
issue の種類
ドキュメント
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
50/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。