Reagir: "isto" é indefinido dentro de uma função de componente

152
class PlayerControls extends React.Component {
  constructor(props) {
    super(props)

    this.state = {
      loopActive: false,
      shuffleActive: false,
    }
  }

  render() {
    var shuffleClassName = this.state.toggleActive ? "player-control-icon active" : "player-control-icon"

    return (
      <div className="player-controls">
        <FontAwesome
          className="player-control-icon"
          name='refresh'
          onClick={this.onToggleLoop}
          spin={this.state.loopActive}
        />
        <FontAwesome
          className={shuffleClassName}
          name='random'
          onClick={this.onToggleShuffle}
        />
      </div>
    );
  }

  onToggleLoop(event) {
    // "this is undefined??" <--- here
    this.setState({loopActive: !this.state.loopActive})
    this.props.onToggleLoop()
  }

Desejo atualizar o loopActiveestado ao alternar, mas o thisobjeto não está definido no manipulador. De acordo com o documento do tutorial, thisdevo me referir ao componente. Estou esquecendo de algo?

Maximus S
fonte

Respostas:

211

O ES6 React.Component não liga automaticamente métodos a si mesmo. Você precisa vinculá-los você mesmo no construtor. Como isso:

constructor (props){
  super(props);

  this.state = {
      loopActive: false,
      shuffleActive: false,
    };

  this.onToggleLoop = this.onToggleLoop.bind(this);

}
Ivan
fonte
23
se você alterar sua propriedade onClick para () => this.onToggleLoopdepois de mover a função onToggleLoop para sua classe de reação, ela também funcionará.
Sam
71
Você realmente precisa vincular todos os métodos de todas as classes de reação? Isso não é um pouco louco?
Alex L
6
@AlexL Existem maneiras de fazer isso sem vincular explicitamente os métodos. se você usar o babel, é possível declarar todos os métodos no componente React como funções de seta. Há exemplos aqui: babeljs.io/blog/2015/06/07/react-on-es6-plus
Ivan
7
Mas por que não está thisdefinido em primeiro lugar? Eu sei que o thisJavascript depende de como a função é chamada, mas o que está acontecendo aqui?
maverick
1
mas como posso fazer isso é a função não é definida até depois do construtor? Eu recebo um "Não é possível ler a propriedade 'bind' de indefinido" se eu tentar fazer isso no construtor, mesmo que minha função seja definida na classe :(
rex
87

Existem algumas maneiras.

Uma é adicionar this.onToggleLoop = this.onToggleLoop.bind(this);no construtor.

Outra são as funções de seta onToggleLoop = (event) => {...}.

E então existe onClick={this.onToggleLoop.bind(this)}.

J. Mark Stevens
fonte
1
Por que onToogleLoop = () => {} funciona? Eu tenho o mesmo problema e liguei-o no meu construtor, mas ele não funcionou ... e agora eu vi o seu post e substitui o meu método por uma sintaxe da função seta e funciona. Você pode explicar isso para mim ?
Guchelkaben 15/09
5
de developer.mozilla.org/pt-BR/docs/Web/JavaScript/Reference/… ; Uma função de seta não cria isso por conta própria, é usado o valor this do contexto de execução em anexo.
J. Mark Stevens
1
Note-se que em linha de ligação na onClickretornará uma nova função a cada render e, assim, parece que um novo valor foi passado para o prop, mexer com shouldComponentUpdateno PureComponents.
Ninjakannon 25/05
24

Escreva sua função desta maneira:

onToggleLoop = (event) => {
    this.setState({loopActive: !this.state.loopActive})
    this.props.onToggleLoop()
}

Funções de seta gorda

a ligação para a palavra-chave é a mesma fora e dentro da função seta gorda. Isso é diferente das funções declaradas com a função, que podem vincular isso a outro objeto na chamada. Manter essa ligação é muito conveniente para operações como mapeamento: this.items.map (x => this.doSomethingWith (x)).

ShaTin
fonte
Se eu fizer isso eu entendo ReferenceError: fields are not currently supported.
Pavel Komarov
Funciona se dentro do construtor eu digo this.func = () => {...}, mas considero isso meio pateta e quero evitá-lo, se possível.
Pavel Komarov
Tão terrível que você não pode usar a sintaxe de classe normal no React!
Kokodoko 10/07/19
11

Corri para uma ligação semelhante em uma função de renderização e acabei passando o contexto da thisda seguinte maneira:

{someList.map(function(listItem) {
  // your code
}, this)}

Eu também usei:

{someList.map((listItem, index) =>
    <div onClick={this.someFunction.bind(this, listItem)} />
)}
duhaime
fonte
Isso é um monte de funções desnecessárias que você está criando ali, cada vez que a lista é processado ...
TJ Crowder
@TJCrowder Sim, é verdade que essas funções são criadas novamente sempre que a renderização é chamada. É melhor para criar as funções como métodos de classe e aprisioná-los uma vez para a classe, mas para iniciantes a ligação contexto manual pode ser útil
duhaime
2

Você deve observar que thisdepende de como a função é chamada, ou seja: quando uma função é chamada como método de um objeto, seuthis é definida como o objeto em que o método é chamado.

thisestá acessível no contexto JSX como seu objeto componente, para que você possa chamar o método desejado embutido como thismétodo.

Se você apenas passar uma referência à função / método, parece que o react o chamará como função independente.

onClick={this.onToggleLoop} // Here you just passing reference, React will invoke it as independent function and this will be undefined

onClick={()=>this.onToggleLoop()} // Here you invoking your desired function as method of this, and this in that function will be set to object from that function is called ie: your component object
Jakub Kutrzeba
fonte
1
Certo, você ainda pode usar a primeira linha ou seja, onClick={this.onToggleLoop}desde que, em sua classe componente que tenha definido um campo (propriedade)onToggleLoop = () => /*body using 'this'*/
gvlax
1

Se você estiver usando babel, vincule 'this' usando o operador de ligação ES7 https://babeljs.io/docs/en/babel-plugin-transform-function-bind#auto-self-binding

export default class SignupPage extends React.Component {
  constructor(props) {
    super(props);
  }

  handleSubmit(e) {
    e.preventDefault(); 

    const data = { 
      email: this.refs.email.value,
    } 
  }

  render() {

    const {errors} = this.props;

    return (
      <div className="view-container registrations new">
        <main>
          <form id="sign_up_form" onSubmit={::this.handleSubmit}>
            <div className="field">
              <input ref="email" id="user_email" type="email" placeholder="Email"  />
            </div>
            <div className="field">
              <input ref="password" id="user_password" type="new-password" placeholder="Password"  />
            </div>
            <button type="submit">Sign up</button>
          </form>
        </main>
      </div>
    )
  }

}
Henry Jacob
fonte
0

Se você chamar seu método criado nos métodos de ciclo de vida como componentDidMount ..., poderá usar apenas a função this.onToggleLoop = this.onToogleLoop.bind(this)e a seta de gorduraonToggleLoop = (event) => {...} .

A abordagem normal da declaração de uma função no construtor não funcionará porque os métodos do ciclo de vida são chamados anteriormente.

Guchelkaben
fonte
0

no meu caso, essa foi a solução = () => {}

methodName = (params) => {
//your code here with this.something
}
Alex
fonte
0

No meu caso, para um componente sem estado que recebeu o ref com o forwardRef, tive que fazer o que foi dito aqui https://itnext.io/reusing-the-ref-from-forwardref-with-react-hooks-4ce9df693dd

A partir disso (onClick não tem acesso ao equivalente a 'this')

const Com = forwardRef((props, ref) => {
  return <input ref={ref} onClick={() => {console.log(ref.current} } />
})

Para isso (funciona)

const useCombinedRefs = (...refs) => {
  const targetRef = React.useRef()

  useEffect(() => {
    refs.forEach(ref => {
      if (!ref) return

      if (typeof ref === 'function') ref(targetRef.current)
      else ref.current = targetRef.current
    })
  }, [refs])

  return targetRef
}

const Com = forwardRef((props, ref) => {
  const innerRef = useRef()
  const combinedRef = useCombinedRefs(ref, innerRef)

  return <input ref={combinedRef } onClick={() => {console.log(combinedRef .current} } />
})
GWorking
fonte