Refactor

Download the completed sample files

Multiple Iterations of the Same Code

Option 1: List of elseif statements

if (in_array($firstThreeLetters, $this->trigraphs)) {
    $newWord = substr($word, 3);
    $newWord .= $firstThreeLetters . 'ay';
} elseif (in_array($firstTwoLetters, $this->digraphs)) {
    $newWord = substr($word, 2);
    $newWord .= $firstTwoLetters . 'ay';
} else {
    $newWord = substr($word, 1);
    $newWord .= $firstLetter . 'ay';
}
return $newWord;

Option 2: Early Return

if (in_array($firstThreeLetters, $this->trigraphs)) {
    return substr($word, 3) . $firstThreeLetters . 'ay';
}
if (in_array($firstTwoLetters, $this->digraphs)) {
    return substr($word, 2)  . $firstTwoLetters . 'ay';
}

return substr($word, 1) . $firstLetter . 'ay';

Option 3: Switch Statement

switch (true) {
    case in_array($firstLetter, $this->vowels):
        $newWord = $word . 'ay';
        break;
    case in_array($firstThreeLetters, $this->trigraphs):
        $newWord = substr($word, 3);
        $newWord .= $firstThreeLetters . 'ay';
        break;
    case in_array($firstTwoLetters, $this->digraphs):
        $newWord = substr($word, 2);
        $newWord .= $firstTwoLetters . 'ay';
        break;
    default:
        $newWord = substr($word, 1);
        $newWord .= $firstLetter . 'ay';
}
return $newWord;

Option 3b: Switch Statement Early Return

switch (true) {
    case in_array($firstLetter, $this->vowels):
        $newWord = $word . 'ay';
        return $newWord;
    case in_array($firstThreeLetters, $this->trigraphs):
        $newWord = substr($word, 3);
        $newWord .= $firstThreeLetters . 'ay';
        return $newWord;
    case in_array($firstTwoLetters, $this->digraphs):
        $newWord = substr($word, 2);
        $newWord .= $firstTwoLetters . 'ay';
        return $newWord;
    default:
        $newWord = substr($word, 1);
        $newWord .= $firstLetter . 'ay';
        return $newWord;
}