Coco\SourceWatcher\Core\Transformers\GuessGenderTransformer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	transform()	0	15	2

<?php

namespace Coco\SourceWatcher\Core\Transformers;

use Coco\SourceWatcher\Core\Row;
use Coco\SourceWatcher\Core\Transformer;

use GenderDetector\GenderDetector;

/**
 *
 */
class GuessGenderTransformer extends Transformer
{
    /**
     * @var string
     */
    protected string $country;

    /**
     * @var string
     */
    protected string $firstNameField;

    /**
     * @var string
     */
    protected string $genderField;

    /**
     * @var array|string[]
     */
    protected array $availableOptions = [ "country", "firstNameField", "genderField" ];

    /**
     * @param Row $row
     */
    public function transform ( Row $row )
    {
        $firstName = $row->get($this->firstNameField);
        echo "firstName = " . $firstName . PHP_EOL;

        $currentGender = $row->get($this->genderField);
        echo "currentGender = " . $currentGender . PHP_EOL;

        if (empty($currentGender)) {
            $genderDetector = new GenderDetector();

            $gender = $genderDetector->detect($firstName, $this->country);

It seems like $firstName can also be of type null; however, parameter $name of GenderDetector\GenderDetector::detect() does only seem to accept string, maybe add an additional type check?

            echo "gender = " . $gender . PHP_EOL;

            $row->set( $this->genderField, $gender );
        }
    }
}