CODEWITHSUNDEEP
reactjsreact-props
ThirdGhostHand
ThirdGhostHand

Reputation: 397

React: Passing a function received as props to another function

The function signOut is defined and passed to the stateless Headercomponent by its parent App like this;

export default class App extends React.Component {
  constructor(props) {
    super(props);
    this.signOut = this.signOut.bind(this);
  }

  signOut() {
    this.setState({
      auth: null,
    });
  }

  render() {
    return (
      <div>
        <Header signOut={this.signOut} />
      </div>
    );
  }
}

Header Has several functions defined outside of it, and I would like to further pass signOut to those functions, ideally like I've demonstrated below, but right now, its wrong. 

//one of several functions defined to be used inside header
function nav() {
  return localStorage.getItem("token") ? (
    <p>hii</p>
  ) : (
    <div className="linkWrapper">
      <Link className="link" to="/login">
        Login
      </Link>
      <Link className="link" to="/admin">
        Admin
      </Link>
      //signOut used here. 
      <div className="link" onClick={this.props.signOut}>
        Logout
      </div>
    </div>
  );
}

const Header = (props) => (
  <div className="headerWrapper">
    {nav()}
  </div>
);

export default Header;

How do I properly pass sighOut to these functions from Header? Is this even a thing? Thanks!

Upvotes: 0

Views: 200

Answers (4)

wbd
wbd

Reputation: 272

Give the nav() a sign out argument:

function link({to, signOut, text}) {
  <Link className="link" to={{
    pathname: to,
    state: {
      signOut
    }
  }}>
  </Link
}
function nav({ signOut }) {
  return localStorage.getItem("token") ? (
    <p>hii</p>
  ) : (
    <div className="linkWrapper">
      <link to="/admin" signOut={signOut} text={"Admin"} />
      <link to="/login" signOut={signOut} text={"Login"} />
      <div className="link" onClick={signOut}>
        Logout
      </div>
    </div>
  );
}
const Header = (props) => (
  <div className="headerWrapper">
    <nav signOut={props.signOut} />
  </div>
);

Upvotes: 1

GalAbra
GalAbra

Reputation: 5148

If you want to pass a variable (which might be a function), you need to pass it as a prop to a child component.

Note that the name is determined upon the parent component's passing name:

function Nav(props) {
  return (
    <div className="link" onClick={props.signOut}>
      Logout
    </div>
  );
}


function Header(props) {
 return (
   <Nav
     signOut={props.headerSignOut}
   />
 );
}


function App() {
  function originalSignOutFunction() {
    console.log('Signed out');
  }

  return (
    <Header
      headerSignOut={originalSignOutFunction}
    />
  );
}

Upvotes: 1

Hagai Harari
Hagai Harari

Reputation: 2877

Because you haven't declared Header as class, props aren't suppose to be called with keyword { this }. also, pass props from { Header } to { nav } (if it outside Header...)

const Header = (props) => (
    <div className="headerWrapper">
      {nav(props)}
    </div>
    );
export default Header;


function nav(props) {
 return localStorage.getItem("token") ? (
  <p>hii</p>
   ) : (
  <div className="linkWrapper">
    <Link className="link" to="/login">
      Login
    </Link>
    <Link className="link" to="/admin">
      Admin
    </Link>
    //signOut used here. 
    <div className="link" onClick={props.signOut}>
      Logout
    </div>
  </div>
);

}

Upvotes: 1

Dennis Vash
Dennis Vash

Reputation: 53874

You don't want to call a function you want to render a functional component and pass properties to it:

function Nav(props) {
  // use props.signOut
  return ...
}

const Header = (props) => (
  <div className="headerWrapper">
    <Nav signOut={props.signOut}/>
  </div>
);

Upvotes: 1

Related Questions