10

I'm extending a base class and overriding a method in the base class. But when I call it, it calls the super class version. How do I override the method?

    var Hello = React.createClass( {

        getName: function() { return "super" },

        render: function() {

            return <div>This is: {this.getName()}</div>;
        }
    });

    class HelloChild extends Hello {
        constructor(props) {
          super(props);

          console.log( this.getName());
        }
        getName()
        {
          return "Child";
        }
    };

I want it to print "This is: Child" but it prints "This is: super"

1

5 Answers 5

5

The problem is that you're mixing ES6 type class declaration (ex. Hello) with old school Javascript declaration (ex. HelloChild). To fix HelloChild, bind the method to the class.

class HelloChild extends Hello {
    constructor(props) {
      super(props);

      this.getName = this.getName.bind(this); // This is important

      console.log( this.getName());
    }

    getName()
    {
      return "Child";
    }
};

Then it'll work.

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

Comments

3

I found the answer (adapted from here: https://gist.github.com/Zodiase/af44115098b20d69c531 ) - the base class needs to also be defined in an ES6 manner:

class Hello extends React.Component {

        //abstract getName()
        getName()
        {
            if (new.target === Hello) {
                throw new TypeError("method not implemented");
            }
        }

        render() {

            return <div>This is: {this.getName()}</div>;
        }
    };

2 Comments

The OP wants to know how to access subclass from superclass (which is impossible AFAIK), not how to make an abstract method-the super and subclass methods have different implementations
That is the OP ^
3

Actually you can override method to execute code from your subclass

class Hello extends React.Component {
getName() {
 super.getName();
 }
}


class HelloChild extends Hello {
getName()
    {
      return "Child";
    }
}

Comments

0

Please note that this answer proposes different approach:

I wonder why you should do this in the first place, my point is that directly coupling two react components is not a right way to implement re-usability in React.

If you are trying to have multiple child components which extends one parent, What I would do is, to have child components and a higher-order component and then implement common functionality with Composition. This way you can skip those methods, which you were trying to override and so everything would stay clear.

Comments

0

If you're reading this far, you've likely found that this still doesn't work correctly:

class Hello extends React.Component {
  getName() {
    if (new.target === Hello) {
      throw new TypeError("method not implemented");
    }
  };

  render() {
    return <div>This is: {this.getName()}</div>;
  }
}

class HelloChild extends Hello {
  getName() {
    return "Child";
  };
}

I used to not think arrow functions on classes were important - it looked like old school javascript, but it's exactly what's needed.

Arrow functions correctly preserve this context, this is the correct way to do it:

class Hello extends React.Component {
  getName = () => {
    if (new.target === Hello) {
      throw new TypeError("method not implemented");
    }
  };

  render() {
    return <div>This is: {this.getName()}</div>;
  }
}

class HelloChild extends Hello {
  getName = () => {
    return "Child";
  };
}

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.