Correct way to handle errors and display message in React using Hooks

Question

What would be the best way to handle errors and display them in a React App using Hooks, at the moment if I try to break the app by mistyping the URL it shows the error but still the data sometimes also, however if I update the state to an empty array in the catch block setData([]);, then it works fine, I just wanted to check and see if this is the ideal way or is there another way?

App.js

import React, {useEffect} from 'react';
import './App.css';
import axios from 'axios';

const App = () => {

  interface DataHolder {
    userId: string;
    id: string;
    title: string;
    body: string;
  }

  const [data, setData] = React.useState<DataHolder[]>([]);
  const [isLoading, setIsLoading] = React.useState<Boolean>(false);
  const [hasError, setHasError] = React.useState<Boolean>(false)

  useEffect( () => {
    const fetchData = async (): Promise<any> => {
    setIsLoading(true);
    setHasError(false);
    try {
        const result = await axios('https://jsonplaceholder.typicode.com/posts');
        setData(result.data);
    } catch (err) {
        setHasError(true);
        setData([]);
        console.log(err);
    }
    setIsLoading(false);
  }
  fetchData()
    return () => {
        console.log('cleaned');
    }
}, [setData]);

  return (
    <>
    {hasError && <p >Something went wrong. problem with the data feed.</p>}
    {isLoading ? (
    <p >Loading ...</p>
  ) : (
    <ul>
      {data.map(item => (
        <li key={item.id} >
          <p>{item.title}</p>
        </li>
      ))}
    </ul>
  )}
    </>
  );
}

export default App;

Amruth · Accepted Answer · 2021-08-03 11:42:38Z

1

Conditional rendering should help when you are dealing with hooks.

Loading
Error
Data display part

You can order like this.

  if (isLoading) {
    return <p>Loading ...</p>;
  }
  if (hasError) {
    return <p>Something went wrong. problem with the data feed.</p>;
  }
  return (
    <>
      <ul>
        {data?.map((item) => (
          <li key={item.id}>
            <p>{item.title}</p>
          </li>
        ))}
      </ul>
    </>
  );

edited Aug 3, 2021 at 11:42

answered Aug 3, 2021 at 11:15

Amruth

5,9122 gold badges30 silver badges41 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Sole Over a year ago

So, your just returning different blocks based on conditions?

Amruth Over a year ago

yeah conditional rendering. priority is like loading, error, data display.

Sole Over a year ago

Thanks, so need to to do anything with the state?

Amruth Over a year ago

no need, based on your state this render works

Amruth Over a year ago

you can also add null check like this ->. data?.map updated ans also.

Sachin K · Accepted Answer · 2021-08-03 11:33:56Z

1

You can also give a condition like this, checking the length of the data

return (
<>
  <ul>
    {data.length > 0 ? data.map((item) => (
      <li key={item.id}>
        <p>{item.title}</p>
      </li>
    )) : null}
  </ul>
</>

);

answered Aug 3, 2021 at 11:33

Sachin K

1371 gold badge2 silver badges11 bronze badges

Collectives™ on Stack Overflow

Correct way to handle errors and display message in React using Hooks

2 Answers 2

5 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related