PHPでデータベースから取り出した文字列の字数制限について

Question

現在の状況

データベースにデータを入れて、HTMLで表示できる所まではできているのですが、
title部分の表示が指定した文字数で表示されず、途中で切れてしまいます。
何故か更新（UPDATE）をしてあげると正しく文字が指定した字数が表示されます。

何か原因はわかりますでしょうか？
宜しくお願い致します。m(_ _)m

<!-- 表示用のHTML -->
<?php foreach($query as $row): ?>
<tr>
    <td><?php echo $row['category']; ?></td>
    <td><a href="###"><?php echo $row['title']; ?></a></td>
    <td><?php echo mb_substr($row['contents'], 0,33); ?></td>
    <td><?php echo mb_substr($row['remarks'], 0, 27); ?></td>
    <td><?php echo $row['history']; ?></td>
    <td>
        <button class="copyBtn">コピー</button>
        <textarea class="copyTxt"><?php echo $row['contents']; ?></textarea>

        <form action="edit.php" method="GET">
            <input type="hidden" name="id" value="<?php echo $row['id']; ?>">
            <input type="submit" value="編集">
        </form>
    </td>
</tr>

<?php endforeach; $dsn = null; ?>

//新規作成画面

<?php

$var = ['category', 'title', 'contents', 'remarks'];

foreach($var as $row){
    ${$row} = filter_input(INPUT_POST, $row, FILTER_SANITIZE_SPECIAL_CHARS);
}

$history = date_default_timezone_set("Asia/Tokyo");
$history = date("Y/m/d H:i");

include dirname(__FILE__) . '/db.php';

$query = "INSERT INTO mail(category, title, contents, remarks, history) VALUES(?, ?, ?, ?, ?)";
$post = $dsn->prepare($query);
$data[] = $category;
$data[] = $title;
$data[] = $contents;
$data[] = $remarks;
$data[] = $history;

$res = $post->execute($data);

$dsn = null;
header("Location: home.php");

?>

//更新画面用

<?php

include dirname(__FILE__) . '/db.php';

$id = $_POST['id'];
$val = ["category", "title", "contents", "remarks"];

$history = date_default_timezone_set("Asia/Tokyo");
$history = date("Y/m/d H:i");

foreach($val as $res){
    ${$res} = filter_input(INPUT_POST, $res);
}

$dsn->query("UPDATE mail SET category='$category' WHERE id='$id'");
$dsn->query("UPDATE mail SET title='$title' WHERE id='$id'");
$dsn->query("UPDATE mail SET contents='$contents' WHERE id='$id'");
$dsn->query("UPDATE mail SET remarks='$remarks' WHERE id='$id'");
$dsn->query("UPDATE mail SET history='$history' WHERE id='$id'");

header("Location: home.php");
?>

上記のテスト2の状態が正しい表示になります。（新規入稿をした後一度そのまま変更なしで更新（UPDATE）をするとこのような表示になります。
現在は下のテストのような状態で文字制限がうまく効いてくれていない状態です。

Comments

[sola-msr]

DBに入っているデータの状態（スクショ）も記載したほうがいいかと思います

[m.ts10806]

文字数ではなくバイト数と言うのを忘れては行けません
https://www.php.net/manual/ja/function.mb-substr.php

[rinrin1137]

ありがとうございます。
参考にさせていただきます。

Answer 1

<?php
    if(mb_strlen($row['contents']) > 38){
        echo mb_substr($row['contents'], 0,35)."...";
    }else{
        echo $row['contents'];
    }
?>

とりあえず上記コードで解決いたしました。

Comments

[退会済みユーザー]

この更新画面コードはデータの破壊を招く危険なコードですよ。

[rinrin1137]

具体的にはどこの部分が危険でしょうか？

[rinrin1137]

可変変数で生成している部分をやめたら上記の字数制限問題はあっさり直りました・・・

[退会済みユーザー]

$dsn->query("UPDATE mail SET category='$category' WHERE id='$id'");
$dsn->query("UPDATE mail SET title='$title' WHERE id='$id'");
$dsn->query("UPDATE mail SET contents='$contents' WHERE id='$id'");
$dsn->query("UPDATE mail SET remarks='$remarks' WHERE id='$id'");
$dsn->query("UPDATE mail SET history='$history' WHERE id='$id'");

ここにSQLインジェクションの危険性があります。
また、一回のクエリで済む処理を5回に分けているのも無駄です。

[rinrin1137]

$state = $db-> prepare("SELECT * FROM users WHERE name=? AND pass=?");
$state-> execute(array($user, $pass));

上記のようにプレースホルダを使いSQLと値の挿入を切り分ければ安全でしょうか？