getting error in react: Line 25: 'totalIncome' is not defined no-undefLoop inside React JSXProgrammatically navigate using react routerReact and Semantic-ui-reactHow get value datapicker in react toobox custom?How to get props value of boolean type in reactUpdate React Grandparent State from onClick in GrandchildReactJS. How to get states of elements inside react componentReact-vis + local CSV: Error in creating data visualizationTrying to make calculations while passing props and functions through different components REACTTrying to pass a function back to App.js and getting stuck REACT

Echo with obfuscation

How can I, as DM, avoid the Conga Line of Death occurring when implementing some form of flanking rule?

How much do grades matter for a future academia position?

Why would five hundred and five be same as one?

How to write Quadratic equation with negative coefficient

Would this string work as string?

How to make money from a browser who sees 5 seconds into the future of any web page?

"Oh no!" in Latin

How do I prevent inappropriate ads from appearing in my game?

If A is dense in Q, then it must be dense in R.

Why the "ls" command is showing the permissions of files in a FAT32 partition?

Typing CO_2 easily

Unable to disable Microsoft Store in domain environment

Possible Eco thriller, man invents a device to remove rain from glass

Limit max CPU usage SQL SERVER with WSRM

What does "tick" mean in this sentence?

What is the smallest number n> 5 so that 5 ^ n ends with "3125"?

Storage of electrolytic capacitors - how long?

How to I force windows to use a specific version of SQLCMD?

How to get directions in deep space?

Origin of pigs as a species

Anime with legendary swords made from talismans and a man who could change them with a shattered body

If the only attacker is removed from combat, is a creature still counted as having attacked this turn?

How can ruler support inventing of useful things?



getting error in react: Line 25: 'totalIncome' is not defined no-undef


Loop inside React JSXProgrammatically navigate using react routerReact and Semantic-ui-reactHow get value datapicker in react toobox custom?How to get props value of boolean type in reactUpdate React Grandparent State from onClick in GrandchildReactJS. How to get states of elements inside react componentReact-vis + local CSV: Error in creating data visualizationTrying to make calculations while passing props and functions through different components REACTTrying to pass a function back to App.js and getting stuck REACT













0















import React, Component from 'react'

class CalculateIncome extends Component
state =
totalIncome:0


calculation = () =>
this.props.lessons.filter(lesson =>
this.props.clients.filter(client =>
if (client.name === lesson.name)
let income = client.price * lesson.time;
let totalIncome = this.state.totalIncome + income
return(
totalIncome
)

return (
this.setState(
totalIncome : totalIncome //this is the error
)
)
)
return (this.state.totalIncome)
)


render()
console.log(this.props.lessons, this.props.clients) //this checks out!
return ( this.props.lessons.length ?
(
<div onLoad=this.calculation>this.state.totalIncome</div>
)
:
(<div className="text-center my-5">You are broke</div>)
)



export default CalculateIncome


The props from App.js are loading in correctly because I can see them showing up when I do console.log();



The overall purpose of this calculation is to take the added new lesson and run it through the list of existing clients and multiply the time or duration of the lesson with the price that is listed with the client so I can get my income. Then I want to add that individual income to the total income and display it.



I am either getting an undefined error or if I move things around I get a "needs a return value" error. I am new at react so I am very confused. Any help would be appreciated, thanks! If you need more of my components, please let me know.










share|improve this question






















  • move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

    – Shivam Gupta
    Mar 7 at 22:02











  • i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

    – Edison Junior
    Mar 7 at 22:03












  • @EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

    – josephT
    Mar 7 at 22:38















0















import React, Component from 'react'

class CalculateIncome extends Component
state =
totalIncome:0


calculation = () =>
this.props.lessons.filter(lesson =>
this.props.clients.filter(client =>
if (client.name === lesson.name)
let income = client.price * lesson.time;
let totalIncome = this.state.totalIncome + income
return(
totalIncome
)

return (
this.setState(
totalIncome : totalIncome //this is the error
)
)
)
return (this.state.totalIncome)
)


