从 PHP5.2 升级到 PHP7.1 遇到的函数参数问题

最近在做一个系统的ID管理系统php升级,从php5.2升级到php7.1,经常会遇到一些问题。今天遇到一个小问题,拿出来分享一下。废话少说,上示例代码。

SC.php

<?php

class SC {
    static function out($str) {
        echo 'input str:' . $str;
    }
}

// 有参数调用
SC::out('one param');
// 无参数调用
SC::out();
  • php5.2中的实行结果:

         input str:one param
         Warning: Missing argument 1 for SC::out(), called in C:\Users\colbe\Documents\SC.php on line 11 and defined in   C:\Users\colbe\Documents\SC.php on line 4
         input str:
  • php7.1中的实行结果:

    input str:one param
    Fatal error: Uncaught ArgumentCountError: Too few arguments to function SC::out(), 0 passed in C:\Users\colbe\Documents\SC.php on line 11 and exactly 1 expected in C:\Users\colbe\Documents\SC.php on line 4
    
    ArgumentCountError: Too few arguments to function SC::out(), 0 passed in C:\Users\colbe\Documents\SC.php on line 11 and exactly 1 expected in C:\Users\colbe\Documents\SC.php on line 4
    
    Call Stack:
    0.0069     348536   1. {main}() C:\Users\colbe\Documents\SC.php:0
    0.0088     348568   2. SC::out() C:\Users\colbe\Documents\SC.php:11

Warning变Fatal,直接就崩了。修改倒是很简单,给函数的参数加个默认值就可以了。修改一下方法的签名部分为[static function out($str = null) {]就可以了。再次运行,结果如下:

    input str:one paraminput str:

我擦,才发现木加换行!就这样吧,太晚了,睡觉去了。

PS:来自TimJuly的重要提示:

TimJuly  1  ⋅ 
#1 ⋅ 2小时前
说白了就是代码不规范。Warning 是错误的一种类型,并不是说你这个没有错,可以这样用。就算是 Notice 的错误也应该处理完再上线。
本作品采用《CC 协议》,转载必须注明作者和本文链接
《L02 从零构建论坛系统》
以构建论坛项目 LaraBBS 为线索,展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
讨论数量: 2
LearnKuJ

说白了就是代码不规范。Warning 是错误的一种类型,并不是说你这个没有错,可以这样用。就算是 Notice 的错误也应该处理完再上线。

6年前 评论

@TimJuly
是的,您说的很对。就算是Notice也不应该容忍的。如果是规范的代码,这种问题根本就不会发生。
近来,由于php7在性能方面的显著提升,公司接到了许多php版本升级的项目,为了节约成本,尽量减少代码的修改,只要系统不倒,warn也忍了。。。
在这里贴出来,就是记录一下,大家权当反面教材看吧。
以上

6年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!