axiosで取得した結果をReactでsetStateしたいが、初回のみうまくセットされない。

前提

Reactでaxiosを使ってAPIの情報を取得
responsの中の必要な情報を連想配列にしてstateに追加したい。
が、1回目の取得時のみ追加されない。

実現したいこと

飲食店の情報を取得できるAPIを使い、
必要な店舗の情報のみを抜き出して連想配列にまとめたい。

JavaScript（React）

以下のようにsetShoplistで店舗に紐づく情報を1店舗ずつ取得したいのですが、
finallyでconsole.log(shopList);すると[]となり、情報が取得できていませんでした。
取得ができないのは画面更新して1回目の時のみで、２回目は取得できます。

取得件数が0件でないことは確認済みです。

const [shopList, setShoplist] = useState([]);
const getShopList = () => {
    axios.get(requestUrl)
    .then(function (response) {
        // handle success
        const responseShopList = response.data.results.shop;
        for (const item of responseShopList) {
        //ここでうまく情報をセットできていない感じがする
        setShoplist((prevState) => [...prevState,
        {
            id: item.id,
            photoPcM: item.photo.pc.m,
            shopName: item.name,
            lunch: item.lunch,
            budgetName: item.budget.name,
            large_areaName: item.large_area.name,
            Access: item.access
        }
        ]);
        }
    })
    .catch(function (error) {
        // handle error
    })
    .finally(function () {
        // always
        console.log(shopList);
    });
}

最終的にshopListの値が以下のようになるのが理想です。

json
1[
2    //1店舗目の情報
3    {
4        id:"XXXX",
5        photoPcM:"XXXX",
6        shopName:"XXXX",
7        lunch:"XXXX",
8        budgetName:"XXXX",
9        large_areaName:"XXXX",
10        Access:"XXXX"
11    },
12    //2店舗目の情報
13    {
14        id:"XXXX",
15        photoPcM:"XXXX",
16        shopName:"XXXX",
17        lunch:"XXXX",
18        budgetName:"XXXX",
19        large_areaName:"XXXX",
20        Access:"XXXX"
21    }...
22]

console.log(shopList);すると[]となり、情報が取得できていませんでした。

この原因がわかる方、よろしくお願いします。

行動規範の内容に同意します

回答2件

ベストアンサー

then のコールバックの中でsetShoplist したことによってstateの変更が即座に反映されて変更後のshopList が finally のコールバックの中ですぐに確認できるわけではないです。

ですので、console.logしている行を、質問にあるコードが記載されたコンポーネントの中で JSXをreturnする直前に移動してみるといかがでしょうか？

サンプル的なコードで説明すると以下のような感じの移動です。

diff
1       .finally(function () {
2         // always
3-        console.log(shopList);
4       });
5   }
6 
7   useEffect(getShopList, []);
8 
9+  console.log(shopList);
10+
11   return (
12     <div className="shop-list">

補足

質問にあるコードだとレスポンスに含まれる配列 response.data.results.shopの含む要素の個数分、forループが回ってその回数だけ setShoplist でstate更新をしてしまうことになりますが、それを避けるため以下のようにするとよいかと思います。

diff
1-        const responseShopList = response.data.results.shop;
2-        for (const item of responseShopList) {
3-          //ここでうまく情報をセットできていない感じがする
4-          setShoplist((prevState) => [...prevState,
5-            {
6-              id: item.id,
7-              photoPcM: item.photo.pc.m,
8-              shopName: item.name,
9-              lunch: item.lunch,
10-              budgetName: item.budget.name,
11-              large_areaName: item.large_area.name,
12-              Access: item.access
13-            }
14-          ]);
15-        }
16+        const responseShopList = response.data.results.shop.map(item => (
17+          {
18+            id: item.id,
19+            photoPcM: item.photo.pc.m,
20+            shopName: item.name,
21+            lunch: item.lunch,
22+            budgetName: item.budget.name,
23+            large_areaName: item.large_area.name,
24+            Access: item.access
25+          }
26+        ));
27+        setShoplist((prevState) => [...prevState, ...responseShopList]);