2017-10-19 27 views
1

我想创建一个命令,返回当前玩特定游戏的公会成员的数量。Discord.NET玩同一游戏的用户

示例(!是我的前缀):!打了英雄联盟。

如果有5名成员打联赛的传奇,输出:

There are 5 users currently playing League of Legends.

我设置以下,从调试我能够拿起v.Game.toString()返回正确的字符串,但由于某种原因,如果语句不触发。它还捕捉到每当成员不玩游戏时引发的异常(我认为它是空的?),是否有解决方法?为什么这不算有多少成员玩某个游戏?

[Command("playing")] 
    public async Task playingGame(params string[] s) 
    { 
     string gameName = ""; 
     int count = 0; 

     for (int i = 0; i < s.Length; i++) 
     { 
      gameName += s[i] + " "; 
     } 

     await Context.Channel.SendMessageAsync("Looking for: " + gameName); 

     var u = Context.Guild.Users; 
     foreach (var v in u) 
     { 
      await Context.Channel.SendMessageAsync("v = " + v.ToString()); 
      try 
      { 
       await Context.Channel.SendMessageAsync(v.Game.ToString()); 
       if (v.Game.ToString().ToLower().Equals(gameName.ToLower())) 
       { 
        count++; 
        await Context.Channel.SendMessageAsync("Found match, count = " + count); 
       } 
      } 
      catch (Exception x) 
      { 
       await Context.Channel.SendMessageAsync("Exception throw caught"); 
      } 
     } 

     if (count > 1) { 
      await Context.Channel.SendMessageAsync("There are " + count + " users currently playing " + gameName + "."); 
     } 
     else if (count == 1) 
     { 
      await Context.Channel.SendMessageAsync("There is " + count + " user currently playing " + gameName + "."); 
     } 
     else if (count == 0) 
     { 
      await Context.Channel.SendMessageAsync("No one is currently playing " + gameName + "."); 
     } 
    } 

这是例外:

System.ArgumentException: Argument cannot be blank 
Parameter name: Content 
at Discord.Preconditions.NotNullOrEmpty(String obj, String name, String msg) 
at Discord.API.DiscordRestApiClient.<CreateMessageAsync>d__77.MoveNext() 
--- End of stack trace from previous location where exception was thrown --- 

产品图,其中if语句应该触发(用户名封锁隐私的原因):

enter image description here

+0

可以共享例外? – aloisdg

+0

@aloisdg添加了if语句应该触发的异常消息和图片。 –

+0

我知道这个字符串是空的,因为他们没有玩游戏,所以它没有被初始化。我主要关心的是if语句没有触发, –

回答

1

没有必要有params string[] s为游戏参数。只需使用Remainder属性即可。

我也简化了这个代码很多

[Command("playing")] 
    public async Task GetUsersPlaying([Remainder]string game) 
    { 
     await Context.Message.DeleteAsync(); 

     var users = Context.Guild.Users.Where(x => x.Game.ToString() == game).Distinct().Select(x => x.Username); 
     var count = users.Count(); 

     var SeparatedList = string.Join(", ", users); 

     string message; 
     if (count > 1) 
      message = $"There are {count} users playing {game}. [{SeparatedList}]"; 
     else if (count == 1) 
      message = $"There is {count} user playing {game}. [{SeparatedList}]"; 
     else 
      message = $"There is no one playing {game}."; 

     await Context.Channel.SendMessageAsync(message); 
    } 
+0

这很优雅,使用Linq。为了在句子后显示球员列表,我将如何创建这些x成员名称的列表? –

+1

你也可以从'UserList'中获得计数,而不是有一个完全不同的变量。 – Unknown