render()
console.log(this.props.lessons, this.props.clients) //this checks out!
return ( this.props.lessons.length ?
(
<div onLoad=this.calculation>this.state.totalIncome</div>
)
:
(<div className="text-center my-5">You are broke</div>)
)



export default CalculateIncome


The props from App.js are loading in correctly because I can see them showing up when I do console.log();



The overall purpose of this calculation is to take the added new lesson and run it through the list of existing clients and multiply the time or duration of the lesson with the price that is listed with the client so I can get my income. Then I want to add that individual income to the total income and display it.



I am either getting an undefined error or if I move things around I get a "needs a return value" error. I am new at react so I am very confused. Any help would be appreciated, thanks! If you need more of my components, please let me know.










share|improve this question






















  • move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

    – Shivam Gupta
    Mar 7 at 22:02











  • i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

    – Edison Junior
    Mar 7 at 22:03












  • @EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

    – josephT
    Mar 7 at 22:38













0












0








0








import React, Component from 'react'

class CalculateIncome extends Component
state =
totalIncome:0


calculation = () =>
this.props.lessons.filter(lesson =>
this.props.clients.filter(client =>
if (client.name === lesson.name)
let income = client.price * lesson.time;
let totalIncome = this.state.totalIncome + income
return(
totalIncome
)

return (
this.setState(
totalIncome : totalIncome //this is the error
)
)
)
return (this.state.totalIncome)
)


render()
console.log(this.props.lessons, this.props.clients) //this checks out!
return ( this.props.lessons.length ?
(
<div onLoad=this.calculation>this.state.totalIncome</div>
)
:
(<div className="text-center my-5">You are broke</div>)
)



export default CalculateIncome


The props from App.js are loading in correctly because I can see them showing up when I do console.log();



The overall purpose of this calculation is to take the added new lesson and run it through the list of existing clients and multiply the time or duration of the lesson with the price that is listed with the client so I can get my income. Then I want to add that individual income to the total income and display it.



I am either getting an undefined error or if I move things around I get a "needs a return value" error. I am new at react so I am very confused. Any help would be appreciated, thanks! If you need more of my components, please let me know.










share|improve this question














import React, Component from 'react'

class CalculateIncome extends Component
state =
totalIncome:0


calculation = () =>
this.props.lessons.filter(lesson =>
this.props.clients.filter(client =>
if (client.name === lesson.name)
let income = client.price * lesson.time;
let totalIncome = this.state.totalIncome + income
return(
totalIncome
)

return (
this.setState(
totalIncome : totalIncome //this is the error
)
)
)
return (this.state.totalIncome)
)


render()
console.log(this.props.lessons, this.props.clients) //this checks out!
return ( this.props.lessons.length ?
(
<div onLoad=this.calculation>this.state.totalIncome</div>
)
:
(<div className="text-center my-5">You are broke</div>)
)



export default CalculateIncome


The props from App.js are loading in correctly because I can see them showing up when I do console.log();



The overall purpose of this calculation is to take the added new lesson and run it through the list of existing clients and multiply the time or duration of the lesson with the price that is listed with the client so I can get my income. Then I want to add that individual income to the total income and display it.



I am either getting an undefined error or if I move things around I get a "needs a return value" error. I am new at react so I am very confused. Any help would be appreciated, thanks! If you need more of my components, please let me know.







html reactjs components jsx






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Mar 7 at 21:50









josephTjosephT

165




165












  • move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

    – Shivam Gupta
    Mar 7 at 22:02











  • i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

    – Edison Junior
    Mar 7 at 22:03












  • @EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

    – josephT
    Mar 7 at 22:38

















  • move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

    – Shivam Gupta
    Mar 7 at 22:02











  • i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

    – Edison Junior
    Mar 7 at 22:03












  • @EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

    – josephT
    Mar 7 at 22:38
















move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

– Shivam Gupta
Mar 7 at 22:02





