Initial implementation

cybercog · Oct 14, 2023 · b35a68d · b35a68d
1 parent 9cc9b96
commit b35a68d
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 13 deletions.
diff --git a/src/FontList.php b/src/FontList.php
@@ -6,21 +6,36 @@
 
 final class FontList
 {
-    private array $fonts;
+    private array $fontList;
 
     /**
-     * @param list<Font> $fonts
+     * @param list<Font> $fontList
      */
-    public function __construct(
-        array $fonts
+    private function __construct(
+        array $fontList
     ) {
-        $this->fonts = $fonts;
+        $this->fontList = $fontList;
+    }
+
+    /**
+     * @param list<Font> $fontList
+     */
+    public static function of(
+        array $fontList
+    ): self {
+        foreach ($fontList as $font) {
+            if (!($font instanceof Font)) {
+                throw new \InvalidArgumentException('Cannot instantiate FontList with ');
+            }
+        }
+
+        return new self($fontList);
     }
 
     public function getById(
         string $id
     ): Font {
-        foreach ($this->fonts as $font) {
+        foreach ($this->fontList as $font) {
             if ($font->id() === $id) {
                 return $font;
             }

diff --git a/src/Parser/SimpleXmlSvgFontFileParser.php b/src/Parser/SimpleXmlSvgFontFileParser.php
@@ -39,15 +39,17 @@ public function parseFile(string $filePath): FontList
         $xml = simplexml_load_file($filePath);
         $xml->registerXPathNamespace('svg', 'http://www.w3.org/2000/svg');
 
-        $fonts = [];
+        $fontLists = [];
 
         $fontElements = $xml->xpath('//svg:defs/svg:font');
 
         foreach ($fontElements as $fontElement) {
-            $fonts[] = $this->initFont($fontElement);
+            $fontLists[] = $this->initFont($fontElement);
         }
 
-        return new FontList($fonts);
+        return FontList::of(
+            $fontLists,
+        );
     }
 
     private function initFont(

diff --git a/src/Parser/XmlReaderSvgFontFileParser.php b/src/Parser/XmlReaderSvgFontFileParser.php
@@ -41,7 +41,7 @@ public function parseFile(
         $xml = new \XMLReader();
         $xml->open($filePath);
 
-        $fonts = [];
+        $fontList = [];
 
         while ($xml->read()) {
             if ($xml->name === self::ELEMENT_NAME_FONT) {
@@ -54,7 +54,7 @@ public function parseFile(
                 }
 
                 if ($xml->nodeType === \XMLReader::END_ELEMENT) {
-                    $fonts[] = new Font(
+                    $fontList[] = new Font(
                         $fontId,
                         $defaultHorizontalAdvance,
                         $fontFace,
@@ -93,8 +93,8 @@ public function parseFile(
 
         $xml->close();
 
-        return new FontList(
-            $fonts,
+        return FontList::of(
+            $fontList,
         );
     }