我正在尝试从 django 数据库中检索对象(电机)列表并将其存储在我的 MachineTemplate 组件的状态中。
这是组件
export default class MachineTemplate extends Component {
constructor(props) {
super(props);
this.state = {
myName: this.props.match.params.id,
motors: {},
error: "None",
loaded: false
}
this.getMachineDetails = this.getMachineDetails.bind(this);
}
componentDidMount() {
this.getMachineDetails();
}
getMachineDetails = () => {
const requestOptions = {
method: "POST",
headers: { "Content-Type": "application/json" },
body: JSON.stringify({
name: this.state.myName,
}),
};
fetch("http://127.0.0.1:8000/get-motors", requestOptions)
.then((response) => {
if (response.ok) {
this.setState({ motors: response.data });
} else {
this.setState({ error: "Motors not found." });
}
})
.catch((error) => {
console.log(error);
});
this.setState({loaded: true});
}
render() {
if (this.state.loaded) {
return (
<div>
<h1>{this.state.name}</h1>
<h3>{this.state.error}</h3>
{this.state.motors.map(motor => {
return <Button key = {motor.number}>{motor.number}</Button>
})}
</div>
)
} else {
return(
<div>
<h1>Awaiting machine details...</h1>
</div>
)
}
}
}
这是我得到的错误
TypeError: Cannot read property 'map' of undefined
我可以确认 fetch 正确返回了我需要的电机对象列表,这些对象在响应中的格式如下:
0: {number: 0, enabled: true, item: "EMPTY", machine: 48}
我能够使用不同组件中的简单名称列表使这个确切的设置正常工作,因此我们将不胜感激。
答案 0 :(得分:2)
您在构造函数中为 this.state.motors 分配了一个空对象。这就是您收到错误的原因。应该是一个空数组。
答案 1 :(得分:1)
fetch()
是异步。这意味着你不能依赖它在下一条语句之前完成它的执行:
fetch(/**/);
// ...
this.setState({loaded: true}) // fetch() might not finish executing when this statement is executed
因此,您必须确保将 loaded
状态变量设置在正确的位置。看看你的代码,这样的事情应该可以工作:
fetch("http://127.0.0.1:8000/get-motors", requestOptions)
.then((response) => {
if (response.ok) {
this.setState({
motors: response.json(), // As suggested by @buzatto
loaded: true
});
} else {
this.setState({ error: "Motors not found." });
}
})
.catch((error) => {
console.log(error);
});
考虑添加 componentDidUpdate
以使用有关道具更改的新数据重新获取和重新渲染您的组件:
componentDidUpdate(prevProps) {
// Typical usage (don't forget to compare props):
if (this.props.match.params.id !== prevProps.props.match.params.id) {
this.setState({loaded: false})
this.getMachineDetails();
}
}
还可以看看 useEffect
钩子,它可以简化您的代码,在 function 组件中合并 componentDidMount
和 componentDidUpdate
的功能:
useEffect(() => {
setState({loaded: false})
getMachineDetails();
}, [props.match.params.id]); // Only re-fetch if props.match.params.id changes