move your calculation() to componentDidMount(), see react lifecycle methods.(reactjs.org/docs/react-component.html#componentdidmount) What you have to do is 1. get props from app.js 2. perform calcuation (CdM()) 3. render based on calculation Error is : You have return this.setState(xyz). But the method does not return anything.

– Shivam Gupta
Mar 7 at 22:02













i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

– Edison Junior
Mar 7 at 22:03






i dont understand what you want todo with "this.props.clients.filter", maybe you should to use a reduce to return only one value. anyway, i think your "totalIncome" in setState on calculator is not define.

– Edison Junior
Mar 7 at 22:03














@EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

– josephT
Mar 7 at 22:38





@EdisonJunior My thought process was filtering through each lesson and for each lesson filter through the list of existing clients until I find the one that matches the same name. There I would do the math and then send the answer back and add it to totalIncome.

– josephT
Mar 7 at 22:38












1 Answer
1






active

oldest

votes


















0














I don't think you need to be using state for this calculation at all (if you did it the way you're trying, your component will re-render for every client-lesson match. Here's a quick (untested) way you could do it.



import React, PureComponent from 'react'

class CalculateIncome extends PureComponent
calculation = () =>
const lessons = [], clients = [] = this.props;
let totalIncome = 0;
lessons.forEach(lesson =>
clients.forEach(client =>
if (client.name === lesson.name)
totalIncome += client.price * lesson.time;

);
);
return totalIncome;


render()
const lessons = this.props;

if (lessons.length)
const totalIncome = this.calculation();
return <div>totalIncome</div>;

return <div className="text-center my-5">You are broke</div>;
)



export default CalculateIncome;





share|improve this answer

























  • oh jeez, this worked, thank you so much! I had been stuck on this for two days.

    – josephT
    Mar 8 at 15:46










Your Answer






StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");

StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "1"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);













draft saved

draft discarded


















StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f55053360%2fgetting-error-in-react-line-25-totalincome-is-not-defined-no-undef%23new-answer', 'question_page');

);

Post as a guest















Required, but never shown

























1 Answer
1






active

oldest

votes








1 Answer
1






active

oldest

votes









active

oldest

votes






active

oldest

votes









0














I don't think you need to be using state for this calculation at all (if you did it the way you're trying, your component will re-render for every client-lesson match. Here's a quick (untested) way you could do it.



import React, PureComponent from 'react'

class CalculateIncome extends PureComponent
calculation = () =>
const lessons = [], clients = [] = this.props;
let totalIncome = 0;
lessons.forEach(lesson =>
clients.forEach(client =>
if (client.name === lesson.name)
totalIncome += client.price * lesson.time;

);
);
return totalIncome;


render()
const lessons = this.props;

if (lessons.length)
const totalIncome = this.calculation();
return <div>totalIncome</div>;

return <div className="text-center my-5">You are broke</div>;
)



export default CalculateIncome;





share|improve this answer

























  • oh jeez, this worked, thank you so much! I had been stuck on this for two days.

    – josephT
    Mar 8 at 15:46















0














I don't think you need to be using state for this calculation at all (if you did it the way you're trying, your component will re-render for every client-lesson match. Here's a quick (untested) way you could do it.



import React, PureComponent from 'react'

class CalculateIncome extends PureComponent
calculation = () =>
const lessons = [], clients = [] = this.props;
let totalIncome = 0;
lessons.forEach(lesson =>
clients.forEach(client =>
if (client.name === lesson.name)
totalIncome += client.price * lesson.time;

);
);
return totalIncome;


render()
const lessons = this.props;

if (lessons.length)
const totalIncome = this.calculation();
return <div>totalIncome</div>;

return <div className="text-center my-5">You are broke</div>;
)



export default CalculateIncome;





share|improve this answer

























  • oh jeez, this worked, thank you so much! I had been stuck on this for two days.

    – josephT
    Mar 8 at 15:46













0












0








0







I don't think you need to be using state for this calculation at all (if you did it the way you're trying, your component will re-render for every client-lesson match. Here's a quick (untested) way you could do it.



import React, PureComponent from 'react'

class CalculateIncome extends PureComponent
calculation = () =>
const lessons = [], clients = [] = this.props;
let totalIncome = 0;
lessons.forEach(lesson =>
clients.forEach(client =>
if (client.name === lesson.name)
totalIncome += client.price * lesson.time;

);
);
return totalIncome;


