2

我正在尝试为注册表单设置错误消息,但因为我使用的是材料 UI,所以我必须为我的项目使用功能组件。我正在关注 youtube 上的教程,但那个人正在使用类组件。我设法将大部分代码转换为功能组件,但我迷失在 componentDidUpdate 函数中。

我也尝试使用 useEffect 功能,但无济于事,当有人单击提交按钮时,如果他们没有输入任何字段,我无法显示警报/消息错误。我还收到一个错误,即 msg 未在 SignUp 组件的 Alert 组件中定义,尽管它已在 useState 函数中定义。

{msg ? <Alert color="danger">{msg}</Alert> : null}

下面是 SignUp 组件的代码

import React, { useState, useEffect } from "react";
import Avatar from "@material-ui/core/Avatar";
import Button from "@material-ui/core/Button";
import CssBaseline from "@material-ui/core/CssBaseline";
import TextField from "@material-ui/core/TextField";
import Link from "@material-ui/core/Link";
import Grid from "@material-ui/core/Grid";
import Box from "@material-ui/core/Box";
import Typography from "@material-ui/core/Typography";
import { makeStyles } from "@material-ui/core/styles";
import Container from "@material-ui/core/Container";
import Logo from "./assets/images/logo_transparent.png";
import { Alert } from "reactstrap";


//redux
import { connect } from "react-redux";
//proptypes
import PropTypes from "prop-types";
import { register } from "./actions/authActions";


const useStyles = makeStyles(theme => ({
//styles- for brevity only
})); 

function SignUp(props) {
  const classes = useStyles();
  const [form, setValues] = useState({
    name: "",
    email: "",
    password: "",
    msg: null
  });

  // Similar to componentDidMount and componentDidUpdate
  //By running an empty array [] as a second argument,
  //we’re letting React know that the useEffect function doesn’t
  //depend on any values from props or state.
  useEffect(() => {
    const { error } = props;
    if (error !== form.error) {
      //check for register error
      if (error.id === "REGISTER_FAIL") {
        setValues({ msg: error.msg.msg });
      } else {
        setValues({ msg: null });
      }
    }
  }, []);

  const onChange = e => {
    setValues({
      ...form,
      [e.target.name]: e.target.value,
      [e.target.email]: e.target.value,
      [e.target.password]: e.target.value
    });
  };

  const handleClick = e => {
    e.preventDefault();
    const { name, email, password } = form;

    //create user object
    const newUser = {
      name,
      email,
      password
    };

    //attempt to register
    props.register(newUser);

    window.confirm("Registration details: " + name + " " + email);
    //window.location.href = "http://localhost:3000/";
  };

  return (
    <Container component="main" maxWidth="xs">
      <CssBaseline />

      <div className={classes.paper}>
        {msg ? <Alert color="danger">{msg}</Alert> : null}
        <form className={classes.form} noValidate>
          <Grid container spacing={2}>
            <Grid item xs={12}>
              <TextField
                autoComplete="name"
                name="name"
                variant="outlined"
                required
                fullWidth
                id="name"
                label="Full Name"
                autoFocus
                onChange={onChange}
              />
            </Grid>
            <Grid item xs={12}>
              <TextField
                variant="outlined"
                required
                fullWidth
                id="email"
                label="Email Address"
                name="email"
                autoComplete="email"
                onChange={onChange}
              />
            </Grid>
            <Grid item xs={12}>
              <TextField
                variant="outlined"
                required
                fullWidth
                name="password"
                label="Password"
                type="password"
                autoComplete="current-password"
                onChange={onChange}
              />
            </Grid>
            <Grid item xs={12}>
              <TextField
                variant="outlined"
                required
                fullWidth
                name="cpassword"
                label="Confirm Password"
                type="password"
                id="cpassword"
                autoComplete="confirm-password"
              />
            </Grid>
            <Grid item xs={12} />
          </Grid>
          <Button
            type="submit"
            fullWidth
            variant="contained"
            color="primary"
            className={classes.submit}
            onClick={handleClick}
          >
            Sign Up
          </Button>

          <Button
            href="http://localhost:3000/"
            fullWidth
            variant="contained"
            color="primary"
            className={classes.home}
          >
            Home
          </Button>
          <br />
          <br />

          <Grid container justify="flex-end">
            <Grid item>
              <Link href="http://localhost:3000/signin" variant="body2">
                Already have an account? Sign in
              </Link>
            </Grid>
          </Grid>
        </form>
      </div>
      <Box mt={5}>
      </Box>
    </Container>
  );
}

SignUp.propTypes = {
  isAuthenticated: PropTypes.bool,
  error: PropTypes.object.isRequired,
  register: PropTypes.func.isRequired
};

const mapStateToProps = state => ({
  isAuthenticated: state.auth.isAuthenticated,
  error: state.error //getting from reducer
});

export default connect(
  mapStateToProps,
  { register } //from redux actions //mapdispatchtoprop
)(SignUp); //component 

