Skip to content

Commit

Permalink
Refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
sebastianbergmann committed Mar 24, 2024
1 parent 12705cc commit dd83dcd
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
11 changes: 7 additions & 4 deletions src/FilePrinter.php → src/FileRenderer.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,26 +11,29 @@

use function array_flip;
use function array_key_exists;
use function printf;
use function rtrim;
use function sprintf;

final readonly class FilePrinter
final readonly class FileRenderer
{
public function print(File $file): void
public function render(File $file): string
{
$buffer = '';
$sourceLines = $file->sourceLines();
$eliminatedLines = array_flip($file->linesEliminatedByOptimizer());
$line = 0;

foreach ($sourceLines as $sourceLine) {
$line++;

printf(
$buffer .= sprintf(
'%s %-6d %s' . PHP_EOL,
array_key_exists($line, $eliminatedLines) ? '-' : ' ',
$line,
rtrim($sourceLine),
);
}

return $buffer;
}
}
6 changes: 3 additions & 3 deletions src/cli/Application.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
use function is_file;
use function printf;
use SebastianBergmann\FOAL\Analyser;
use SebastianBergmann\FOAL\FilePrinter;
use SebastianBergmann\FOAL\FileRenderer;

final readonly class Application
{
Expand Down Expand Up @@ -70,9 +70,9 @@ public function run(array $argv): int

$file = $this->analyser->analyse($arguments->file());

$printer = new FilePrinter;
$renderer = new FileRenderer;

$printer->print($file);
print $renderer->render($file);

return 0;
}
Expand Down

0 comments on commit dd83dcd

Please sign in to comment.