PerfectWorks［001号］天降卡片1.4FORD41

说下135行错误是为什么.?

怎么添加后还是说没有添加任何卡片的?

我和101的问题一样

把card.htm放到default目录里
奇怪,我的能建立两个一样的default目录
其中一个default目录里只有一个card.htm文件
把他转移到另一个就好了
135的错误

压缩包里怎么多了一份?

............

哪位高人提供下..卡片名称?描述哟....

谢了..

初始安装完成,测试发现楼主发的能掉落.其他的回复都不掉落卡片`

添加卡片后什么都没有~~~

第1次看文文下的东西.提点建议.
1.

1.                 if($posts<$cardsittings['cardpostnum']){
   
2.                         showmessage('你没有制作卡片的资格请参看使用说明','plugin.php?identifier=card&module=card');
   
3.                 }
   
4. 
   
5.                 if($posts<$cardsittings['cardpostnum'] || $credits<$$cardsittings['cardcredit']){
   
6.                         showmessage('你没有制作卡片的资格请参看使用说明','plugin.php?identifier=card&module=card');
   
7.                 }

复制代码

没眼花应该是重复了吧.?

1. if ($cardsittings['open']==0 && !$ismaster)showmessage("$cardsittings[reason]");

复制代码

这样写可读性差了.建议

1. if($cardsittings['open']==0 && !$ismaster) showmessage("$cardsittings[reason]");

复制代码

如果求效率

1. $cardsittings['open']==0 && !$ismaster && showmessage("$cardsittings[reason]");

复制代码

1. $cardsittings['open']==0

复制代码

建议

1. !$cardsittings['open']

复制代码

1. if (!isset($action) or $action=='')$action="see";

复制代码

建议

1. $action = isset($action) ? $action : 'see';

复制代码

1. $action=="add"

复制代码

等建议改为

1. $action == 'add'

复制代码

1.         if (!$ismaster){
   
2. 
   
3.                 if($posts<$cardsittings['cardpostnum']){
   
4.                         showmessage('你没有制作卡片的资格请参看使用说明','plugin.php?identifier=card&module=card');
   
5.                 }
   
6. 
   
7.                 if($posts<$cardsittings['cardpostnum'] || $credits<$$cardsittings['cardcredit']){
   
8.                         showmessage('你没有制作卡片的资格请参看使用说明','plugin.php?identifier=card&module=card');
   
9.                 }
   
10.         }

复制代码

第1个判断和第2个矛盾了.如果$posts<$cardsittings['cardpostnum']不成立但$credits<$$cardsittings['cardcredit']条件成立时无法执行.if建议用&&

'$cname','$ceffect','$cint','$cardico'是否应该过滤下.?

1.         $query = $db->query("SELECT * FROM {$tablepre}card ORDER BY id DESC LIMIT $start_limit, $num");
   
2.         $break = 1;
   
3.         while($carddata = $db->fetch_array($query)){
   
4.                          $carddata[tr] = $break;
   
5.                          $break++;
   
6.                          $cardlist[] = $carddata;
   
7.         }

复制代码

缩进似乎缩错了.

1.         if(!$ismaster == 1) {
   
2.                 showmessage('非管理人员不能使用，请返回。');
   
3.         }

复制代码

这个写法有误.要不是$ismaster == '1'要不是$ismaster != '1'

1. @include DISCUZ_ROOT.'./include/cache.func.php';

复制代码

建议在需要更新缓存处才使用.节省效率.

Step 1.1到1.4不建议直接这样写进cache文件.这样写升级很不方便.而且会导致文件乱.
建议: https://discuz.dismall.com/thread-274657-1-1.html 这个插件的缓存写法很不错.建议参考.

html部分建议全部小写.比如<td><div>等.规范问题.