1 回答
TA贡献1865条经验 获得超7个赞
您需要将值存储在状态信息中。实际上,您只是将它们传递给立即返回然后被丢弃的元素的函数。(你也没有fetch
正确处理错误。你不是唯一一个,fetch
API 有一个设计缺陷,鼓励这种脚步,我写在这里。)
看评论:
class Display extends Component {
constructor(props) { // *** Constructor with initial state
super(props);
this.state = {
interestRate: 0, // *** Use appropriate initial values, 0 probably isn't the right choice
monthlyPayment: 0
});
}
// *** SOMETHING needs to call this function. You might do it from componentDidMount, for instance.
calculateAPR = () => {
let x = JSON.parse(localStorage.getItem('amount'));
a=x[0].amount;
t=x[0].years;
fetch("https://ftl-frontend-test.herokuapp.com/interest?amount="+a+"&numMonths="+t)
.then(res => { //
if (!res.ok) { // *** Note the necessary error handling
throw new Error("HTTP error " + res.status); //
} //
return res.json();
})
.then(
(result) => {
this.setState({
interestRate: result.interestRate,
monthlyPayment: result.monthlyPayment.amount
});
},
)
.catch(err => {
// *** Handle/display error here
});
};
// *** You can have these as functions if you want, but since they're pure functions
// it A) Isn't necessary to re-create them for every instance like this, and B) Is
// entirely possible for them to be `static` (or even outside the component and closed
// over).
calculateMonthlyRepayment = (z) => {
return <p>${z}</p>;
};
percentageAPR = (z) => {
return <p>{z * 100}%</p>;
};
render() {
// *** You may want logic here to render differently when you don't have the data yet
return (
<div className="flex">
<DisplayChild func={this.percentageAPR()} text="interest rate" />
<DisplayChild
func={this.calculateMonthlyRepayment()}
text=" monthly repayment"
/>
</div>
);
}
}
添加回答
举报