255

Trying to get react-router (v4.0.0) and react-hot-loader (3.0.0-beta.6) to play nicely, but getting the following error in the browser console:

Warning: React.createElement: type is invalid -- expected a string
(for built-in components) or a class/function (for composite
components) but got: undefined. You likely forgot to export your
component from the file it's defined in.

index.js:

import React from 'react';
import ReactDom from 'react-dom';
import routes from './routes.js';
require('jquery');
import 'bootstrap/dist/css/bootstrap.min.css';
import 'bootstrap/dist/js/bootstrap.min.js';
import './css/main.css';

const renderApp = (appRoutes) => {
    ReactDom.render(appRoutes, document.getElementById('root'));
};

renderApp( routes() );

routes.js:

import React from 'react';
import { AppContainer } from 'react-hot-loader';
import { Router, Route, browserHistory, IndexRoute } from 'react-router';
import store from './store/store.js';
import { Provider } from 'react-redux';
import App from './containers/App.jsx';
import Products from './containers/shop/Products.jsx';
import Basket from './containers/shop/Basket.jsx';

const routes = () => (

    <AppContainer>
        <Provider store={store}>
            <Router history={browserHistory}>
                <Route path="/" component={App}>
                    <IndexRoute component={Products} />
                    <Route path="/basket" component={Basket} />
                </Route>
            </Router>
        </Provider>
    </AppContainer>

);

export default routes;
2
  • If you use react-router-config make sure you use the component property instead of render, because the package doesn't support the later. See more on GitHub.
    – totymedli
    Feb 20, 2018 at 3:08
  • Check if you have created the component you are trying to import or it actually exists in the project directory. Oct 30, 2021 at 9:43

46 Answers 46

270

Most of the time this is due to an incorrect export/import.

Common error:

// File: LeComponent.js
export class LeComponent extends React.Component { ... }

// File: App.js
import LeComponent from './LeComponent';

Possible option:

// File: LeComponent.js 
export default class LeComponent extends React.Component { ... }

// File: App.js
import LeComponent from './LeComponent';

There are a few ways it could be wrong, but that error is because of an import/export mismatch 60% of the time, everytime.

Edit

Typically you should get a stacktrace that indicates an approximate location of where the failure occurs. This generally follows straight after the message you have in your original question.

If it doesn't show, it might be worth investigating why (it might be a build setting that you're missing). Regardless, if it doesn't show, the only course of action is narrowing down where the export/import is failing.

Sadly, the only way to do it, without a stacktrace is to manually remove each module/submodule until you don't get the error anymore, then work your way back up the stack.

Edit 2

Via comments, it was indeed an import issue, specifically importing a module that didn't exist

15
  • Have looked at the trace which points to line 12 of routes.js. That line is <IndexRoute component={Products} />
    – JoeTidee
    Mar 15, 2017 at 15:52
  • 2
    Are you sure index route is actually part of RR4? (I'm pretty sure it isn't)
    – Chris
    Mar 15, 2017 at 15:57
  • 3
    Had the issue with import { Select } from 'react-select'; and changed it to import Select from 'react-select';
    – JJP
    Jun 12, 2019 at 13:02
  • 1
    @PriyaRanjanSingh It's impossible to tell from just a little comment. Can you post a new question with details?
    – Chris
    Jul 11, 2019 at 2:40
  • 1
    Import issue for me (used brackets on an import of a "default export"). So glad it was that simple... thanks for the help <3
    – amota
    Feb 22, 2022 at 3:45
58

I was getting this error as well.

I was using:

import BrowserRouter from 'react-router-dom';

Fix was doing this, instead:

import { BrowserRouter } from 'react-router-dom';

1
  • 3
    or vice versa - I was using import {AppLoading} from 'expo-app-loading' instead of import AppLoading from 'expo-app-loading'
    – Bo rislav
    Mar 9, 2022 at 11:57
17

Try this

npm i react-router-dom@next

