Fixing #3 (after only so many years)

Author: fanatique

README.md

@@ -45,7 +45,7 @@ $parser = new \Fanatique\Parser\FixedLengthFileParser();
 // Set the chopping map (aka where to extract the fields)
 $parser->setChoppingMap([
     ['field_name' => 'id', 'start' => 0, 'length' => 2],
-    ['field_name' => 'name', 'start' => 2, 'length' => 5],
+    ['field_name' => 'name', 'start' => 2, 'length' => 5, 'align' => 'left'],
     ['field_name' => 'team', 'start' => 7, 'length' => 5],
 ]);
 
@@ -66,6 +66,7 @@ var_dump($parser->getContent());
 
 `field_name` and `length` are required. `start` is optional — if omitted, it is
 calculated from the previous entry's `start + length`.
+`align` is optional and configures which padding spaces to trim: `'left'` (removes trailing), `'right'` (removes leading), or `'both'` (the default).
 
 ### Registering a pre-flight check
 

example/parsing.php

@@ -21,7 +21,7 @@
 // Set the chopping map (aka where to extract the fields)
 $parser->setChoppingMap([
     ['field_name' => 'id', 'length' => 2],
-    ['field_name' => 'name', 'start' => 2, 'length' => 5],
+    ['field_name' => 'name', 'start' => 2, 'length' => 5, 'align' => 'left'],
     ['field_name' => 'team', 'length' => 5], // start is the sum of name:start(2) plus name:length(5) = 7
 ]);
 

src/FixedLengthFileParser.php

@@ -25,7 +25,7 @@ class FixedLengthFileParser implements ParserInterface
      * Arrays describing field_name, start, and length for each value
      * encoded in a line of the file to be parsed.
      *
-     * @var array<int, array{field_name: string, start?: int, length: int}>
+     * @var array<int, array{field_name: string, start?: int, length: int, align?: 'left'|'right'|'both'}>
      */
     protected array $choppingMap = [];
 
@@ -66,9 +66,10 @@ class FixedLengthFileParser implements ParserInterface
      *
      * Each entry must contain 'field_name' and 'length'. The 'start' key is
      * optional — if omitted, it is calculated from the previous entry's
-     * start + length.
+     * start + length. The 'align' key is optional and can be 'left', 'right',
+     * or 'both' (default is 'both').
      *
-     * @param array<int, array{field_name: string, start?: int, length: int}> $map
+     * @param array<int, array{field_name: string, start?: int, length: int, align?: 'left'|'right'|'both'}> $map
      */
     public function setChoppingMap(array $map): void
     {
@@ -167,9 +168,15 @@ protected function parseLine(string $buffer): array
                 : $start + $this->choppingMap[$i]['length'];
 
             $name = $this->choppingMap[$i]['field_name'];
-            $currentLine[$name] = trim(
-                substr($buffer, $start, $this->choppingMap[$i]['length'])
-            );
+            $rawValue = substr($buffer, $start, $this->choppingMap[$i]['length']);
+
+            $align = $this->choppingMap[$i]['align'] ?? 'both';
+
+            $currentLine[$name] = match ($align) {
+                'left' => rtrim($rawValue),
+                'right' => ltrim($rawValue),
+                default => trim($rawValue),
+            };
         }
 
         // Apply callback if registered

tests/FixedLengthFileParserTest.php

@@ -293,4 +293,30 @@ public function testEmptyChoppingMapReturnsEmptyRows(): void
             $this->assertSame([], $row);
         }
     }
+    public function testParseWithFieldAlignment(): void
+    {
+        $map = [
+            ['field_name' => 'left_aligned', 'length' => 10, 'align' => 'left'],
+            ['field_name' => 'right_aligned', 'length' => 10, 'align' => 'right'],
+            ['field_name' => 'default_align', 'length' => 10],
+        ];
+
+        $parser = new FixedLengthFileParser();
+        $parser->setChoppingMap($map);
+        
+        $tempFile = sys_get_temp_dir() . '/alignment_test.dat';
+        file_put_contents($tempFile, "   abc       123       xyz    \n");
+        
+        $parser->setFilePath($tempFile);
+        $parser->parse();
+        
+        $content = $parser->getContent();
+        
+        $this->assertCount(1, $content);
+        $this->assertSame('   abc', $content[0]['left_aligned']); // left removes right padding
+        $this->assertSame('123    ', $content[0]['right_aligned']); // right removes left padding
+        $this->assertSame('xyz', $content[0]['default_align']); // default removes both padding
+        
+        unlink($tempFile);
+    }
 }