Use Media object instead of array

Author: jhaoda

src/Contract/Label.php

@@ -18,9 +18,9 @@
 
 interface Label extends Job, \IteratorAggregate
 {
-    public function getMedia(): array;
+    public function getCopies(): int;
 
     public function getCharset(): ?Charset;
 
-    public function getCopies(): int;
+    public function getMedia(): Media;
 }

src/Contract/Media.php

@@ -0,0 +1,24 @@
+<?php
+
+/**
+ * This file is part of Appwilio LabelPrinter package.
+ *
+ *  © appwilio (https://appwilio.com)
+ *  © JhaoDa (https://github.com/jhaoda)
+ *
+ *  For the full copyright and license information, please view the LICENSE
+ *  file that was distributed with this source code.
+ */
+
+namespace PhpAidc\LabelPrinter\Contract;
+
+use PhpAidc\LabelPrinter\Enum\Unit;
+
+interface Media
+{
+    public function getUnit(): ?Unit;
+
+    public function getWidth(): ?float;
+
+    public function getHeight(): ?float;
+}

src/Label/Label.php

@@ -19,12 +19,11 @@
 use PhpAidc\LabelPrinter\Enum\Direction;
 use PhpAidc\LabelPrinter\Contract\Command;
 use PhpAidc\LabelPrinter\Contract\Condition;
+use PhpAidc\LabelPrinter\Contract\Media as MediaContract;
 use PhpAidc\LabelPrinter\Contract\Label as LabelContract;
 
 final class Label implements LabelContract
 {
-    private $media;
-
     /** @var int */
     private $copies = 1;
 
@@ -34,6 +33,9 @@ final class Label implements LabelContract
     /** @var Direction|null */
     private $direction;
 
+    /** @var Media */
+    private $media;
+
     /** @var Command[]|Condition[] */
     private $statements = [];
 
@@ -48,7 +50,7 @@ public function __construct(?Unit $unit = null, ?float $width = null, ?float $he
             throw new \InvalidArgumentException();
         }
 
-        $this->media = \compact('unit', 'width', 'height');
+        $this->media = new Media($unit, $width, $height);
     }
 
     public function add(Command $command)
@@ -94,7 +96,7 @@ public function getDirection(): ?Direction
         return $this->direction;
     }
 
-    public function getMedia(): array
+    public function getMedia(): MediaContract
     {
         return $this->media;
     }

src/Label/Media.php

@@ -15,8 +15,9 @@
 namespace PhpAidc\LabelPrinter\Label;
 
 use PhpAidc\LabelPrinter\Enum\Unit;
+use PhpAidc\LabelPrinter\Contract\Media as MediaContract;
 
-final class Media
+final class Media implements MediaContract
 {
     /** @var Unit|null */
     private $unit;

src/Language/Fingerprint.php

@@ -23,6 +23,7 @@
 use PhpAidc\LabelPrinter\Command\ExternalImage;
 use PhpAidc\LabelPrinter\Command\InternalImage;
 use PhpAidc\LabelPrinter\Contract\Label;
+use PhpAidc\LabelPrinter\Contract\Media;
 use PhpAidc\LabelPrinter\Contract\Command;
 use PhpAidc\LabelPrinter\Contract\Language;
 use PhpAidc\LabelPrinter\Enum\Unit;
@@ -100,16 +101,14 @@ private function translateCharset(?Charset $charset): iterable
         }
     }
 
-    private function translateMedia(array $media): iterable
+    private function translateMedia(Media $media): iterable
     {
-        ['unit' => $unit, 'width' => $width, 'height' => $height] = $media;
-
-        if ($width) {
-            yield \sprintf('SETUP "MEDIA,MEDIA SIZE,WIDTH,%d'.self::EOC, $this->valueToDots($width, $unit));
+        if ($media->getWidth()) {
+            yield \sprintf('SETUP "MEDIA,MEDIA SIZE,WIDTH,%d'.self::EOC, $this->valueToDots($media->getWidth(), $media->getUnit()));
         }
 
-        if ($height) {
-            yield \sprintf('SETUP "MEDIA,MEDIA SIZE,HEIGHT,%d'.self::EOC, $this->valueToDots($height, $unit));
+        if ($media->getHeight()) {
+            yield \sprintf('SETUP "MEDIA,MEDIA SIZE,HEIGHT,%d'.self::EOC, $this->valueToDots($media->getHeight(), $media->getUnit()));
         }
     }
 

src/Language/Tspl.php

@@ -22,6 +22,7 @@
 use PhpAidc\LabelPrinter\Command\TextBlock;
 use PhpAidc\LabelPrinter\Command\InternalImage;
 use PhpAidc\LabelPrinter\Contract\Label;
+use PhpAidc\LabelPrinter\Contract\Media;
 use PhpAidc\LabelPrinter\Contract\Command;
 use PhpAidc\LabelPrinter\Contract\Language;
 use PhpAidc\LabelPrinter\Enum\Unit;
@@ -83,20 +84,18 @@ private function translateCharset(?Charset $charset): iterable
         }
     }
 
-    private function translateMedia(array $media): iterable
+    private function translateMedia(Media $media): iterable
     {
-        ['unit' => $unit, 'width' => $width, 'height' => $height] = $media;
-
-        if ($width && $height === null) {
-            yield \sprintf('SIZE %s'.self::EOC, $this->valueWithUnit($width, $unit));
+        if ($media->getWidth() && $media->getHeight() === null) {
+            yield \sprintf('SIZE %s'.self::EOC, $this->valueWithUnit($media->getWidth(), $media->getUnit()));
 
             return;
         }
 
-        if ($width && $height) {
+        if ($media->getWidth() && $media->getHeight()) {
             yield \vsprintf('SIZE %s,%s'.self::EOC, [
-                $this->valueWithUnit($width, $unit),
-                $this->valueWithUnit($height, $unit),
+                $this->valueWithUnit($media->getWidth(), $media->getUnit()),
+                $this->valueWithUnit($media->getHeight(), $media->getUnit()),
             ]);
         }
     }