来个懂C语法的

lovehuai

下面代码可运行无错误但是 7 Warnings.
另外就是cmd[32],
if (strcmp(cmd, "2")==0)
囧Rz 不太懂C语法类型都不识几个 能想到的只有这个了
来个懂的帮修改优化下

1. #include <amxmodx>
   
2. #include <amxmisc>
   
3. #include <cstrike>
   
4. #include <string>
   
5. 
   
6. public plugin_init()
   
7. {
   
8.         register_plugin("Set_Team", "0.1", "LoveHuai");
   
9.   register_clcmd("set_team_menu", "Set_Team_Menu")
   
10. }
    
11. 
    
12. public Set_Team_Menu(id, level, cid)
    
13. {
    
14.         new menu = menu_create("队伍转移:", "Test_Menu1_Handler")
    
15.         menu_additem(menu, "转移到反恐精英", "1", ADMIN_RESERVATION)
    
16.         menu_additem(menu, "转移到恐怖分子", "2", ADMIN_RESERVATION)
    
17.         menu_setprop(menu, MPROP_EXIT, MEXIT_NEVER)
    
18.         menu_display(id, menu, 0)
    
19.         return PLUGIN_HANDLED
    
20. }
    
21. 
    
22. 
    
23. 
    
24. 
    
25. public Test_Menu1_Handler(id, menu, item)
    
26. {
    
27.         if (item == MENU_EXIT)
    
28.         {
    
29.                 menu_destroy(menu)
    
30.                 return PLUGIN_HANDLED
    
31.         }
    
32.         new cmd[32], name[32], access
    
33.         menu_item_getinfo(menu, item, access, cmd, 31, name, 31, access)
    
34.                 if (strcmp(cmd, "1")==0)
    
35.                 {
    
36.                                 if (cs_get_user_team(id) == CS_TEAM_T){
    
37.                                                                 if (is_user_alive(id)) {
    
38.                                 user_kill(id,1);
    
39.                         }
    
40.                                         cs_set_user_team(id,CS_TEAM_CT,CS_CT_URBAN);
    
41.                                 }
    
42.                                 menu_destroy(menu)
    
43.                return PLUGIN_HANDLED       
    
44.                 }
    
45.                 if (strcmp(cmd, "2")==0)
    
46.                 {
    
47.                                 if (cs_get_user_team(id) == CS_TEAM_CT){
    
48.                                                                 if (is_user_alive(id)) {
    
49.                                 user_kill(id,1);
    
50.                         }
    
51.                                         cs_set_user_team(id,CS_TEAM_T,CS_T_TERROR);
    
52.                                 }
    
53.                                 menu_destroy(menu)
    
54.                return PLUGIN_HANDLED       
    
55.                 }
    
56.         menu_destroy(menu)
    
57.         return PLUGIN_HANDLED
    
58. }

复制代码

Pepent

哎呀。好深奥。。  这回真的是只可远观不可亵玩了。

103125369

这个是干什么的？

Edoo.Left

说实话。
我也就看不懂而已

MaMaMiYa

Warnning 是在什么位置呢？LZ能否提示一下？

fantasist

waring都是loose indentation，意思是排版不好
每个语句前最好用tab进行缩进，不要用空格

MaMaMiYa

简单的一个小优化就是在if 后面跟上else，避免形成无谓的判断。当然更推荐使用switch语句。
还有一点是为什么不把：return PLUGIN_HANDLED 写在函数最后？写一个就可以了，这样函数不会很乱。此外请LZ自己添加注释