closes #1: bugfix

Author: baygin

src/BinarySearch/BinarySearch.php

@@ -35,6 +35,9 @@ public function search(): void
             throw new InvalidArgumentException('The $directionCallback attribute must be instance of the type callback. Set attribute using by $class->setDirectionClass(fn($a, $b) => $a > $b)');
         }
 
+        $this->foundIndex = false;
+        $this->foundValue = null;
+
         $this->loop();
     }
 

src/SearchAbstract.php

@@ -21,17 +21,17 @@ abstract class SearchAbstract implements SearchInterface
     /**
      * The found data in which index
      */
-    protected int|string $foundIndex = -1;
+    protected int|string|bool $foundIndex = false;
 
     /**
      * The found data
      */
     protected mixed $foundValue = null;
 
     /**
-     * @return int|string
+     * @return int|string|bool
      */
-    public function getFoundIndex(): int|string
+    public function getFoundIndex(): int|string|bool
     {
         return $this->foundIndex;
     }

src/SearchInterface.php

@@ -12,7 +12,7 @@ public function setSearchValue(int|string $searchValue): self;
 
     public function getSearchValue(): null|int|string;
 
-    public function getFoundIndex(): null|int|string;
+    public function getFoundIndex(): null|int|string|bool;
 
     public function getFoundValue(): mixed;
 

tests/BinarySearchTest.php

@@ -76,10 +76,16 @@ public function testBinarySearchLoopNormalArray(): void
 
         $this->assertSame(98588, $search->getFoundIndex());
         $this->assertSame(98589, $search->getFoundValue());
+    }
+
+    public function testBinarysearchNotFoundData(): void
+    {
+        $array = [];
+
+        for ($index = 0; $index < 100 * 10000; $index++) {
+            $array[] = $index + 1;
+        }
 
-        /**
-         * There is not found data
-         */
         $search = new BinarySearch();
         $search
             ->setCompareCallback(fn ($current, $searchValue) => $current === $searchValue)
@@ -88,7 +94,9 @@ public function testBinarySearchLoopNormalArray(): void
             ->setSearchValue(999999999)
             ->search();
 
-        $this->assertSame(-1, $search->getFoundIndex());
+        $foundIndex = $search->getFoundIndex();
+
+        $this->assertFalse($foundIndex);
     }
 
     public function testBinarySearchLoopArraysInArray(): void