英文:
Want to update 2 input state values dynamically without recursive calls
问题
I am making a React application for a currency converter website. I have two <input>
fields where the user can input the number to get the exchange rate as shown here:
Screenshot of Component
The <input>
fields are as follows:
<>
<input
id="1"
placeholder="0"
type="number"
value={state1.value}
onChange={handleChange}
disabled={loading}
/>
// rest of the component
<input
id="2"
placeholder="0"
type="number"
value={state2.value}
onChange={handleChange}
disabled={loading}
/>
The states for the input fields are:
const [state1, setState1] = useState({
value: '',
currency: 'USD',
src: 'https://hatscripts.github.io/circle-flags/flags/us.svg',
});
const [state2, setState2] = useState({
value: '',
currency: 'INR',
src: 'https://hatscripts.github.io/circle-flags/flags/in.svg',
});
I want the state2
to be updated when the user types in state1
. And vice-versa. I don't want the auto-update of state2
again changing state1
. The state change should happen only when the user gives input.
I have tried using session state to avoid unnecessary API calls. It solves the problem except for the infinite recursion.
I have already written a useEffect
for state1
which works as expected. Now I want to do the same for state2
without an infinite useEffect loop.
async function fetchCurrencyRate1(from, to) {
setLoading(true);
try {
const response = await axios.get('http://localhost:4000/convert', {
params: {
from: from.currency,
to: to.currency,
},
});
setState2((state2) => ({
...state2,
value: (from.value * response.data.value).toFixed(4),
}));
console.log(response.data);
} catch (error) {
console.error(error);
}
setLoading(false);
}
useEffect(() => {
setTimeout(() => {
state1.value && fetchCurrencyRate1(state1, { ...state2 });
if (state1.value === '') setState2({ ...state2, value: '' });
}, 2000);
}, [state1]);
英文:
I am making a React application for a currency converter website. I have two <input>
fields where the user can input the number to get the exchange rate as shown here:
Screenshot of Component
The <input>
fields are as follows:
<>
<input
id="1"
placeholder="0"
type="number"
value={state1.value}
onChange={handleChange}
disabled={loading}
/>
// rest of the component
<input
id="2"
placeholder="0"
type="number"
value={state2.value}
onChange={handleChange}
disabled={loading}
/>
The states for the input fields are:
const [state1, setState1] = useState({
value: '',
currency: 'USD',
src: 'https://hatscripts.github.io/circle-flags/flags/us.svg',
});
const [state2, setState2] = useState({
value: '',
currency: 'INR',
src: 'https://hatscripts.github.io/circle-flags/flags/in.svg',
});
I want the state2
to be updated when the user types in state1
. And vice-versa. I don't want the auto-update of state2
again changing state1
. The state change should happen only when the user gives input.
I have tried using session state to avoid unnecessary API calls. It solves the problem except for the infinite recursion.
I have already written a useEffect
for state1
which works as expected. Now I want to do the same for state2
without an infinite useEffect loop.
async function fetchCurrencyRate1(from, to) {
setLoading(true);
try {
const response = await axios.get('http://localhost:4000/convert', {
params: {
from: from.currency,
to: to.currency,
},
});
setState2((state2) => ({
...state2,
value: (from.value * response.data.value).toFixed(4),
}));
console.log(response.data);
} catch (error) {
console.error(error);
}
setLoading(false);
}
useEffect(() => {
setTimeout(() => {
state1.value && fetchCurrencyRate1(state1, { ...state2 });
if (state1.value === '') setState2({ ...state2, value: '' });
}, 2000);
}, [state1]);
答案1
得分: 0
无限渲染发生是因为state1
的useEffect
每次state1
取新值时都会触发,由于在effect内部更新了state2
,这会触发state2
的useEffect
,而那里的代码会更新state1
,然后state1
的useEffect
将被无限调用,因为state1
是一个对象,所以其值总是新的。
你不需要为此使用useEffect
,而且你也不希望在每次输入更改时组件重渲染两次,你可以简单地为每个输入创建两个单独的handleChange
函数,并在其中的每一个中进行逻辑处理,类似于这样:
const handleChangeOne = async (e) => {
// 在这里你可以获取数据
setState1((prev) => {
return {
...prev,
value: e.target.value,
};
});
setState2((prev) => {
return {
...prev,
value: (from.value * response.data.value).toFixed(4),
};
});
};
const handleChangeTwo = async (e) => {
// 获取数据
setState2((prev) => {
return {
...prev,
value: Number(e.target.value),
};
});
setState1((prev) => {
//
return {
...prev,
value: (from.value * response.data.value).toFixed(4),
};
});
};
希望这可以帮助你解决问题。
英文:
The infinite render occurs because the useEffect
of state1
will fire each time state1
take a new value and since inside the effect you update state2
and this triggers useEffect
of state2
and the code there updates state1
then useEffect
of state1
will be called infinitely because state1
is an object so its value is always new.
you don't need useEffect
for that, also you don't want to rerender the component twice on each input change you can simply make two separate handleChange
functions for each input and make your logic in each one of them <br>something like this:
const handleChangeOne = async (e) => {
// you can fetch data here instead
setState1((prev) => {
return {
...prev,
value: e.target.value,
};
});
setState2((prev) => {
return {
...prev,
value: (from.value * response.data.value).toFixed(4),
};
});
};
const handleChangeTwo = async (e) => {
// fetch data
setState2((prev) => {
return {
...prev,
value: Number(e.target.value),
};
});
setState1((prev) => {
//
return {
...prev,
value: (from.value * response.data.value).toFixed(4),
};
});
};
通过集体智慧和协作来改善编程学习和解决问题的方式。致力于成为全球开发者共同参与的知识库,让每个人都能够通过互相帮助和分享经验来进步。
评论