这是原来的 componentDidUpdate 函数

componentDidUpdate(prevProps) {
  const {error} = this.props;
  if(error !== prevProps.error){
    //check for register error
    if(error.id === "REGISTER_FAIL"){
      this.setState({msg:error.msg.msg});
    }else {
        setValues({ msg: null });
      }
  }
}
4

3 回答 3

3

当您将其useEffect视为在使用“副作用”(更改状态)时运行的函数时componentDidUpdate,您需要注意errormsg状态的变化。

代替

  // Similar to componentDidMount and componentDidUpdate
  //By running an empty array [] as a second argument,
  //we’re letting React know that the useEffect function doesn’t
  //depend on any values from props or state.
  useEffect(() => {
    const { error } = props;
    if (error !== form.error) {
      //check for register error
      if (error.id === "REGISTER_FAIL") {
        setValues({ msg: error.msg});
      } else {
        setValues({ msg: null });
      }
    }
  }, []);

你需要做

  useEffect(() => {
    const { error } = props;
    if (error !== form.error) {
      //check for register error
      if (error.id === "REGISTER_FAIL") {
-        setValues(({ msg: error.msg});
+       setValues({ ...form, msg: error.msg});
      } else {
-       setValues({ msg: null });
+       setValues({ ...form, msg: null });
      }
    }
  }, [error, form.msg]);

我传播的原因...formsetValues({ ...form, msg: error.msg.msg })因为返回的更新器函数(第二个参数)React.useState不会更新状态中的其他属性。

所以setValues({ msg: null });form

{
  name: 'previous name',
  password: 'previous password',
  email: 'previous email',
  msg: 'previous message...'
}

删除{msg: null}属性name, email, password

由于's 更新程序(如React 文档React.useState的“别名” )之间的行为不同,最终有人创建了use-legay-state NPM 包,除非绝对必要,否则您可能不应该使用它。setStatesetState

边注

与您的问题无关,您还希望 1.name/email/password/msg分成单独的状态 2. “或”使用useReducer钩子。

案例 1 的实施变更

const [name, setName] = useState('');
const [email, setEmail] = useState('');
const [password, setPassword] = useState('');
const [message, setMessage] = useState('');

如果您想继续使用一个对象,您可能希望通过更改为orform来遵循命名约定。setValuessetFormconst [formValues, setFormValues] = useState({...})

您可以更新每个字段,例如,

<TextField
  autoComplete="name"
  name="name"
  variant="outlined"
  required
  fullWidth
  id="name"
  label="Full Name"
  autoFocus
  value={name}
  onChange={e => setName(e.target.value)}
/>

您可以在此处使用useCallback,但除非onChangeonChange={useCallback(e => setName(e.target.value), [name])}遇到性能问题,否则不是必需的。最初从简单开始

而且它只会让你useEffect依赖msg

  useEffect(() => {
    const { error } = props;
    if (error !== form.error) {
      //check for register error
      if (error.id === "REGISTER_FAIL") {
       setMessage(error.msg);
      } else {
       setMessage(null);
      }
    }
  }, [error, message]);

在这里,我将依赖数组从to更改setValues为。setMessage[error, form][error, message]

案例 2 的实施变更

如果您想使用useReducer,因为您正在更改单独的表单字段,这似乎需要更多的工作,这些表单字段是单独更改的。useReducer当您更新应该一起更改的相关状态集时,效果会更好。但是在您的情况下,每个状态(password/name等)都会随着用户输入而独立更改,因此从上面的案例 #1 开始会更容易。

于 2019-06-23T12:45:37.097 回答
1

我创建了一个带有表单字段验证和功能组件的最小示例。请注意,您可以将任何组件编写为函数类。我个人更喜欢函数,因为它们通常更简洁。

在此处运行示例: https ://codesandbox.io/s/zen-antonelli-noovi?fontsize=14

import React, { useState } from "react";
import ReactDOM from "react-dom";

const App = () => {
  const [error, setError] = useState(null);
  return (
    <div>
     {error && <div>{error}</div>}
      <form>
        <input
          type="email"
          onInvalid={() => setError("That is not an email.")}
          required
        />
        <input type="submit" />
      </form>
    </div>
  );
};

const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

于 2019-06-23T12:33:01.720 回答
1

useEffect带有空依赖项数组的行为不会componentDidUpdate,它只会在组件第一次挂载时运行,而不会在组件时运行re-renders

模仿componentDidUpdate提供error作为依赖。

const [errorMsg, setErrorMsg] = useState('');

useEffect(() => {
  if (error.id === "REGISTER_FAIL") {
    setErrorMsg(props.error.msg.msg);
  }
}, [props.error])

return (
  <div className="App">
  {errorMsg && <span>{errorMsg}</span>}
  </div>
);

我还建议您每次提交表单以调度操作并将错误消息重置为''商店中的虚假值

这样每次提交时您都不会看到最后一个error,也不需要比较errors

于 2019-06-23T12:38:23.057 回答