Discuz! 官方站

 找回密码
 立即注册

QQ登录

只需一步,快速开始

搜索
查看: 1158|回复: 15

现在的民审都是这么不耐烦的吗?

[复制链接]
发表于 2019-3-7 16:45:25 | 显示全部楼层 |阅读模式
[应用审核] “[君华]会员中心 1.0” 审核失败,现已被撤销 看你写的插件非常费神,明明很多函数DX自带 非要自作聪明去重写一次;不想看下去了 驳回反省


您如果看不懂或者觉得有些函数没意义,您可以直接告诉我哪些函数没用。
我写的函数,我自然是认为有用途的,如果没作用,我肯定不可能去浪费时间去再写一遍。

另外我希望民审能耐心点,我知道您很忙,但很忙不代表您可以随着您自己的心情随意践踏开发者辛苦码出来的代码。

18年我也审核了很多插件,也用到了现在自己封装的框架,那时候的民审一直都是耐心的审核完,每个函数也都能理解,并且也上架了,为什么19年的民审突然就耐不下心情了?

dz的应用市场是 开发者  民审  用户  三者共同建立的。不是单独一个就可以建立现在这么繁荣的应用市场。

@各审核版主们,这是我一个普通插件开发者给的建议。希望民审们能看到!
发表于 2019-3-7 16:51:54 | 显示全部楼层
民审还是那些民审,从哪里看出来践踏你的代码了?

你所谓的很多插件是几个?以前审核相对比现在宽松,不代表不可以变严谨,提高自己的代码质量才是根本
回复

使用道具 举报

发表于 2019-3-7 16:52:18 | 显示全部楼层
审核本来就不包括反馈问题,应用中心这么多开发者,又不是只审核你一个应用,审核工作量有多大可想而知!我们谁不遇到过打回,按开发文档标准好好做好应用就是咯
回复

使用道具 举报

发表于 2019-3-7 16:59:41 | 显示全部楼层
民审K星客 发表于 2019-3-7 16:51
民审还是那些民审,从哪里看出来践踏你的代码了?

你所谓的很多插件是几个?以前审核相对比现在宽松,不 ...

民审,有没有dz函数库的文档?如果有的话 这样我们开发照着文档 用dz原生函数也快点 我记得以前是有的。
回复

使用道具 举报

发表于 2019-3-7 17:02:39 | 显示全部楼层
ゞ╃緈褔擵忝輪 发表于 2019-3-7 16:59
民审,有没有dz函数库的文档?如果有的话 这样我们开发照着文档 用dz原生函数也快点 我记得以前是有的。

我群里都发了N遍了
回复

使用道具 举报

发表于 2019-3-7 17:02:53 | 显示全部楼层
你好!首先让你感觉审核服务不太友好非常抱歉,但是兄弟请你换位思考,你才几个插件审核每天要审核多少!

至于你反馈的问题我现在正是来回答你下,以上是从你代码中抄录:
1、格式化时间的方法DX自古就有,比如:dgmdate等等一些不一一列举;
2、strtolower(CHARSET) == 'gbk' 一个编码格式判断,DZ本身就把小写转换为大写,你非要写一个函数来转换下;
3、不知道这算不算多此一举;
  1. if(!function_exists('junhua_utf82gbk')){
  2.     function junhua_utf82gbk($pram){
  3.         return diconv($pram, 'utf-8');
  4.     }
  5. }
复制代码


综上所述,要想别人对你好 还需加强自身的内功修炼,每天面对成千上万的代码且免费帮审核,不是为了为难你而是为了让你养成一个DX开发者的习惯,俗话说无规矩不成方圆,现在审核力度越来越大不要全部寄希望于审核一一给你指名错误!!
回复

使用道具 举报

发表于 2019-3-7 17:05:50 | 显示全部楼层
ACGI 发表于 2019-3-7 17:02
我群里都发了N遍了

我找找 没想到ACGI会上传干货
回复

使用道具 举报

发表于 2019-3-7 17:11:39 | 显示全部楼层
可想而知你代码积累了民审多大的怨念
回复

使用道具 举报

 楼主| 发表于 2019-3-7 19:29:47 | 显示全部楼层
民审K星客 发表于 2019-3-7 16:51
民审还是那些民审,从哪里看出来践踏你的代码了?

你所谓的很多插件是几个?以前审核相对比现在宽松,不 ...

我代码写的有问题 您可以直接说 “有些函数代码没必要再去封装”,如果更好点的话,您可以说“ xxx些函数是有累赘的 没必要这么写”
为什么非要说我写的代码是 “自作聪明 还不想看下去了 ”,您一直说的换位思考,如果您站在一个开发者的角度,别人说你的代码是自作聪明 您又做何感想。

忙不是借口,哪个人不忙,既然选择了这个工作,就请端正态度。
当然我们开发者也不能随便乱写代码,也尽最大程度的提高代码质量。尽力减轻审核员的工作量


回复

使用道具 举报

 楼主| 发表于 2019-3-7 19:50:29 | 显示全部楼层
民审员乙 发表于 2019-3-7 17:02
你好!首先让你感觉审核服务不太友好非常抱歉,但是兄弟请你换位思考,你才几个插件审核每天要审核多少!
...

