質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

TypeScript

TypeScriptは、マイクロソフトによって開発された フリーでオープンソースのプログラミング言語です。 TypeScriptは、JavaScriptの構文の拡張であるので、既存の JavaScriptのコードにわずかな修正を加えれば動作します。

React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

Q&A

解決済

1回答

2171閲覧

useEffectのクリーンアップに関するエラーを解消したい

k49977

総合スコア27

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

TypeScript

TypeScriptは、マイクロソフトによって開発された フリーでオープンソースのプログラミング言語です。 TypeScriptは、JavaScriptの構文の拡張であるので、既存の JavaScriptのコードにわずかな修正を加えれば動作します。

React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

0グッド

0クリップ

投稿2021/06/09 06:03

前提・実現したいこと

現在Reactの勉強をしています。
useEffectでデータをGETし、それをuseStateに格納する処理をしているのですが、
onSubmit時の画面遷移時にコンソールエラーを吐きます

Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.

実装は以下のようなコードです。
画面表示項目のために初期表示時にuseEffectでGETメソッドでAPIを実行し、
保存ボタン押下時のonSubmit処理で画面遷移しています。
どちらも非同期処理なのでクリーンアップする必要があるのかなと思っていますが、
いろいろ試してみてもなかなか解消されません。
「window.location.href」を削除するとエラーが吐かれなくなります

tsx

1React.useEffect(() => { 2 (async () => { 3 /* axiosを使ってAPIからデータをgetする処理(省略) */ 4 5 setHoge(response); 6 })(); 7}, []); 8 9const onSubmit = React.useCallback( 10 async (form: HogeForm) => { 11 /* axiosを使ってAPIからデータをPUTする処理(省略) */ 12 13 window.location.href = `/hogepage`; //これを削除するとエラーにならない 14 }, 15 [] 16);

試したこと

色々なページをググりましたが代表するなら以下のようにuseEffectにクリーンアップ処理を入れました。
参考:https://zenn.dev/maktub_bros/articles/3366d053275d60

しかし解決されませんでした。

tsx

1import { useState, useEffect } from 'react' 2 3export default function App() { 4 const [post, setPost] = useState(null) 5 6 useEffect(() => { 7 let isMounted = true 8 const fetchPost = async () => { 9 const res = await fetch("https://jsonplaceholder.typicode.com/posts/1") 10 const post = await res.json() 11 12 if (isMounted) { 13 setPost(post) 14 } 15 } 16 17 fetchPost() 18 19 return () => { 20 isMounted = false 21 } 22 }, []) 23 24 return post ? <div>{post.title}</div> : null 25}

AbortControllerも試しましたが解消されませんでした。
画面遷移をしなければエラーが吐かれないのも気になります。

もしなにかおわかりでしたらアドバイスいただけないでしょうか?

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

StackOverflowに記載されているこれらのやり方は試しましたか?

Can't perform a React state update on an unmounted component

1人目の回答はisMountedを明示的にtrue/falseにして、返す方法
2人目の回答はcomponentDidmountとcomponentWillUnmountを使う方法です。

投稿2021/06/09 08:44

FKM

総合スコア3640

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

k49977

2021/06/09 15:36 編集

2人目の回答でうまくいきました!ありがとうございます
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問