Welcome to ShenZhenJia Knowledge Sharing Community for programmer and developer-Open, Learning and Share
menu search
person
Welcome To Ask or Share your Answers For Others

Categories

I have an If statement and returning the same component with the extra props based on the state. Any idea how to simplify this? Can I use recursion? Any idea?

iconRight is only difference.

renderInput = () => {
    if (isLoading) {
      return (
        <Input
          iconRight={(
              <Spinner />
          )}
          autoComplete="off"
          id="unique-id-2"
          aria-autocomplete="both"
        />
      );
    }
    return (
      <Input
        autoComplete="off"
        id="unique-id-2"
        aria-autocomplete="both"
      />
    );
  }
}
question from:https://stackoverflow.com/questions/65601628/how-to-simplify-overriding-props-in-react

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
thumb_up_alt 0 like thumb_down_alt 0 dislike
94 views
Welcome To Ask or Share your Answers For Others

1 Answer

You can spread props onto the component:

renderInput = () => {
    const props = {
        autoComplete: 'off',
        id: 'unique-id-2',
        'aria-autocomplete': 'off'
    };
    if (isLoading) {
      return (
        <Input
          iconRight={(
              <Spinner />
          )}
          {...props}
        />
      );
    }
    return (
      <Input {...props} />
    );
  }
}

But i'd suggest changing your Input component to accept a loading prop and let the Input component handle that logic. It'll make your consuming code a lot easier to read also.


与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
thumb_up_alt 0 like thumb_down_alt 0 dislike
Welcome to ShenZhenJia Knowledge Sharing Community for programmer and developer-Open, Learning and Share
...