DEV Community

Cover image for 10 Common Mistakes to Avoid in React Development
Sufian mustafa
Sufian mustafa

Posted on

10 Common Mistakes to Avoid in React Development

10 Common Mistakes to Avoid in React Development

React has become a cornerstone in modern web development, but like any powerful tool, it comes with its own set of pitfalls. In this blog post, we'll explore ten common mistakes that developers often make when working with React. By understanding and avoiding these pitfalls, you can write cleaner, more efficient, and maintainable React code.

10 Common Mistakes to Avoid in React Development

1. Ignoring the Virtual DOM

One of React's core strengths is its Virtual DOM, which efficiently updates only the parts of the actual DOM that have changed. Ignoring this mechanism and directly manipulating the DOM can lead to performance issues. Always embrace React's declarative approach and let it handle the updates.

// Incorrect: Directly manipulating the DOM
const BadExample = () => {
  const handleClick = () => {
    document.getElementById('myElement').style.color = 'red';
  };

  return (
    <div>
      <button onClick={handleClick}>Change Color</button>
      <p id="myElement">Hello, World!</p>
    </div>
  );
};


// Correct: Using React's state to manage color
const GoodExample = () => {
  const [color, setColor] = useState('black');

  const handleClick = () => {
    setColor('red');
  };

  return (
    <div>
      <button onClick={handleClick}>Change Color</button>
      <p style={{ color }}>Hello, World!</p>
    </div>
  );
};

Enter fullscreen mode Exit fullscreen mode

2. Not Using Keys Properly

Keys are crucial when rendering lists of components in React. Forgetting to assign unique keys can result in unexpected behavior, such as components not updating correctly or unnecessary re-rendering. Understand the importance of keys and use them appropriately to optimize your app's performance.

// Incorrect: Forgetting to assign unique keys
const BadList = () => {
  const items = [1, 2, 3];

  return (
    <ul>
      {items.map(item => (
        <li>{item}</li>
      ))}
    </ul>
  );
};


// Correct: Using unique keys for list items
const GoodList = () => {
  const items = [1, 2, 3];

  return (
    <ul>
      {items.map(item => (
        <li key={item}>{item}</li>
      ))}
    </ul>
  );
};
Enter fullscreen mode Exit fullscreen mode

3. Uncontrolled State Mutations

Manipulating the state directly without using the setState function can lead to subtle bugs and unpredictable behavior. Always update the state immutably using the functional form of setState to ensure a reliable and consistent state management.

// Incorrect: Directly modifying state
class BadCounter extends React.Component {
  state = {
    count: 0,
  };

  incrementCount() {
    this.state.count++; // Direct mutation
    this.forceUpdate(); // Incorrect usage of forceUpdate
  }

  render() {
    return (
      <div>
        <p>Count: {this.state.count}</p>
        <button onClick={() => this.incrementCount()}>Increment</button>
      </div>
    );
  }
}


// Correct: Updating state immutably
class GoodCounter extends React.Component {
  state = {
    count: 0,
  };

  incrementCount() {
    this.setState(prevState => ({
      count: prevState.count + 1,
    }));
  }

  render() {
    return (
      <div>
        <p>Count: {this.state.count}</p>
        <button onClick={() => this.incrementCount()}>Increment</button>
      </div>
    );
  }
}

Enter fullscreen mode Exit fullscreen mode

4. Overusing Component State

While React component state is powerful, it's essential to recognize when to use it. Avoid excessive reliance on component state for global state management. For complex applications, consider using state management libraries like Redux to centralize and manage the application's state more effectively.

// Incorrect: Excessive use of component state
const BadApp = () => {
  const [name, setName] = useState('');
  const [age, setAge] = useState(0);
  const [isLoggedIn, setIsLoggedIn] = useState(false);

  // ... rest of the component
};

// Correct: Using state management library for global state
const GoodApp = () => {
  // Using a state management library like Redux
  const { name, age, isLoggedIn } = useSelector(state => state.user);

  // ... rest of the component
};
Enter fullscreen mode Exit fullscreen mode

5. Poor Component Lifecycle Management

Misusing the component lifecycle methods can result in inefficient code and performance issues. Be aware of when to use componentDidMount, componentDidUpdate, and componentWillUnmount to handle side effects, updates, and cleanup appropriately.

6. Inefficient Rendering

Rendering unnecessary components or rendering too frequently can impact your application's performance. Optimize your components by using techniques like shouldComponentUpdate or PureComponent to prevent unnecessary re-renders.

7. Not Handling Errors Effectively

Failing to implement proper error boundaries and error handling can lead to a poor user experience. Wrap your components or specific parts of your application in error boundaries to gracefully handle errors and provide users with helpful information.

8. Mixing Presentational and Container Components

Separation of concerns is crucial for maintainable code. Avoid the mistake of mixing presentational and container components. Keep your presentation components focused on rendering UI, while container components manage state and logic.

9. Neglecting Prop Types

PropTypes provide a valuable tool for documenting and enforcing the expected types of props passed to your components. Neglecting to define PropTypes can make your code harder to maintain and lead to runtime errors. Always define PropTypes for your components.

