PHP三項演算子で、値がないときはどうするのがいいでしょうか？

Question

前提・実現したいこと

引数を展開する際に、その値は必ずしもある、というわけではないですよね。

そういったときに三項演算子が便利と知り取り入れたのですが、値がないときはどうするのがいいか知りたいです。

コード規約があれば従うべきですが、個人開発の初心者につきそういったものはございませんので、皆様の方法を教えて頂きたく存じます。

初心者につき至らない点多々あるかと思いますが、どうぞよろしくお願い申し上げます。

発生している問題・エラーメッセージ

以下hogeという関数でargsを受け取っています。

未定義や空文字に備え、これらの値の展開にあたって下記で三項演算子を使っていて、値がないときは``をセットするようにしました。

しかしoffsetは0がありえますので、nameやorder_byと同様にemptyで判定することができず、offsetだけ次のように複雑な書き方になってしまいました。

このような書き方はおかしいでしょうか。どうするのがいいか、アドバイス頂けましたら幸いです。

該当のソースコード

<?php
$args = [
    'name'    => 'smith',
    'order_by'=> 'created_at',
    'offset'  => 1,
];

function hoges( $args ) { 
    // 展開
    $name     = !empty($args['name']) ? $args['name'] : '';
    $order_by = !empty($args['order_by']) ? $args['order_by'] : '';
    $offset   = ( !empty($args['offset']) || ( isset($args['offset']) && $args['offset'] == 0 ) ) ? (int)$args['offset']  : '';

    // 展開した値を使い以下から色々な処理をやっていく    
}

試したこと

上記三つどれでも使える判定専用の関数を作ったらサッパリできると思ったのですが、未定義のときに「Undefined variable: v」のエラーとなってしまうために採用できませんでした。

// 未定義、null、空文字のときにtrueを返す
function my_is_empty( $v ) {
    $is = false;
    if ( !isset( $v ) ) {
        $is = true;
    }
    if ( is_null( $v ) ) {
        $is = true;
    }
    if ( $v === '' ) {
        $is = true;
    }    
    return $is;
}

// 実行すると未定義のときに「Undefined variable: v」のエラーとなってしまう
$v = '';    // ok
$v = null;  // ok
$v;         // 機能しない
var_dump( my_is_empty( $v ) );

補足情報（FW/ツールのバージョンなど）

レンタルサーバーの事情によりPHP var7.3.1 使用ですが、アップデートで楽になるならサーバーを変更しようと思っています。

Answer 1

しかしoffsetは0がありえますので、nameやorder_byと同様にemptyで判定することができず

issetやarray_key_existsを使いましょう。

Comments

[minorun]

ありがとうございます。

Answer 2

下の関数は未定義の場合でもif($v === '')まで行ってしまうのが原因だと思われますので
$isに代入せずにその時点でreturnすればいいように見えます。

今回のケースの場合は
Null合体演算子を使用してみてはどうでしょうか？

$offset   = $args['offset']??"";

で空の場合は空文字列が、0の場合は0が代入されると思います。

Comments

[minorun]

Null合体演算子、なるほどそれを知りたかったです。関数もありがとうございます。

Answer 3

これは「要件次第」「設計次第」になります。
falseの時にどうしたいのか、その時の情報をどう扱うか
設計、要件で決めることになります。
場合により三項演算子ではないことが良い時もあります。

PHP var7.3.1 使用ですが、アップデートで楽になるなら

今のところ特にはないと思います。

Comments

[minorun]

ありがとうございます。

Answer 4

個人開発でもフレームワークを使用するなら、フレームワークにバリデーションロジックが一般的には実装されているので、それを利用します。

フレームワークを使用しないなら、自身で好きなように実装しますが、可読性を高めるのであれば、
filter_var
を使用するとそこそこきれいに書けます。

使い慣れたフレームワークから、バリデーション用のクラスを移植するのも、後でメンテナンスしやすいです。

Comments

[minorun]

ありがとうございます。

Answer 5

PHP7.0で追加された??Null 合体演算子が求めてる機能だろう。
https://www.php.net/manual/ja/migration70.new-features.php

$name = $args['name'] ?? '';

強制したいならarrayではなく個別の引数にしたほうが良さそうだけど。

function hoges($name, $order_by = 'created_at', $offset = 1)

Comments

[minorun]

ありがとうございます。

Answer 6

直接的な回答に関しては既にあるもので行うことができるかと思うので、少し視点を変えて、、

一般的にはデータを挿入する際にバリデーションという形で必須項目やデータの整形は行います。その際に例えば必須でないものはそれに対してどの様に管理していくかを考える必要があります。NULLなのかデフォルトとして何か指定するのかなど。

そうすることにより展開する際の処理は削減することができます。

Comments

[minorun]

ありがとうございます。