PGC视频爬坑史

由来

接手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']);

填坑完毕

其他

不好意思，没想起来那么多，想起来再补上：）

反思

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