validateメソッドが実行されない

Question

**先に質問文を書かせていただいた後に、ソースコードと、実行した際の画像を添付しております。

ーーー起きている問題ーーー
読書ログを作っており、
タイトルを入力せずに登録した場合、もしくは255文字以上で登録した場合、エラーメッセージを表示したいのですが、
ターミナルからタイトルを空で登録を実行したところ、登録が可能で、mysqlから確認しても登録ができてしまいます。

ーーー聞きたいことーーー
２つあります。
ーー１つ目ーー
validate関数、もしくはcreateReview関数の中の、validate関数を呼び出し元で間違いがございますでしょうか？

ーー２つ目ーー
$validated 定義下のif文の条件ですが、元々は、　if (count $($validated) > 0)
という条件式でやっていたのですが、countが指定できるのは配列に対してのみという以下のエラー文が出てきたため、issetで代替しています。
count(): Parameter must be an array or an object that implements Countable

もともとvalidate関数では＄errorsという配列に文章を入れているため、何故countが指定できないのかがわかっていません。
issetで代替できることはネットの記事を見つけて知って当てはめているのですが、
私のコードにもこのまま適応可能でしょうか？

以上２点気になっております。

ーー問題解決するために試したこと、考えたことーー
登録自体はできているため、登録する際の変数や連想配列には問題がないと考え、
validateメソッドもしくは、createReview関数の中の、validate関数を呼び出している$validatedの箇所に問題があるのかと思い英文字の打ち間違いなど確認しましたが、
間違いが見つけられなかったためこちらで質問させていただきました。

以下、ソースコードを貼らせていただきます

コード
<?php

$link = dbConnect();
$valuation = [];
$reviews = [];


function validate($valuation)
{
    $errors = [];
    //書籍名が正しく入力されているかチェック
if (!strlen($valuation['title'])) {
    $errors['title'] = '書籍名を入力してください';
} elseif (strlen($valuation['title']) > 255) {
    $errors['title'] = '書籍名は255文字以内で入力してください';
    return $errors;
}
}

function dbConnect() {
    $link = mysqli_connect('db', 'book_log', 'pass', 'book_log');
    if (!$link) {
        echo 'データベースに接続できませんでした' . PHP_EOL;
        echo 'debugging_error: ' . mysqli_connect_error() . PHP_EOL;
        exit;
    }
    echo 'データベースに接続できました' . PHP_EOL;

    return $link;
}



function createReview($link)
{
    echo '書籍名:';
    $valuation['title'] = trim(fgets(STDIN));

    echo '著者名:';
    $valuation['writer'] = trim(fgets(STDIN));

    echo '読書状況(未読、読んでる、読了):';
    $valuation['status'] = trim(fgets(STDIN));

    echo '評価(1~5):';
    $valuation['$score'] = trim(fgets(STDIN));

    echo '感想:';
    $valuation['$thoughts'] = trim(fgets(STDIN));


    $validated = validate($valuation);
    if (isset($validated)) {
        foreach($validated as $error) {
            echo $error . PHP_EOL;
        }
        return;
    }

    $sql = <<<EOT
INSERT INTO book_log (
    title,
    writer,
    status,
    score,
    thoughts
) VALUES (
    "{$valuation['title']}",
    "{$valuation['writer']}",
    "{$valuation['status']}",
    "{$valuation['$score']}",
    "{$valuation['$thoughts']}"
)
EOT;

    $result = mysqli_query($link, $sql);
    if ($result) {
        echo '登録が完了しました' . PHP_EOL . PHP_EOL;
    } else {
        echo 'データの追加に失敗しました' . PHP_EOL;
        echo 'debuggingError: ' . mysqli_error($link) . PHP_EOL . PHP_EOL;
    }
}


function displayReview ($reviews) {
    echo '読書ログを表示します' . PHP_EOL . PHP_EOL;

    foreach ($reviews as $review) {
        echo '書籍名:' . $review['title'] . PHP_EOL;
        echo '著者名:' . $review['writer'] . PHP_EOL;
        echo '読書状況:' . $review['status'] . PHP_EOL;
        echo '評価:' . $review['score'] . PHP_EOL;
        echo '感想:' . $review['thoughts'] . PHP_EOL;
        echo '-----------------'  . PHP_EOL;
    }
}



while (true) {
    echo '1.読書ログを登録' . PHP_EOL;
    echo '2.読書ログを表示' . PHP_EOL;
    echo '9.読書ログを終了' . PHP_EOL;
    echo '番号を選択してください(1,2,9) :';
    $num = trim(fgets(STDIN));

    if ($num === '1') {
        //複数登録
        createReview($link);
    } elseif($num === '2') {
        //読書ログを表示
        displayReview($reviews);
    } elseif($num === '9'){
        mysqli_close($link);
        echo 'データベースとの接続を切断しました' . PHP_EOL;
        break;
    }
}

以上ご教授いただければ幸いです。
また、私自身プログラミング初心者であり、質問の仕方があまり上手くわかっておりません。上記の質問で何か理解が難しい点、失礼な点がございましたら誠に申し訳ございません。

Answer 1

validate関数が255文字超のときかreturn $errorsしていませんね
$errors['title']が空のときや問題がないときの戻り値がありません

なおmysqliであってもprepareで処理して下さい

Comments

[hiro4040]

回答ありがとうございます。
仰る通り、if文の中にreturnを書いていて、
全体のreturn $errors ができていない状態でした。訂正したところ問題なく実行できました。

mysqli_queryとしているところのことでしょうか、そのような指定方法もあり、役割が少し違うのですね。prepareについてももう少し調べてみます。

とても早い回答で大変助かりました。