func isInstalled()での判定

Question

function isInstalled() {
		global $dbconfig;
		if (empty($dbconfig) || empty($dbconfig['db_name']) || $dbconfig['db_name'] == '_DBC_TYPE_') {
			return false;
		}
		return true;
	}

上記、人の書いたコードなのですが、if (empty($dbconfig) || empty($dbconfig['db_name']) || $dbconfig['db_name'] == 'DBC_TYPE')の部分がなんか気持ち悪いです。こういう場合、$dbconfigの一個一個のパラメータをチェックすべきだと思うのですが、書き方は好き好きというやつでしょうか。

Comments

[s8_chu]

「$dbconfigの一個一個のパラメータをチェックする」というのはどのような書き方のことをあらわしているのでしょうか？

Answer 1

そのように書くのには理由があるので、理由も分からないで「気持ち悪い」と言っていないで書かれた方に聞くのが一番かと思います。