in your App.js

import { BrowserRouter as Router, Route } from 'react-router-dom'

const Home = () => <h1>Home</h1>

const App = () =>(
  <Router>
    <Route path="/" component={Home} />
  </Router>
)

export default App;
1
  • how will you send/receive props to home component then? Dec 3, 2022 at 8:02
14

Array of components

A common way to get this error is using an array of components, with a positional index used to select the component to render from the array. I saw a code like this many times:

const checkoutSteps = [Address, Shipment, Payment]

export const Checkout = ({step}) => {

  const ToRender = checkoutSteps[step]

  return (
    <ToRender />
  )
}

This is not necessary bad code, but if you call it with a wrong index (eg -1, or 3 in this case), the ToRender component will be undefined, throwing the React.createElement: type is invalid... error:

<Checkout step={0} /> // <Address />
<Checkout step={1} /> // <Shipment />
<Checkout step={2} /> // <Payment />
<Checkout step={3} /> // undefined
<Checkout step={-1} /> // undefined

A rational solution

You should protect yourself and your collegues from this hard-to-debug code using a more explicit approach, avoiding magic numbers and using PropTypes:

const checkoutSteps = {
  address: Address,
  shipment Shipment,
  payment: Payment
}

const propTypes = {
  step: PropTypes.oneOf(['address', 'shipment', 'payment']),
}

/* TIP: easier to maintain
const propTypes = {
  step: PropTypes.oneOf(Object.keys(checkoutSteps)),
}
*/

const Checkout = ({step}) => {

  const ToRender = checkoutSteps[step]

  return (
    <ToRender />
  )
}

Checkout.propTypes = propTypes

export default Checkout

And your code will look like this:

// OK
<Checkout step="address" /> // <Address />
<Checkout step="shipment" /> // <Shipment />
<Checkout step="payment" /> // <Payment />

// Errors
<Checkout step="wrongstep" /> // explicit error "step must be one of..."
<Checkout step={3} /> // explicit error (same as above)
<Checkout step={myWrongVar} /> // explicit error (same as above)

Benefits of this approach

  • code is more explicit, you can clearly see what you want to render
  • you don't need to remember the numbers and their hidden meaning (1 is for Address, 2 is for...)
  • errors are explicit too
  • no headache for your peers :)
10

You need to be aware of named export and default export. See When should I use curly braces for ES6 import?

In my case, I fixed it by changing from

import Provider from 'react-redux'

to

import { Provider } from 'react-redux'
1
  • i've narrowed my issue down to this Provider. however, i'm already using the { Provider } notation.
    – Jason G
    Jun 10, 2020 at 17:10
5

I had this problem when I added a css file to the same folder as the component file.

My import statement was:

import MyComponent from '../MyComponent'

