|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2015-12-14 18:28 UTC] chazmead89 at gmail dot com
Description:
------------
PHP Warning: Declaration of B::foo(string $boo) should be compatible with A::foo(array $bar) in php shell code on line 3
This should not be a warning... this is basic method overriding, and should be perfectly good code.
For some reason I get a fatal error on apache handler, but only warning on CLI, possibly Xdebug RC4 escalating it as a bug in XDdebug.
Test script:
---------------
class A {
public function foo(string $bar) { print_r($bar); }
}
class B extends A {
public function foo(array $bar) { print_r($bar); }
}
Actual result:
--------------
PHP Warning: Declaration of B::foo(string $boo) should be compatible with A::foo(array $bar) in php shell code on line 3
Warning: Declaration of B::foo(string $boo) should be compatible with A::foo(array $bar) in php shell code on line 3
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Tue Dec 16 06:00:02 2025 UTC |
What about this test case then, called Method overloading.. class A { public function foo(array $bar) { print_r($bar); } } class B extends class A { public function foo(array $bar, string $baz) { print_r([$bar, $baz]); } } Is it now impossible to override / overload a class methods EVEN if there is no interface defined... if there was an interface defined I'd appreciate that any subclass needs to conform to that interface.. but otherwise the class is just a class and methods should be completely overridable and overloadable. There are literally millions of cases where this may be required, and if you don't think its a bug, then I'll raise another ticket and try get someone else with a brain to look at this.I'm not sure why you're putting a slash between overriding and overloading, these two are completely different matters. Overloading is a form of dynamic dispatch when the concrete method is found during runtime based on the instance type AND argument types. PHP doesn't have that meaning that. Now overriding is when you override a method with another polymorphic method. They have to be compatible, otherwise due to the nature of PHP's method dispatch (when all methods are virtual) you'd have an error in otherwise type-safe cases like this: class A { function foo(A $a) {} } class B { function foo(B $a) {} } $a = new B; bar($a); function bar(A $a) { $a->foo(new A); }So here's my usecass.. I have a class, which is invokable, I use a dependancy injector to call the invoke method, the class defines a bunch of other methods which invoke uses to create a callable. simplified: class A { protected $_class = NULL; protected $_method = NULL; public function __construct(string $class, string $method) { $this->_class = $class; $this->_method = $method; } public function __invoke(Dependancy $dep) { $c = $this->getCallable(); return $c($dep); } protected function getCallable(): callable { return [ new $this->_class, $this->_method ]; } } Now I want to add extra dependancies to an extending class for additional functionality: class B extends A { public function __invoke(Dependancy $dep, Additional $dep2) { // DO some additional stuff... if ( $dep2 === True ) { $c = [ $this, 'anotherCallableBecauseTrue' ]; return $c($dep, $dep2); } return parent::__invoke($dep); } } This was never an issue in PHP5.. so I guess can you link me to the RFC discussion where this was changed?