首先 感谢您的提出
以下是我插件中的所有函数
  1. <?php
  2. if (!defined('IN_DISCUZ')) {
  3.     exit('Access Denied');
  4. }

  5. if(!function_exists('junhua_url')){
  6.     function junhua_url($url, $param = '', $isreturn = false){
  7.         $ary = explode('/', $url);
  8.         $url = 'plugin.php?id='.CURMODULE.':'.$ary[0].'_'.$ary[1].'&a='.$ary[2].($param ? '&'.$param : '');
  9.         if($isreturn){
  10.             return $url;
  11.         }else{
  12.             echo $url;
  13.         }
  14.     }
  15. }

  16. if(!function_exists('junhua_redirect')){
  17.     function junhua_redirect($url, $time=0, $msg='') {
  18.         //多行URL地址支持
  19.         $url        = str_replace(array("\n", "\r"), '', $url);
  20.         if (empty($msg))
  21.             $msg    = "系统将在{$time}秒之后自动跳转到{$url}!";
  22.         if (!headers_sent()) {
  23.             // redirect
  24.             if (0 === $time) {
  25.                 header('Location: ' . $url);
  26.             } else {
  27.                 header("refresh:{$time};url={$url}");
  28.                 echo($msg);
  29.             }
  30.             exit();
  31.         } else {
  32.             $str    = "<meta http-equiv='Refresh' content='{$time};URL={$url}'>";
  33.             if ($time != 0)
  34.                 $str .= $msg;
  35.             exit($str);
  36.         }
  37.     }
  38. }

  39. if(!function_exists('junhua_ajaxReturn')){
  40.     function junhua_ajaxReturn($data,$type='',$json_option=0) {
  41.         header('Content-Type:application/json; charset=utf-8');
  42.         exit(junhua_json_encode($data,$json_option));
  43.     }
  44. }


  45. if(!function_exists('junhua_json_encode')){
  46.     function junhua_json_encode($data, $json_option=0){
  47.         $ret = array();
  48.         if (strtolower(CHARSET) == 'gbk') {
  49.             $ret = junhua_array_map('junhua_gbkutf8', $data);
  50.         }else{
  51.             $ret = $data;
  52.         }
  53.         return json_encode($ret, $json_option);
  54.     }
  55. }

  56. if(!function_exists('junhua_gbkutf8')){
  57.     function junhua_gbkutf8($pram){
  58.         if (strtolower(CHARSET) == 'gbk') {
  59.             return diconv($pram, 'gbk', 'utf-8');
  60.         }else{
  61.             return $pram;
  62.         }
  63.     }
  64. }


  65. if(!function_exists('junhua_utf82gbk')){
  66.     function junhua_utf82gbk($pram){
  67.         return diconv($pram, 'utf-8');
  68.     }
  69. }


  70. if(!function_exists('junhua_I')){
  71.     function junhua_I($name){
  72.         if (strtolower(CHARSET) == 'gbk') {
  73.             $_GET[$name] = is_array($_GET[$name]) ? junhua_array_map('junhua_utf82gbk', $_GET[$name]) : junhua_utf82gbk($_GET[$name]);
  74.             return $_GET[$name];
  75.         }else{
  76.             return $_GET[$name];
  77.         }
  78.     }
  79. }

  80. if(!function_exists('junhua_array_map')){
  81.     function junhua_array_map($filter, $data) {
  82.         $result = array();
  83.         foreach ($data as $key => $val) {
  84.             $result[$key] = is_array($val) ? junhua_array_map($filter, $val) : call_user_func($filter, $val);
  85.         }
  86.         return $result;
  87.     }
  88. }

  89. if (!function_exists('junhua_getorderno')) {
  90.     function junhua_getorderno($prefix = ''){
  91.         return $prefix.date('ymd') . substr(time() , -5) . substr(microtime() , 2, 5);
  92.     }
  93. }

  94. if (!function_exists('junhua_check_agent')) {
  95.     function junhua_check_agent(){
  96.         $browser = array('1'=>'MicroMessenger', '2' => 'iphone', '3' => 'android');
  97.         $browser_id = 0;
  98.         $agent = $_SERVER['HTTP_USER_AGENT'];
  99.         if(!empty($browser)){
  100.             foreach($browser as $key => $rs){
  101.                 if(strpos(strtolower($agent),strtolower($rs)) != false){
  102.                     $browser_id = $key;
  103.                     break;
  104.                 }
  105.             }
  106.         }
  107.         return $browser_id;
  108.     }
  109. }
复制代码

strtolower(CHARSET) == 'gbk' 一个编码格式判断,DZ本身就把小写转换为大写,你非要写一个函数来转换下;

此处的操作是为了让判断万无一失,而不是为了多此一举。

当然我很赞同您的说法,民审确实是工作量大,但我们开发者提交的时候,肯定也是觉得没问题才回去提交,不可能随便去写几条代码,就丢上去让民审去测试,如果真有这样的开发者,那就是我们开发者的悲哀了。

我的投诉只是想说明,请民审也尊重开发者的成果,毕竟这个生态圈是 开发者 民审  用户 一起组成的。
不要动不动就居高临下的感觉,觉得我是民审我最大。这个平台离了三者中的哪一个 都会变的失败。

再次感谢您的提出,我会继续排查我的代码
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

腾讯云市场 一站式企业服务 爆款建站、小程序等1元起

QQ|小黑屋|Discuz! 官方站 ( 粤B2-20090059-165 )star

GMT+8, 2019-12-6 19:01

Powered by Discuz! X3.3

Copyright © 2001-2019 Tencent Cloud.

快速回复 返回顶部 返回列表