render()
const lessons = this.props;

if (lessons.length)
const totalIncome = this.calculation();
return <div>totalIncome</div>;

return <div className="text-center my-5">You are broke</div>;
)



export default CalculateIncome;





share|improve this answer















I don't think you need to be using state for this calculation at all (if you did it the way you're trying, your component will re-render for every client-lesson match. Here's a quick (untested) way you could do it.



import React, PureComponent from 'react'

class CalculateIncome extends PureComponent
calculation = () =>
const lessons = [], clients = [] = this.props;
let totalIncome = 0;
lessons.forEach(lesson =>
clients.forEach(client =>
if (client.name === lesson.name)
totalIncome += client.price * lesson.time;

);
);
return totalIncome;


render()
const lessons = this.props;

if (lessons.length)
const totalIncome = this.calculation();
return <div>totalIncome</div>;

return <div className="text-center my-5">You are broke</div>;
)



export default CalculateIncome;






share|improve this answer














share|improve this answer



share|improve this answer








edited Mar 7 at 23:27

























answered Mar 7 at 23:21









Gregory FriedmanGregory Friedman

563




563












  • oh jeez, this worked, thank you so much! I had been stuck on this for two days.

    – josephT
    Mar 8 at 15:46

















  • oh jeez, this worked, thank you so much! I had been stuck on this for two days.

    – josephT
    Mar 8 at 15:46
















oh jeez, this worked, thank you so much! I had been stuck on this for two days.

– josephT
Mar 8 at 15:46





oh jeez, this worked, thank you so much! I had been stuck on this for two days.

– josephT
Mar 8 at 15:46



















draft saved

draft discarded
















































Thanks for contributing an answer to Stack Overflow!


  • Please be sure to answer the question. Provide details and share your research!

But avoid …


  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.

To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f55053360%2fgetting-error-in-react-line-25-totalincome-is-not-defined-no-undef%23new-answer', 'question_page');

);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

Thal And Out Agency railway station See also References External links Navigation menuOfficial Web Site of Pakistan RailwaysArchivedOfficial Web Site of Pakistan Railwayseeexpanding ite

Posting a File and Associated Data to a RESTful WebService preferably as JSON2019 Community Moderator ElectionREST URI convention - Singular or plural name of resource while creating itUploading both data and files in one form using Ajax?How do I upload a file with metadata using a REST web service?REST API - file (ie images) processing - best practicesSpring: JSON data and file in the same requestREST API Put large dataPosting a file and JSON data to Spring rest serviceRESTful Web Service Upload/Download Large Data With JSONREST API Design sending JSON data and a file to the api in same requestPost JSONArray to REST servicePUT vs. POST in RESTParsing values from a JSON file?JavaScript/jQuery to download file via POST with JSON dataHow do I upload a file with metadata using a REST web service?How to POST JSON data with Curl from Terminal/Commandline to Test Spring REST?Separate REST JSON API server and client?what's the correct way to send a file from REST web service to client?How do I write JSON data to a file?What is “406-Not Acceptable Response” in HTTP?REST API - file (ie images) processing - best practices

How can I change the color of pagination dots of UIPageControl?How to change UIPageControl dotsIs there a way to change page indicator dots colorCustomize dot with image of UIPageControl at index 0 of UIPageControlNo visible @interface for 'NSObject<PageControlDelegate>' declares the selector 'pageControlPageDidChange:'How to change the color of pagination dots in UIPageControl with a different color per pagepagecontrol indicator custom image instead of DefaultChanging the colour of UIPageControl dots in MonoTouchpagecontrol selectable page visibility color?Alternative way to load ViewControllers on a UIPageControlHow to set only layer.border-color for UIpage control dots in swiftHow can I develop for iPhone using a Windows development machine?How to change the name of an iOS app?UITableView - change section header coloruipagecontrol indicator(dot)issueCustom UIPageControl dots color not changingchange the interspace between UIPageControl dotsHow to change Status Bar text color in iOSHow can I change image tintColor in iOS and WatchKitUIPageControl dots with larger space in between each dotsHow to change the color of pagination dots in UIPageControl with a different color per page