CodeIgniter：防止 row_array() 与 row() 拼写错误？

发布于 2024-12-07 02:36:05 字数 487 浏览 3 评论 0原文

现在，在获取单行结果时，我两次错误地使用 row() 而不是 row_array() 。通常，它不会受到检查，也不会发出任何警告消息。 15-30 分钟后我终于注意到了这个问题；呵呵呵！

有谁对防止这种错误的方法有什么好的建议吗？对象格式不会引发 PHP 警告，但是，当然，当您希望从模型方法获取数组时，代码将不起作用。

我不小心使用了：

if ($query->num_rows() > 0) {
    return $query->row();
}

应该是：

if ($query->num_rows() > 0) {
    return $query->row_array();
}

我考虑重写 row() 数据库方法并添加 log_message() 语句，这至少会让我知道（在调试级别）我正在使用对象格式，而不是更典型的（对于我的代码）格式。

原文

Twice now I've mistakenly used row() instead of row_array() when fetching a single row result. Usually, it goes unchecked without any warning messages. 15-30 minutes later I finally notice the issue; doh doh!

Does anyone have any good suggestions as to ways to prevent this mistake. The object format doesn't throw a PHP warning, but - of course - the code doesn't work when you expect an array from a model method.

I accidentally used:

if ($query->num_rows() > 0) {
    return $query->row();
}

Which should instead be:

if ($query->num_rows() > 0) {
    return $query->row_array();
}

I thought about overriding the row() database method and adding in a log_message() statement that would at least let me know (on debug level) that I'm using the object format, instead of the much more typical (for my code) format.

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

囚我心虐我身 2024-12-14 02:36:05

如何避免在 CodeIgniter 模型方法中出现数据类型错误？很简单，声明方法的返回数据类型。

// returns null or an object
public function getFlatObject(int $id): ?object
{
    return $this->db->get_where("your_table", ['id' => $id])->row();
}

// unconditionally returns an array
public function getFlatArray(int $id): array
{
    return $this->db->get_where("your_table", ['id' => $id])->row_array();
}

如果您的返回数据与声明的类型（: 之后）不匹配，那么 php 将为您生成错误。

让我们借此机会消除任何混乱...

如果您调用 row() 或 row_array() 并且您的 sql 逻辑提供了超过一行数据——那么您只会获得结果集的第一行。
如果调用 row() 但结果集中没有行，则返回值为 null。
如果调用 row_array() 但结果集中没有行，您将收到一个空数组作为返回值。
因此，row() 可能返回可迭代或不可迭代的数据类型；而 row_array() 始终可以安全地直接传递到迭代器（如 foreach() 或 array_map()） - 它可能不会有任何东西要迭代。

How do you avoid making data-type errors in your CodeIgniter model methods? Simple, declare your methods' return data type.

// returns null or an object
public function getFlatObject(int $id): ?object
{
    return $this->db->get_where("your_table", ['id' => $id])->row();
}

// unconditionally returns an array
public function getFlatArray(int $id): array
{
    return $this->db->get_where("your_table", ['id' => $id])->row_array();
}

If your return data does not match the declared type(s) (after :), then php is going to generate errors for you.

Let's take this opportunity to clear up any confusion...

If you call either row() or row_array() and your sql logic affords the possibility of more than one row of data -- then you are only going to get the first row of the result set.
If you call row() but there are no rows in the result set, you will have a return value of null.
If you call row_array() but there are no rows in the result set, you will receive an empty array as the return value.
Hence, row() may return an iterable or non-iterable data type; whereas row_array() can ALWAYS be safely passed directly into an iterator (like foreach() or array_map()) -- it just might not having anything to iterate.