Sync transpose (exercism#1016)

Author: resu-xuniL

bin/auto-sync.txt

@@ -91,6 +91,7 @@ state-of-tic-tac-toe
 strain
 sublist
 swift-scheduling
+transpose
 tournament
 triangle
 twelve-days

config.json

@@ -489,22 +489,6 @@
           "floating_point_numbers"
         ]
       },
-      {
-        "slug": "transpose",
-        "name": "Transpose",
-        "uuid": "cb413881-99b6-493f-bea2-89563dc173a6",
-        "practices": [],
-        "prerequisites": [],
-        "difficulty": 1,
-        "topics": [
-          "arrays",
-          "lists",
-          "loops",
-          "matrices",
-          "strings",
-          "text_formatting"
-        ]
-      },
       {
         "slug": "triangle",
         "name": "Triangle",
@@ -763,6 +747,22 @@
           "math"
         ]
       },
+      {
+        "slug": "transpose",
+        "name": "Transpose",
+        "uuid": "cb413881-99b6-493f-bea2-89563dc173a6",
+        "practices": [],
+        "prerequisites": [],
+        "difficulty": 2,
+        "topics": [
+          "arrays",
+          "lists",
+          "loops",
+          "matrices",
+          "strings",
+          "text_formatting"
+        ]
+      },
       {
         "slug": "twelve-days",
         "name": "Twelve Days",

exercises/practice/transpose/.docs/instructions.md

@@ -17,7 +17,8 @@ BE
 CF
 ```
 
-Rows become columns and columns become rows. See <https://en.wikipedia.org/wiki/Transpose>.
+Rows become columns and columns become rows.
+See [transpose][].
 
 If the input has rows of different lengths, this is to be solved as follows:
 
@@ -55,5 +56,6 @@ BE
 ```
 
 In general, all characters from the input should also be present in the transposed output.
-That means that if a column in the input text contains only spaces on its bottom-most row(s),
-the corresponding output row should contain the spaces in its right-most column(s).
+That means that if a column in the input text contains only spaces on its bottom-most row(s), the corresponding output row should contain the spaces in its right-most column(s).
+
+[transpose]: https://en.wikipedia.org/wiki/Transpose

exercises/practice/transpose/.meta/config.json

@@ -6,7 +6,8 @@
     "arueckauer",
     "kunicmarko20",
     "kytrinyx",
-    "petemcfarlane"
+    "petemcfarlane",
+    "resu-xuniL"
   ],
   "files": {
     "solution": [
@@ -21,5 +22,5 @@
   },
   "blurb": "Take input text and output it transposed.",
   "source": "Reddit r/dailyprogrammer challenge #270 [Easy].",
-  "source_url": "https://www.reddit.com/r/dailyprogrammer/comments/4msu2x/challenge_270_easy_transpose_the_input_text"
+  "source_url": "https://web.archive.org/web/20230630051421/https://old.reddit.com/r/dailyprogrammer/comments/4msu2x/challenge_270_easy_transpose_the_input_text/"
 }

exercises/practice/transpose/.meta/example.php

@@ -1,27 +1,5 @@
 <?php
 
