美文网首页
PGC视频爬坑史

PGC视频爬坑史

作者: 探路者杨腾飞 | 来源:发表于2017-01-16 15:20 被阅读0次

    由来

    接手PGC视频,类似于接手一个新项目,对很多东西不了解,这时候开始追忆,导师让我看之前视频的东西时,没有静下心来好好看,说多了都是一把辛酸泪。接手一个项目前,学会了解这个项目,具体方法请百度

    坑坑坑,爬爬爬

    后来就是各种入坑,各种爬坑。最后发现我们的QA说到的坑,几乎都被我撞上了,默默地膜拜QA的美女帅哥们,崇拜的眼神~~。
    列举一下入的坑:

    编码

    编码这个坑以前没遇到过啊,当时QA跟我说过这个点,就想糊弄过去也没放在心上,结果。。。出来混总归是要还的(捂脸哭),应该听QAmm的话。就不描述我的惨相了,说说坑和爬坑。

    • no1: mysql数据库中对应的那个表,用的是utf8编码,而不是utf-8编码,传进去的汉字应该是utf8编码的
    • no2:video模块中的编码之前貌似是gbk,需要在service/Video中对新加的方法设置一下编码格式。有人提醒我这么做了,我直接进行设置,并没有什么效果,反而删掉后编码格式对了。具体为什么会酱紫,需要后续研究,看来video编码这块有个大坑

    封面id

    不多说了,直接上代码:

           if($arrInput['channel_id'] > 0){
               $arrVideoInfo['ext_attr']['channel_id'] = $arrInput['channel_id'];
               $arrVideoInfo['ext_attr']['video_format'] = $arrInput['video_format'];
               $arrVideoInfo['ext_attr']['thumbnail_picid'] = $arrInput['thumbnail_picid'];
           }
    

    从代码可以看出,thumbnail_picid/video_format两个字段与channel_id耦合在一起了,写代码时没多想,顺手就把他们放一起了,自己给自己挖坑。

    返回值

    照例,来一盘香喷喷的代码片段:

     $arrOut = Tieba_Service::call('video', 'savePGCData', $input, null, null, 'post', 'php', 'gbk');
     if(null == $arrOut || !isset($arrOut['errno']) || Tieba_Errcode::ERR_SUCCESS != $arrOut['errno']){
         Bingo_Log::fatal('mis/service/xiaoying call video savePGCData fail. input:['.serialize($input).'] out:['.serialize($arrOut).']');
     }
    

    调用的方法里有一段是酱紫的:

     $arrOut = self::addPgcDelivery($input);
     if(null == $arrOut || !isset($arrOut['errno']) || Tieba_Errcode::ERR_SUCCESS != $arrOut['errno']){
         Bingo_Log::warning('video call video addPgcDelivery fail. input:['.serialize($input).'] out:['.serialize($arrOut).']');
         return self::_errRet(isset($arrOut['errno'])?$arrOut['errno']:Tieba_Errcode::ERR_CALL_SERVICE_FAIL);
     }
    

    从上面的代码可以看出,当$arrOut有问题时,才会有返回值,没有问题时不进行处理,这怎么行,这逻辑有漏洞啊,然而我就是这么善于自己给自己挖坑。于是改成:

    $arrOut = self::addPgcDelivery($input);
    if(null == $arrOut || !isset($arrOut['errno']) || Tieba_Errcode::ERR_SUCCESS != $arrOut['errno']){
        Bingo_Log::warning('video call video addPgcDelivery fail. input:['.serialize($input).'] out:['.serialize($arrOut).']');
        return self::_errRet(isset($arrOut['errno'])?$arrOut['errno']:Tieba_Errcode::ERR_CALL_SERVICE_FAIL);
    }
    return self::_errRet($arrOut['errno']);
    

    填坑完毕

    其他

    不好意思,没想起来那么多,想起来再补上:)

    反思

    导致上述状况的原因:
    不得不说之前一直精神懈怠,这次的项目很认真地去做了,然而还是不如人意。一方面是项目坑多,不好测;另一方面也是自己能力的问题。针对这两个没说的:项目坑多、不好测,就积累这方面的经验:在接手一个项目时,学会如何去了解这个项目,并一步步落实,在测的时候不要乱用数据,胡乱测一通,应该有方法地去选择数据,比如这次有很多权限和粒度控制的问题。下次接手一个项目时,先百度一下,如何接手一个未知的项目,然后再去做。

    相关文章

      网友评论

          本文标题:PGC视频爬坑史

          本文链接:https://www.haomeiwen.com/subject/agzubttx.html