10. Not Keeping Up with React Best Practices

React evolves, and best practices change over time. Failing to stay updated with the latest recommendations and best practices can result in outdated and less efficient code. Regularly check the React documentation and community discussions to ensure you're following current best practices.

In conclusion,

React development offers incredible power, but avoiding common mistakes is key to harnessing that power effectively. By steering clear of these pitfalls, you'll be well on your way to writing clean, performant, and maintainable React code.

Remember, we all make mistakes, and learning from them is an integral part of the development journey.
Please give heart πŸ’– if u find it useful! Happy coding! πŸš€

Top comments (12)

Collapse
 
lebbe profile image
Lars-Erik Bruce

What is the need or purpose of "container components" when we can keep state and logic in Context, custom hooks or a state management library?

Collapse
 
sufian profile image
Sufian mustafa

Thank you for your insightful question! While it's true that we can manage state and logic using Context, custom hooks, or state management libraries, 'container components' are a design pattern that helps with the separation of concerns.

Container components typically handle the data-fetching, state management, and business logic, while presentational components focus on rendering UI. This separation enhances code organization and maintainability, making it easier to understand and manage complex applications.

Collapse
 
lebbe profile image
Lars-Erik Bruce

I'm sorry, but I still don't see the usage of container components as opposed to a custom hook. Let's say you make a widget named Widget, you can make a hook named useWidget containing all the logic, state and data fetching. So you still have separation of concerns, but without the extra container component.

Not only that, but with custom hooks you can also make a mock hook useMockWidget, that you can use when testing the GUI aspects of the Widget, without invoking network calls, etc.

<Widget {...useWidget()} />

<Widget {...useMockWidget()} />

So you get the exact same separation of concerns you are talking about, but with added benefits.

All in all, using container components seems like a rather dated design pattern, after the introduction of hooks.

Thread Thread
 
sufian profile image
Sufian mustafa

It's a valid point that custom hooks, especially with the introduction of hooks in React, provide a powerful way to encapsulate logic and achieve separation of concerns. Your example of using useWidget and useMockWidget for testing is a clever and effective approach.

The choice between using container components and custom hooks often comes down to personal preference, project requirements, and team workflows. While custom hooks are indeed a more modern and concise pattern, some developers and teams still find value in the explicit separation that container components offer, particularly in larger applications or those with more complex state management needs.

It's great to see different approaches to achieving clean architecture in React projects. If container components work well for your use case, that's fantastic! The React ecosystem is diverse, and the beauty lies in choosing the patterns and techniques that best fit the specific context of your project.

I hope it will be helpful.

Thread Thread
 
lebbe profile image
Lars-Erik Bruce

"some developers and teams still find value in the explicit separation that container components offer,"

Could you indulge me, in what way is container components more explicit than hooks?

Thread Thread
 
sufian profile image
Sufian mustafa

I appreciate your curiosity! When I mentioned 'explicit separation` I was referring to the physical organization of concerns in different files or components. With container components, there's a clear distinction between components that handle state management, data fetching, and business logic (containers) and components that focus on rendering UI (presentational components).

While custom hooks also achieve a separation of concerns, some developers find that container components provide a more visual and structured way of organizing this separation. In a large codebase, having a dedicated folder or directory for container components can make it easier for developers to quickly identify where state and logic are managed..

Thread Thread
 
lebbe profile image
Lars-Erik Bruce

I still don't understand how "container components" is any different from custom hooks in this regard: Custom hooks can also be inside their own files, and be located inside dedicated folders. There simply is nothing about container components thats makes it more visual or structural than custom hooks. Why do you keep repeating these claims?

Thread Thread
 
sufian profile image
Sufian mustafa

I appreciate your diligence in seeking clarification. I want to emphasize that the distinction between container components and custom hooks isn't about technical capabilities but more about the mental model and preferences of developers and teams.

Container components, traditionally associated with class components and higher-order components (HOCs), were often explicitly created to manage state and logic. In contrast, custom hooks, introduced with the advent of functional components and hooks, offer a more concise and composable way to encapsulate and share logic.

While both can be organized in files and folders, some developers find that container components, due to their historical association with state management, offer a more recognizable and explicit separation in the codebase. The mental model of having specific components dedicated to managing state and logic can be easier for some developers to grasp.

However, I completely acknowledge that custom hooks, when organized effectively, can achieve the same level of clarity and separation. It all comes down to personal preference, team workflows, and the particular dynamics of the project.🫑🫑

Collapse
 
brense profile image
Rense Bakker

10 is somewhat ironic since class components are no longer a best practice since React 16.8

Collapse
 
sufian profile image
Sufian mustafa

Yes, You're absolutely right that React has shifted towards functional components and hooks since React 16.8. The mention of class components in point 10 is indeed a bit ironic considering the current best practices. It highlights the importance of staying up-to-date with the evolving React ecosystem. Thank you for pointing that out!

Collapse
 
glntn101 profile image
Luke

Absolure agree with your arguments, thank you.

Collapse
 
sufian profile image
Sufian mustafa

Your welcome brother I'm glad you found the arguments convincing. πŸ’–