drngnh708353 2018-02-03 15:13
浏览 28
已采纳

将功能与不同的呼叫方法结合起

I have 2 functions

This one generates numbers only

//Generate a string of only numbers.
function GenerateNUM($length)
{
    $alphabet = '1234567890';
    $tip = array();
    $alphaLength = strlen($alphabet) - 1;
    for ($i = 0; $i < $length; $i ++) {
        $n = rand(0, $alphaLength);
        $tip[] = $alphabet[$n];
    }
    return implode($tip);
}

This one generates numbers and letters

//Generate a string of numbers and letters.
function GenerateAll($length)
{
    $alphabet = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
    $pass = array();
    $alphaLength = strlen($alphabet) - 1;
    for ($i = 0; $i < $length; $i ++) {
        $n = rand(0, $alphaLength);
        $pass[] = $alphabet[$n];
    }
    return implode($pass);
}

and I call it like this

$ValidationCode = GenerateNUM(12);

or this

$ValidationCode = GenerateAll(12);

It works great but my question is do I really need 2 blocks of code? is there a way to create one function and be able to decide if numbers and letters are called or just numbers? or am I over thinking it?

  • 写回答

1条回答 默认 最新

  • dongzao9044 2018-02-03 15:22
    关注

    Example of a combined singular function based off your code:

    function GenerateCode($length,$type='ALPHA')
    {
        $alphabet = (($type == 'ALPHA')?'ABCDEFGHIJKLMNOPQRSTUVWXYZ':'') .'1234567890';
        $code = array();
        $alphaLength = strlen($alphabet) - 1;
        for ($i = 0; $i < $length; $i ++) {
            $n = rand(0, $alphaLength);
            $code[] = $alphabet[$n];
        }
        return implode($code);
    }
    
    $ValidationCode = GenerateCode(12,'NUM');// just makes a numbers only code
    $ValidationCode = GenerateCode(12,'ALPHA');// makes an alphanumeric code
    

    But there is nothing wrong with multiple small utility functions though. The naming keeps them clean and obvious their intent. However if you have a LOT of duplicated code between a bunch of utility functions, then you can combine them with a second parameter and adjusting small pieces inside based on that param (like above).


    A slightly better example of generating a random string like your function (includes upper and lower case lettering):

    function GenerateCode($length,$type='ALPHA')
    {
        $string = '';
        for ($n=1; $n <= $length; $n++) {
            if ($type == 'NUM') {
                $string .= mt_rand(0,9);
            } else {
                $randnum = mt_rand(0,61);
                $string .= ( ($randnum < 10) ? chr($randnum+48) :  // number chr 48 - 57
                            (($randnum < 36) ? chr($randnum+55) :  // upperletter chr 65 - 90
                                               chr($randnum+61) ));// lowerletter chr 97 - 122
            }
        }
        return $string;
    }
    
    // example GenerateCode(12,'ALPHA') output: XNu1n833b2ox
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 怎么制作一个人脸识别门禁系统
  • ¥20 大华dss监控平台网络关闭登不进去
  • ¥15 请使用蚁群算法解决下列问题,并给出我完整的代码
  • ¥20 关于php录入完成后,批量更新数据库
  • ¥15 请教往复密封润滑问题
  • ¥15 cocos creator发布ios包
  • ¥15 comsol压电材料数据
  • ¥35 用python实现除法算法中goldschmidt算法
  • ¥15 汇编代码转换成C代码
  • ¥15 除法算法中的归一化具体是怎么变的?