【问题标题】:php - call to a member function on null [duplicate]php - 在 null 上调用成员函数 [重复]
【发布时间】:2015-10-26 08:26:59
【问题描述】:

抱歉,如果这是 真正的 基本的,但是当 PHP 进入函数时,我无法理解。

我有一个论坛插件,它加载一个 flash cookie 作为检测重复帐户的方法。

脚本在大约 0.1% 的页面浏览量上失败,导致 WSOD。错误指向这一行:

return $this->registry->output->getTemplate( 'dml' )->duplicatesLoadMovie( $host, $path, $id, $md5 );

错误信息是:

致命错误:在第 75 行的 /var/www/.../web/forums/hooks/duplicatesLoadMovie.php 中的 null 上调用成员函数 duplicatesLoadMovie()

对我来说,这读作 $host$path$id$md5 之一正在返回 null,但因为这不是我的脚本(而且编码器当然没有响应),我想以最简单的方式解决这个问题(除了删除它,这是后备位置)。

我可以简单地做一些$id = 0 if id.null 的效果吗? (对不起Rubyspeak)每个$variables

文件的完整来源:

class duplicatesLoadMovie
{ 
/**
 * Registry object shortcuts
 *
 * @var     $registry
 * @var     $settings
 * @var     $member
 * @var     $memberData
 **/
public $registry;
public $settings;
public $member;
public $memberData;

/**
 * Main function executed automatically by the controller
 *
 * @access  public
 * @return  @e void
 **/
public function __construct()
{
    $this->registry   = ipsRegistry::instance();
    $this->settings   =& $this->registry->fetchSettings();
    $this->member     = $this->registry->member();
    $this->memberData =& $this->registry->member()->fetchMemberData();
}

/**
 * Get the output
 *
 * @access  public
 * @return  @e string
 **/
public function getOutput()
{
    /* Grab host URL and installation path of the community */
    $host = $this->settings['board_url'];

    $parsed = parse_url( $host );
    $path = trim( $parsed['path'], '/' );
    if( $path == '' )
    {
        $path = '/';
    }

    $host = 'host=' . $host;
    $path = 'path=' . $path;

    /* Determine the appropriate identification method (member_id for members and session_id for guests) */
    if( $this->memberData['member_id'] )
    {
        $id = 'mid=' . $this->memberData['member_id'];
    }
    else
    {
        $id = 'sid=' . $this->member->session_id;
    }

    /* Grab the secure key for AJAX */
    $md5 = 'md5=' . $this->member->form_hash;

    /* Finally, call the template bit */
    return $this->registry->output->getTemplate( 'dml' )->duplicatesLoadMovie( $host, $path, $id, $md5 );
}

【问题讨论】:

    标签: php


    【解决方案1】:

    我怀疑这个getTemplate( 'dml' ) 没有返回值。如果你做一个var_dump(getTemplate( 'dml' )),它会显示什么吗?

    您可以在执行该行代码之前检查“null”,并在“else”语句中输出错误消息或其他适合该错误的操作。

    【讨论】:

    • 对不起,我不关注。这是通过网页调用的。你希望我从命令行或 php 运行它吗?
    • 嗯?我知道。出于测试目的,就在您的“返回”行之前,输入: var_dump(getTemplate( 'dml' ));出口();这会将它的值输出到您的浏览器窗口,以便您可以查看它是否实际上返回了一个值。我怀疑它不是,所以因为它没有返回一个值(即:null),所以你不能在它上面调用 duplicatesLoadMovie 方法。
    • 看,我告诉过你 PHP 不是我的菜 :) 我会试一试的。
    • 是的,这把事情搞砸了。所有页面都变成 WSOD 并出现错误:调用未定义的函数 getTemplate()
    • /* 最后,调用模板位 */ // var_dump(getTemplate( 'dml' )); // 出口(); return $this->registry->output->getTemplate('dml')->duplicatesLoadMovie($host, $path, $id, $md5);
    【解决方案2】:

    您的问题很久以前就解决了,但我想很多寻找解决方案的人都在为这个错误而苦苦挣扎。 PHP 8.0 引入了新的运算符nullsafe operator (?->)。这不是为了抑制错误(!),而是对于允许变量为null 的情况,然后我们不需要仔细检查它的值。

    所以在 OP 的问题中,最后一行看起来像:

    return $this->registry->output->getTemplate( 'dml' )?->duplicatesLoadMovie( $host, $path, $id, $md5 );
    

    应该非常小心地使用它,因为它不是每次出现这种错误的解决方案!

    【讨论】:

      猜你喜欢
      • 2016-05-22
      • 2017-02-23
      • 1970-01-01
      • 2016-06-27
      • 1970-01-01
      • 2017-12-06
      • 1970-01-01
      • 1970-01-01
      • 2022-01-08
      相关资源
      最近更新 更多