which was fine when there was only a single file, MyComponent.jsx. (I saw this format in an example and gave it a try, then forgot I'd done it)

When I added MyComponent.scss to the same folder, the import then failed. Maybe JavaScript loaded the .scss file instead, and so there was no error.

My conclusion: always specify the file extension even if there is only one file, in case you add another one later.

1
  • Hi all, noob here. I think it was certainly a naming convention that was the issue for me, to me the logic above sounds right.
    – tony2tones
    Oct 26, 2022 at 13:44
4

I was getting this error as well.

I was using:

import ReactDOM from 'react-dom';

Fix was doing this, instead:

import {ReactDOM} from 'react-dom';

4
  • dude that does not work, its not even worth a thumbs down Apr 16, 2021 at 1:35
  • bro you are a genius, exactly my problem.
    – myworldbox
    May 21, 2021 at 7:45
  • In my case, it was another way around and resolved, anyway thanks.
    – Jay
    Aug 9, 2021 at 3:05
  • in my case it was other way around so i was importing like this import { SidebarOption } from '../components/sidebarOption' and it was throwing error but when i removed the { } brackets. puff error gone
    – Mohsin
    Mar 6, 2022 at 7:02
4

In my case, VS Code let me down.

Here is the hierarchy of my components:

<HomeScreen> =>  <ProductItemComponent> =>  <BadgeProductComponent>

I had the wrong import of the ProductItemComponent. The fact is that this component used to be in the shared folder, but then it was moved to the home folder. But when I moved the file to another folder, the import did not update and remained the same:

../shared/components

At the same time, the component worked fine and VS Code did not highlight the error. But when I added a new BadgeProductComponent to the ProductItemComponent, I had a Render Error and thought that the problem was in the new BadgeProductComponent, because when this component was removed, everything worked!

And even more than that, if I went through a hotkey to the ProductItemComponent which had the ../shared/components address, then VS Code redirected me to the Home folder with the address ../home/components.

In general, check the correctness of all imports at all component levels.

3

For future googlers:

My solution to this problem was to upgrade react and react-dom to their latest versions on NPM. Apparently I was importing a Component that was using the new fragment syntax and it was broken in my older version of React.

3

This issue has occurred to me when I had a bad reference in my render/return statement. (point to a non existing class). Also check your return statement code for bad references.

3

Most of the time this indicates an import/export error. But be careful to not only make sure the referenced file in the stack trace is well exported itself, but also that this file is importing other components correctly. In my case the error was like this:

import React from 'react';

// Note the .css at the end, this is the cause of the error!
import SeeminglyUnimportantComponent from './SeeminglyUnimportantComponent.css';

const component = (props) => (            
  <div>
    <SeeminglyUnimportantComponent />
    {/* ... component code here */}
  </div>    
);

export default component;

1
  • Thank you for this comment! I have this issue since updating some packages, and I've been scrolling through all of these comments and none of them are in issue in this code. But this was it- the problem file imports something else that is causing the error! Nov 5, 2019 at 17:50
3

I think the most important thing to realize when troubleshooting this bug is that it manifests when you attempt to instantiate a component that doesn't exist. This component doesn't have to be imported. In my case I was passing components as properties. I forgot to update one of the calls to properly pass the component after some refactoring. Unfortunately, since JS isn't statically typed my bug wasn't caught, and it took some time to figure out what was happening.

To troubleshoot this bug inspect the component before you render it, to make sure that it's the type of component you expect.

1
  • "it manifests when you attempt to instantiate a component that doesn't exist" -- this is what happened to me. The component was imported/exported correctly but I was passing it as a prop through an authorization route in react-router and forgot to change the render={props => <Component {...props} />} to component={Component}. Mar 10, 2020 at 20:15
3

It means your import/export is incorrect.

  • Check newly added import/exports.
  • In my case I was using curly brackets unnecessary. Issue got resolved automatically when I removed these curly brackets.
import { OverlayTrigger } from 'react-bootstrap/OverlayTrigger';
2

Circular dependency is also one of the reasons for this. [in general]

0
2

I was missing a React Fragment:


function Bar({ children }) {

  return (
    <div>
     {children}
    </div>
  );
}

function Foo() {
  return (
    <Bar>
      <Baz/>
      <Qux/>
    </Bar>
  );
}

The code above throws the error above. But this fixes it:

<Bar>
  <>
    <Baz/>
    <Qux/>
  </>
</Bar>
2
  • Happened to me when I added ReactCollapsingTable (yarn add react-collapsing-table). I managed to alternate between this warning (that resulted in a hydration error) and a 'window is not defined' that was detected inside the compiled react-collapsing-table module... Even wrapping with <div>...</div> doesn't help anymore
    – superk
    Nov 23, 2019 at 14:06
  • Eventually I re-transpiled the whole project from start (instead of hot-reload) and the problem recurred. Found the solution here: stackoverflow.com/a/43969990/2821963 (instead of import module from 'module-name', use require(module-name).default)
    – superk
    Nov 23, 2019 at 14:21
2

My case was not an import issue like many of the answers above say. In mine we were using a wrapper component to do some translation logic and I was passing the child component in incorrectly like so:

const WrappedComponent = I18nWrapper(<ChildForm {...additionalProps} />);

When I should have been passing it in as a function:

const WrappedComponent = I18nWrapper(() => <ChildForm {...additionalProps} />);
2

For me, I had two files with the same name but different extensions (.js and .tsx). Thus in my import I had to specify the extension.

1

What missing for me was I was using

import { Router, Route, browserHistory, IndexRoute } from 'react-router';

instead or correct answer should be :

import { BrowserRouter as Router, Route } from 'react-router-dom';

Ofcourse you need to add npm package react-router-dom:

npm install react-router-dom@next --save
1

If you have this error when testing a component, make sure that every child component render correctly when run alone, if one of your child component depend on external resources to render, try to mock it with jest or any other mocking lib:

Exemple:

jest.mock('pathToChildComponent', () => 'mock-child-component')
1

In my case, the error occurred when trying to use ContextApi. I have mistakenly used:

const MyContext = () => createContext()

But it should have been defined as:

const MyContext = createContext()

I am posting it here so that future visitors who get stuck on such a silly mistake will get it helpful to avoid hours of headache, coz this is not caused by incorrect import/export.

1

In my case I forgot to import and export my (new) elements called by the render in the index.js file.

1

It's quite simple, really. I got this issue when I started coding React, and the problem is almost always because the import:

import React, { memo } from 'react';

You can use destructuring this because react lib has a property as memo, but you can not destructuring something like this

import { user } from 'assets/images/icons/Profile.svg';

because it's not a object.

Hope it helps!

1

React.Fragment

fixed the issue for me

Error Code:

 return (
            <section className={classes.itemForm}>
             <Card>
             </Card> 
            </section>
      );

Fix

 return (
      <React.Fragment>
        <section className={classes.itemForm}>
         <Card>
         </Card> 
        </section>
      </React.Fragment>
  );
1
xxxxx.prototype = {
  dxxxx: PropTypes.object.isRequired, // eslint-disable-line react/forbid-prop-types
};

You must add // eslint-disable-line react/forbid-prop-types, then it work!

1

The application that I was working on, stored the name of react components as a widget configuration in the browser storage. Some of these components got deleted and the app was still trying to render them. By clearing the browser cache, I was able to resolve my issue.

1

For me removing Switch solved the issue

import React from "react";
import "./styles.css";
import { Route, BrowserRouter, Routes } from "react-router-dom";
import LoginPage from "./pages/LoginPage";
import HomePage from "./pages/HomePage";

export default function App() {
  return (
    <BrowserRouter>
      <Routes>
        <Route exact path="/" element={<HomePage />} />
        <Route path="/login" element={<LoginPage />} />
      </Routes>
    </BrowserRouter>
  );
}
1

I have had this problem already. My workaround is:

At file config routes:

const routes = [
    { path: '/', title: '', component: Home },
    { path: '*', title: '', component: NotFound }
]

to:

const routes = [
    { path: '/', title: '', component: <Home /> },
    { path: '*', title: '', component: <NotFound /> }
]
1

For me this happened while I have tried to import a named import as default import, SO I got this error

import ProductCard from '../../../components/ProductCard' // that what caused the issue
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

Check the render method of `TopVente`.

So I had to fix it by name import

import { ProductCard } from '../../../components/ProductCard'
1

In my case, while using Jest, I mocked some modules, and it causes the error because the mocked module doesn't include some component that is used in other areas.

So please check if you have some mocking function within your test component.

1

You might be returning object from some specific file instead just return the statement

WRONG:

import React from 'react'
const About = () => {   return ( <div>About</div>   ) }
export default About

CORRECT:

import React from 'react'
const About = () => {   return 
<div>About</div>} 
export default About

Not the answer you're looking for? Browse other questions tagged or ask your own question.