-/*
- * By adding type hints and enabling strict type checking, code can become
- * easier to read, self-documenting and reduce the number of potential bugs.
- * By default, type declarations are non-strict, which means they will attempt
- * to change the original type to match the type specified by the
- * type-declaration.
- *
- * In other words, if you pass a string to a function requiring a float,
- * it will attempt to convert the string value to a float.
- *
- * To enable strict mode, a single declare directive must be placed at the top
- * of the file.
- * This means that the strictness of typing is configured on a per-file basis.
- * This directive not only affects the type declarations of parameters, but also
- * a function's return type.
- *
- * For more info review the Concept on strict type checking in the PHP track
- * <link>.
- *
- * To disable strict typing, comment out the directive below.
- */
-
 declare(strict_types=1);
 
 /**
@@ -34,25 +12,19 @@
  */
 function transpose($text)
 {
-    $findMaxLength = function ($lines) {
-        return array_reduce($lines, function ($max, $line) {
-            return max($max, strlen($line));
-        }, 0);
-    };
+    $pad = function ($lines) {
+        for ($i = count($lines) - 2; $i >= 0; $i--) {
+            $lines[$i] = str_pad($lines[$i], strlen($lines[$i + 1]), ' ', STR_PAD_RIGHT);
+        }
 
-    $pad = function ($lines, $length) {
-        return array_map(function ($line) use ($length) {
-            return str_pad($line, $length, ' ', STR_PAD_RIGHT);
-        }, $lines);
+        return $lines;
     };
 
     if ($text === ['']) {
         return $text;
     }
 
-    $maxLength = $findMaxLength($text);
-
-    $lines = $pad($text, $maxLength);
+    $lines = $pad($text);
 
     $result = [];
 

exercises/practice/transpose/.meta/tests.toml

@@ -1,6 +1,13 @@
-# This is an auto-generated file. Regular comments will be removed when this
-# file is regenerated. Regenerating will not touch any manually added keys,
-# so comments can be added in a "comment" key.
+# This is an auto-generated file.
+#
+# Regenerating this file via `configlet sync` will:
+# - Recreate every `description` key/value pair
+# - Recreate every `reimplements` key/value pair, where they exist in problem-specifications
+# - Remove any `include = true` key/value pair (an omitted `include` key implies inclusion)
+# - Preserve any other key/value pair
+#
+# As user-added comments (using the # character) will be removed when this file
+# is regenerated, comments can be added via a `comment` key.
 
 [404b7262-c050-4df0-a2a2-0cb06cd6a821]
 description = "empty string"
@@ -34,3 +41,6 @@ description = "rectangle"
 
 [b80badc9-057e-4543-bd07-ce1296a1ea2c]
 description = "triangle"
+
+[76acfd50-5596-4d05-89f1-5116328a7dd9]
+description = "jagged triangle"

exercises/practice/transpose/TransposeTest.php

@@ -1,30 +1,9 @@
 <?php
 
-/*
- * By adding type hints and enabling strict type checking, code can become
- * easier to read, self-documenting and reduce the number of potential bugs.
- * By default, type declarations are non-strict, which means they will attempt
- * to change the original type to match the type specified by the
- * type-declaration.
- *
- * In other words, if you pass a string to a function requiring a float,
- * it will attempt to convert the string value to a float.
- *
- * To enable strict mode, a single declare directive must be placed at the top
- * of the file.
- * This means that the strictness of typing is configured on a per-file basis.
- * This directive not only affects the type declarations of parameters, but also
- * a function's return type.
- *
- * For more info review the Concept on strict type checking in the PHP track
- * <link>.
- *
- * To disable strict typing, comment out the directive below.
- */
-
 declare(strict_types=1);
 
 use PHPUnit\Framework\TestCase;
+use PHPUnit\Framework\Attributes\TestDox;
 
 class TransposeTest extends TestCase
 {
@@ -33,13 +12,21 @@ public static function setUpBeforeClass(): void
         require_once 'Transpose.php';
     }
 
+    /**
+     * uuid: 404b7262-c050-4df0-a2a2-0cb06cd6a821
+     */
+    #[TestDox('Empty string')]
     public function testEmptyString(): void
     {
         $input = [""];
         $expected = [""];
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: a89ce8a3-c940-4703-a688-3ea39412fbcb
+     */
+    #[TestDox('Two characters in a row')]
     public function testTwoCharactersInARow(): void
     {
         $input = [
@@ -52,6 +39,10 @@ public function testTwoCharactersInARow(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: 855bb6ae-4180-457c-abd0-ce489803ce98
+     */
+    #[TestDox('Two characters in a column')]
     public function testTwoCharactersInAColumn(): void
     {
         $input = [
@@ -64,6 +55,10 @@ public function testTwoCharactersInAColumn(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: 5ceda1c0-f940-441c-a244-0ced197769c8
+     */
+    #[TestDox('Simple')]
     public function testSimple(): void
     {
         $input = [
@@ -78,6 +73,10 @@ public function testSimple(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: a54675dd-ae7d-4a58-a9c4-0c20e99a7c1f
+     */
+    #[TestDox('Single line')]
     public function testSingleLine(): void
     {
         $input = [
@@ -100,6 +99,10 @@ public function testSingleLine(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: 0dc2ec0b-549d-4047-aeeb-8029fec8d5c5
+     */
+    #[TestDox('First line longer than second line')]
     public function testFirstLineLongerThanSecondLine(): void
     {
         $input = [
@@ -127,6 +130,10 @@ public function testFirstLineLongerThanSecondLine(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: 984e2ec3-b3d3-4b53-8bd6-96f5ef404102
+     */
+    #[TestDox('Second line longer than first line')]
     public function testSecondLineLongerThanFirstLine(): void
     {
         $input = [
@@ -154,6 +161,44 @@ public function testSecondLineLongerThanFirstLine(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: eccd3784-45f0-4a3f-865a-360cb323d314
+     */
+    #[TestDox('Mixed line length')]
+    public function testMixedLineLength(): void
+    {
+        $input = [
+            "The longest line.",
+            "A long line.",
+            "A longer line.",
+            "A line.",
+        ];
+        $expected = [
+            "TAAA",
+            "h   ",
+            "elll",
+            " ooi",
+            "lnnn",
+            "ogge",
+            "n e.",
+            "glr",
+            "ei ",
+            "snl",
+            "tei",
+            " .n",
+            "l e",
+            "i .",
+            "n",
+            "e",
+            ".",
+        ];
+        $this->assertEquals($expected, transpose($input));
+    }
+
+    /**
+     * uuid: 85b96b3f-d00c-4f80-8ca2-c8a5c9216c2d
+     */
+    #[TestDox('Square')]
     public function testSquare(): void
     {
         $input = [
@@ -173,6 +218,10 @@ public function testSquare(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: b9257625-7a53-4748-8863-e08e9d27071d
+     */
+    #[TestDox('Rectangle')]
     public function testRectangle(): void
     {
         $input = [
@@ -194,6 +243,10 @@ public function testRectangle(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: b80badc9-057e-4543-bd07-ce1296a1ea2c
+     */
+    #[TestDox('Triangle')]
     public function testTriangle(): void
     {
         $input = [
@@ -215,6 +268,32 @@ public function testTriangle(): void
         $this->assertEquals($expected, transpose($input));
     }
 
+    /**
+     * uuid: 76acfd50-5596-4d05-89f1-5116328a7dd9
+     */
+    #[TestDox('Jagged triangle')]
+    public function testJaggedTriangle(): void
+    {
+        $input = [
+            "11",
+            "2",
+            "3333",
+            "444",
+            "555555",
+            "66666",
+        ];
+        $expected = [
+            "123456",
+            "1 3456",
+            "  3456",
+            "  3 56",
+            "    56",
+            "    5",
+        ];
+        $this->assertEquals($expected, transpose($input));
+    }
+
+    #[TestDox('Many lines')]
     public function testManyLines(): void
     {
         $input = [