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

光吃天下 · 发表于 2019-3-7 16:45:25

[应用审核] “[君华]会员中心 1.0” 审核失败，现已被撤销看你写的插件非常费神，明明很多函数DX自带非要自作聪明去重写一次；不想看下去了驳回反省

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

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

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

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

@各审核版主们，这是我一个普通插件开发者给的建议。希望民审们能看到!

民审K星客 · 发表于 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原生函数也快点我记得以前是有的。

ACGI · 发表于 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、不知道这算不算多此一举；

if(!function_exists('junhua_utf82gbk')){
function junhua_utf82gbk($pram){
return diconv($pram, 'utf-8');
}
}

复制代码

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

ゞ╃緈褔擵忝輪 · 发表于 2019-3-7 17:05:50

ACGI 发表于 2019-3-7 17:02
我群里都发了N遍了

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

mlsjla · 发表于 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
你好！首先让你感觉审核服务不太友好非常抱歉，但是兄弟请你换位思考，你才几个插件审核每天要审核多少！
...

首先感谢您的提出
以下是我插件中的所有函数

<?php
if (!defined('IN_DISCUZ')) {
exit('Access Denied');
}
if(!function_exists('junhua_url')){
function junhua_url($url, $param = '', $isreturn = false){
$ary = explode('/', $url);
$url = 'plugin.php?id='.CURMODULE.':'.$ary[0].'_'.$ary[1].'&a='.$ary[2].($param ? '&'.$param : '');
if($isreturn){
return $url;
}else{
echo $url;
}
}
}
if(!function_exists('junhua_redirect')){
function junhua_redirect($url, $time=0, $msg='') {
//多行URL地址支持
$url = str_replace(array("\n", "\r"), '', $url);
if (empty($msg))
$msg = "系统将在{$time}秒之后自动跳转到{$url}！";
if (!headers_sent()) {
// redirect
if (0 === $time) {
header('Location: ' . $url);
} else {
header("refresh:{$time};url={$url}");
echo($msg);
}
exit();
} else {
$str = "<meta http-equiv='Refresh' content='{$time};URL={$url}'>";
if ($time != 0)
$str .= $msg;
exit($str);
}
}
}
if(!function_exists('junhua_ajaxReturn')){
function junhua_ajaxReturn($data,$type='',$json_option=0) {
header('Content-Type:application/json; charset=utf-8');
exit(junhua_json_encode($data,$json_option));
}
}
if(!function_exists('junhua_json_encode')){
function junhua_json_encode($data, $json_option=0){
$ret = array();
if (strtolower(CHARSET) == 'gbk') {
$ret = junhua_array_map('junhua_gbkutf8', $data);
}else{
$ret = $data;
}
return json_encode($ret, $json_option);
}
}
if(!function_exists('junhua_gbkutf8')){
function junhua_gbkutf8($pram){
if (strtolower(CHARSET) == 'gbk') {
return diconv($pram, 'gbk', 'utf-8');
}else{
return $pram;
}
}
}
if(!function_exists('junhua_utf82gbk')){
function junhua_utf82gbk($pram){
return diconv($pram, 'utf-8');
}
}
if(!function_exists('junhua_I')){
function junhua_I($name){
if (strtolower(CHARSET) == 'gbk') {
$_GET[$name] = is_array($_GET[$name]) ? junhua_array_map('junhua_utf82gbk', $_GET[$name]) : junhua_utf82gbk($_GET[$name]);
return $_GET[$name];
}else{
return $_GET[$name];
}
}
}
if(!function_exists('junhua_array_map')){
function junhua_array_map($filter, $data) {
$result = array();
foreach ($data as $key => $val) {
$result[$key] = is_array($val) ? junhua_array_map($filter, $val) : call_user_func($filter, $val);
}
return $result;
}
}
if (!function_exists('junhua_getorderno')) {
function junhua_getorderno($prefix = ''){
return $prefix.date('ymd') . substr(time() , -5) . substr(microtime() , 2, 5);
}
}
if (!function_exists('junhua_check_agent')) {
function junhua_check_agent(){
$browser = array('1'=>'MicroMessenger', '2' => 'iphone', '3' => 'android');
$browser_id = 0;
$agent = $_SERVER['HTTP_USER_AGENT'];
if(!empty($browser)){
foreach($browser as $key => $rs){
if(strpos(strtolower($agent),strtolower($rs)) != false){
$browser_id = $key;
break;
}
}
}
return $browser_id;
}
}

复制代码

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

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

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

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

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

		自动登录	找回密码
密码			立即注册