美女们，帮我看一下下面的代码怎么优化啊。

现在有青岛和济南两个分公司，但是角色是一样的，我要通过当前登录用户的部门里的字，进行相应角色选择，现在的代码只能出一个，假如角色是“产品经理”，济南有4个，青岛有两个。我要在流程中将所有是济南的产品经理全部找出来做为流程的审批人，帮我优化一下下面的代码吧，不会写了。

1. function GetUserByJiaoSe(JiaoSeName)
   
2. {
   
3.      //println("======================"+JiaoSeName);
   
4.         //获取用户对象
   
5.         var userVO = getUserById(getWebUser().getId());
   
6.         //获取用户所属部门,这里返回的是用户所属的所有部门
   
7.         var WebUserdept = userVO.getDepartments();
   
8.         //println("名称:--------------"+ WebUserdept);
   
9.         var WebUserDeptname="";
   
10.         var suoshu="";        //所属
    
11.         for(var iter = WebUserdept.iterator();iter.hasNext();)
    
12.         {
    
13.                 var value1 = iter.next();
    
14.                 WebUserDeptname +=value1.getName()+";"          
    
15.                 //println("部门名称:--------------"+WebUserDeptname);       
    
16.                 //println("截取前两个字:"+WebUserDeptname.substring(0,2));
    
17.         }
    
18.         suoshu=WebUserDeptname.substring(0,2);
    
19. 
    
20.         //找出指定角色的人
    
21.         var roleid = getRoleIdByName(JiaoSeName);       
    
22.         //println("角色ID号：================"+roleid);
    
23.         //找出角色下人员所属部门
    
24.         var users = getUsersByRoleId(roleid);
    
25.         var shenpiren="";
    
26.         for(var iter2=users.iterator();users !=null && iter2.hasNext();)
    
27.         {
    
28.                 var user=iter2.next();
    
29.                 var Names="";
    
30.                 //---------------------------------------------------
    
31.                 //println("用户：================"+user.getName());
    
32.                 //获取用户对象
    
33.                 var userVO2 = getUserById(user.getId());
    
34.                 //获取用户所属部门,这里返回的是用户所属的所有部门
    
35.                 var dept2 = userVO2.getDepartments();
    
36.                 var deptname2="";
    
37.                 for(var iter3 = dept2.iterator();iter3.hasNext();)
    
38.                 {
    
39.                         var value2 = iter3.next();
    
40.                         deptname2 +=user.getName()+":"+value2.getName()+";";
    
41.                        
    
42.                         //查找字符串是否包含特定字符，如：“青岛”
    
43.                         var k =deptname2.indexOf(suoshu);
    
44.                         //println("是否包含特定所属：-------------------"+k);
    
45.                        
    
46.                         if(k>0)
    
47.                         {
    
48.                                 var arr = splitText(deptname2, ":");
    
49.                                 shenpiren=user;
    
50.                         }
    
51.                 }       
    
52.                 //---------------------------------------------------
    
53.                 //Names += user.getName();//用户名称集合
    
54.         }
    
55.         //println("ssssssssssssssssssss"+shenpiren);
    
56.         return shenpiren;
    
57. }

复制代码

您好：您现在的代码您是说只能返回一个正确的用户么？看了你的代码如果您是需要返回多个用户应该是把这些符合条件的用户添加的集合里面去啊，而不是返回单个的用户对象啊，把符合条件的用户添加到集合中去可参考帖子：http://www.teemlink.com/bbs/view ... B%C9%F3%C5%FA%C8%CB
您先尝试，有问题我们再沟通。

好了，谢谢。

恩好的，感谢您的反馈。