code style

robsonvleite · Oct 3, 2018 · 5f458ee · 5f458ee
1 parent 414cbe4
commit 5f458ee
Showing 1 changed file with 17 additions and 14 deletions.
diff --git a/src/Paginator.php b/src/Paginator.php
@@ -13,6 +13,9 @@ class Paginator
     /** @var int */
     private $page;
 
+    /** @var int */
+    private $pages;
+
     /** @var int */
     private $rows;
 
@@ -31,6 +34,9 @@ class Paginator
     /** @var string */
     private $title;
 
+    /** @var string */
+    private $class;
+
     /** @var string */
     private $hash;
 
@@ -40,9 +46,6 @@ class Paginator
     /** @var array */
     private $last;
 
-    /** @var string */
-    private $cssClass;
-
     /**
      * Paginator constructor.
      * @param string|null $link
@@ -103,16 +106,16 @@ public function offset(): int
      */
     public function render(string $cssClass = "paginator"): ?string
     {
-        $this->cssClass = $cssClass;
+        $this->class = $cssClass;
+        $this->pages = (int)ceil($this->rows / $this->limit);
 
         if ($this->rows > $this->limit):
-            $pages = ceil($this->rows / $this->limit);
-            $paginator = "<nav class=\"{$this->cssClass}\">";
-            $paginator .= "<a class='{$this->cssClass}_item' title=\"{$this->first[0]}\" href=\"{$this->link}1{$this->hash}\">{$this->first[1]}</a>";
+            $paginator = "<nav class=\"{$this->class}\">";
+            $paginator .= "<a class='{$this->class}_item' title=\"{$this->first[0]}\" href=\"{$this->link}1{$this->hash}\">{$this->first[1]}</a>";
             $paginator .= $this->beforePages();
-            $paginator .= "<span class=\"{$this->cssClass}_item {$this->cssClass}_active\">{$this->page}</span>";
-            $paginator .= $this->afterPages($pages);
-            $paginator .= "<a class='{$this->cssClass}_item' title=\"{$this->last[0]}\" href=\"{$this->link}{$pages}{$this->hash}\">{$this->last[1]}</a>";
+            $paginator .= "<span class=\"{$this->class}_item {$this->class}_active\">{$this->page}</span>";
+            $paginator .= $this->afterPages();
+            $paginator .= "<a class='{$this->class}_item' title=\"{$this->last[0]}\" href=\"{$this->link}{$this->pages}{$this->hash}\">{$this->last[1]}</a>";
             $paginator .= "</nav>";
             return $paginator;
         endif;
@@ -128,7 +131,7 @@ private function beforePages(): ?string
         $before = null;
         for ($iPag = $this->page - $this->range; $iPag <= $this->page - 1; $iPag++):
             if ($iPag >= 1):
-                $before .= "<a class='{$this->cssClass}_item' title=\"{$this->title} {$iPag}\" href=\"{$this->link}{$iPag}{$this->hash}\">{$iPag}</a>";
+                $before .= "<a class='{$this->class}_item' title=\"{$this->title} {$iPag}\" href=\"{$this->link}{$iPag}{$this->hash}\">{$iPag}</a>";
             endif;
         endfor;
 
@@ -139,12 +142,12 @@ private function beforePages(): ?string
      * @param int $pages
      * @return null|string
      */
-    private function afterPages(int $pages): ?string
+    private function afterPages(): ?string
     {
         $after = null;
         for ($dPag = $this->page + 1; $dPag <= $this->page + $this->range; $dPag++):
-            if ($dPag <= $pages):
-                $after .= "<a class='{$this->cssClass}_item' title=\"{$this->title} {$dPag}\" href=\"{$this->link}{$dPag}{$this->hash}\">{$dPag}</a>";
+            if ($dPag <= $this->pages):
+                $after .= "<a class='{$this->class}_item' title=\"{$this->title} {$dPag}\" href=\"{$this->link}{$dPag}{$this->hash}\">{$dPag}</a>";
             endif;
         endfor;