Merge pull request #14 from veewee/getproperty-speed

Introduce native function invocation rule + Speed up finding a single…

.php-cs-fixer.dist.php

@@ -53,5 +53,6 @@
         'static_lambda' => true,
         'strict_comparison' => true,
         'strict_param' => true,
+        'native_function_invocation' => true,
     ])
 ;

src/ArrayAccess/index_get.php

@@ -3,6 +3,7 @@
 namespace VeeWee\Reflecta\ArrayAccess;
 
 use VeeWee\Reflecta\ArrayAccess\Exception\ArrayAccessException;
+use function array_key_exists;
 
 /**
  * @pure

src/Reflect/Type/ReflectedClass.php

@@ -12,6 +12,7 @@
 use ReflectionProperty;
 use Throwable;
 use VeeWee\Reflecta\Reflect\Exception\UnreflectableException;
+use function is_string;
 use function Psl\Dict\filter;
 use function Psl\Dict\reindex;
 use function Psl\Vec\map;
@@ -125,12 +126,15 @@ public function docComment(): string
 
     public function property(string $property): ReflectedProperty
     {
-        $properties = $this->properties();
-        if (!array_key_exists($property, $properties)) {
-            throw UnreflectableException::unknownProperty($this->fullName(), $property);
+        if ($this->class->hasProperty($property)) {
+            return new ReflectedProperty($this->class->getProperty($property));
         }
 
-        return $properties[$property];
+        if ($parent = $this->parent()->unwrapOr(null)) {
+            return $parent->property($property);
+        }
+
+        throw UnreflectableException::unknownProperty($this->fullName(), $property);
     }
 
     /**

tests/unit/Lens/LensTest.php

@@ -6,6 +6,7 @@
 use PHPUnit\Framework\TestCase;
 use RuntimeException;
 use VeeWee\Reflecta\Lens\Lens;
+use function array_key_exists;
 use function VeeWee\Reflecta\Lens\index;
 
 final class LensTest extends TestCase

tests/unit/Lens/OptionalTest.php

@@ -6,6 +6,7 @@
 use PHPUnit\Framework\TestCase;
 use RuntimeException;
 use VeeWee\Reflecta\Lens\Lens;
+use function array_key_exists;
 use function VeeWee\Reflecta\Lens\optional;
 
 final class OptionalTest extends TestCase

tests/unit/Reflect/ClassAttributesTest.php

@@ -9,6 +9,7 @@
 use VeeWee\Reflecta\TestFixtures\AbstractAttribute;
 use VeeWee\Reflecta\TestFixtures\CustomAttribute;
 use VeeWee\Reflecta\TestFixtures\InheritedCustomAttribute;
+use function get_class;
 use function VeeWee\Reflecta\Reflect\class_attributes;
 
 final class ClassAttributesTest extends TestCase

tests/unit/Reflect/ClassHasAttributeTest.php

@@ -7,6 +7,7 @@
 use VeeWee\Reflecta\TestFixtures\AbstractAttribute;
 use VeeWee\Reflecta\TestFixtures\CustomAttribute;
 use VeeWee\Reflecta\TestFixtures\InheritedCustomAttribute;
+use function get_class;
 use function VeeWee\Reflecta\Reflect\class_has_attribute;
 
 final class ClassHasAttributeTest extends TestCase

tests/unit/Reflect/ClassIsDynamicTest.php

@@ -6,6 +6,7 @@
 use AllowDynamicProperties;
 use PHPUnit\Framework\TestCase;
 use stdClass;
+use function get_class;
 use function VeeWee\Reflecta\Reflect\class_is_dynamic;
 use const PHP_VERSION_ID;