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

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

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

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

React.js

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

Q&A

解決済

1回答

5158閲覧

map is not a functionというエラー

Kana-T

総合スコア5

JavaScript

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

React.js

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

0グッド

1クリップ

投稿2020/09/06 03:18

編集2020/09/06 04:42

前提・実現したいこと

Reactで計算アプリを作っています。
オブジェクトの配列に対してmapメソッドを使おうとしたら、下記メッセージが出ました。

発生している問題・エラーメッセージ

TypeError: incomeItems.map is not a function

該当のソースコード

IncomeExpense.js

JavaScript

1import React from 'react'; 2 3export const IncomeExpense = (incomeItems, expenseItems ) => { 4 5 const incomeAmounts = incomeItems.map(incomeItem => incomeItem.amount); 6 7 const incomeTotal = incomeAmounts.reduce((acc, cur) => { 8 return acc = acc + cur; 9 }); 10 11 return ( 12 <div> 13 <div> 14 <h4>Income</h4> 15 <p>{incomeTotal}</p> 16 </div> 17 <div> 18 <h4>Expense</h4> 19 <p>-5,000</p> 20 </div> 21 </div> 22 ) 23}

AddItem.js
こちらでstateをsetしています。

JavaScript

1import React, { useState } from 'react'; 2 3export const AddItem = ({ inputText, setInputText, inputAmount, setInputAmount, incomeItems, setIncomeItems, expenseItems, setExpenseItems, type, setType}) => { 4 5 6 const typeHandler = (e) => { 7 setType(e.target.value) 8 console.log(type) 9 } 10 11 const inputTextHandler = (e) => { 12 setInputText(e.target.value); 13 console.log(inputText) 14 }; 15 16 const inputAmountHandler = (e) => { 17 setInputAmount(e.target.value) 18 console.log(inputAmount) 19 } 20 21 const submitItemHandler = (e) => { 22 e.preventDefault(); 23 if ( type === 'inc' ) { 24 setIncomeItems([ 25 ...incomeItems, {text: inputText, amount:inputAmount, id: Math.random() * 1000 } 26 ]); 27 } else if ( type === 'exp' ) { 28 setExpenseItems([ 29 ...expenseItems, {text: inputText, amount:inputAmount, id: Math.random() * 1000 } 30 ]); 31 } 32 console.log(incomeItems) 33 } 34 35 return ( 36 <> 37 <form > 38 <select onChange={typeHandler}> 39 <option value="inc"></option> 40 <option value="exp" >-</option> 41 </select> 42 <label>Text</label> 43 <input type="text" placeholder="Text" value={inputText} onChange={inputTextHandler} /> 44 <label>Amount</label> 45 <input type="number" placeholder="Value" value={inputAmount} onChange={inputAmountHandler} /> 46 <button type="submit" onClick={submitItemHandler}>Add</button> 47 </form> 48 </> 49 ) 50 51}

Home.js

JavaScript

1import React, {useState} from 'react'; 2import { app } from "../firebase/Firebase"; 3import { Header } from './Header'; 4import { Balance } from './Balance'; 5import { IncomeExpense } from './IncomeExpense'; 6import { AddItem } from './AddItem'; 7import { ItemsList } from './ItemsList'; 8 9function Home (props) { 10 11 const [inputText, setInputText] = useState(""); 12 const [inputAmount, setInputAmount] = useState(0); 13 const [incomeItems, setIncomeItems] = useState([]); 14 const [expenseItems, setExpenseItems] = useState([]); 15 const [type, setType] = useState("inc"); 16 17 return ( 18 <div> 19 <Header /> 20 <Balance /> 21 <IncomeExpense 22 incomeItems={incomeItems} 23 expenseItems={expenseItems} 24 /> 25 <AddItem 26 inputText={inputText} 27 setInputText={setInputText} 28 inputAmount={inputAmount} 29 setInputAmount={setInputAmount} 30 incomeItems={incomeItems} 31 setIncomeItems={setIncomeItems} 32 expenseItems={expenseItems} 33 setExpenseItems={setExpenseItems} 34 type={type} 35 setType={setType} 36 /> 37 <ItemsList 38 incomeItems={incomeItems} 39 setIncomeItems={setIncomeItems} 40 expenseItems={expenseItems} 41 setExpenseItems={setExpenseItems} 42 /> 43 <button onClick={() => app.auth().signOut()}>log out</button> 44 </div> 45 ) 46} 47 48export default Home; 49 50

試したこと

メインからincomeItemsをpropsとして渡しています。
オブジェクト配列なのに使えないのは何故でしょうか?

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

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

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

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

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

hentaiman

2020/09/06 03:41

ソースはそれで全部ですか?
miyabi_takatsuk

2020/09/06 04:11

引数、incomeItemsをコンソールで表示してみたりはしましたか?
Kana-T

2020/09/06 04:45 編集

関係するコンポーネントのソースコード全部記載しました。 ItemやItemlistなどは載せていません。 incomeItemsをAdditems.jsでconsole上チェックした時は、リストがArrayとして表示されました。
miyabi_takatsuk

2020/09/06 04:57 編集

ふむ・・・。 記法的に、CLIを使っているような気がするのですが、使っていますか? そして、TypeScript開発ではないでしょうか? であれば、引数の型をしっかり定義せねば、配列として扱ってもらえず、コンパイルでエラーが出る可能性はあります。 もしくは、その引数の元が、非同期取得で、コンソールでは中身を表示できても(console.logは少々処理が遅め)、 即時では中身がまだ入ってないため、処理ができない、とかの可能性です。
Kana-T

2020/09/06 05:13

ありがとうございます! TypeScriptはまだ使ったことはありません(いずれ使いたいです) propsの渡し方で中カッコ{ } がぬけてました。。。 こちらを修正したら動きました。ありがとうございました!
guest

回答1

0

ベストアンサー

export const IncomeExpense = (incomeItems, expenseItems ) => { const incomeAmounts = incomeItems.map(incomeItem => incomeItem.amount);

export const IncomeExpense = (props ) => { const incomeItems = props.incomeItems const expenseItems = props.expenseItems const incomeAmounts = incomeItems.map(incomeItem => incomeItem.amount);

※未検証

投稿2020/09/06 04:53

編集2020/09/06 04:54
hentaiman

総合スコア6421

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

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

Kana-T

2020/09/06 05:11

propsの渡し方で、({ ))中カッコを忘れていた為でした!! 解決しました・・ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問