From 6f94de5a14579b13080862b05c91e4e3df706256 Mon Sep 17 00:00:00 2001 From: Daan Boerlage Date: Mon, 1 Nov 2021 00:16:42 +0100 Subject: [PATCH] Reduce ulong to long casts in the roll command --- src/Commands/Roll/Roll.cs | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/src/Commands/Roll/Roll.cs b/src/Commands/Roll/Roll.cs index e068033..a431bd9 100644 --- a/src/Commands/Roll/Roll.cs +++ b/src/Commands/Roll/Roll.cs @@ -1,6 +1,3 @@ -using System; -using System.Linq; -using System.Threading.Tasks; using Geekbot.Core; using Geekbot.Core.Database; using Geekbot.Core.Database.Models; @@ -26,16 +23,15 @@ namespace Geekbot.Commands.Roll public async Task RunFromGateway(ulong guildId, ulong userId, string userName, string unparsedGuess) { int.TryParse(unparsedGuess, out var guess); - return await this.Run(guildId, userId, userName, guess); + return await this.Run(guildId.AsLong(), userId.AsLong(), userName, guess); } public async Task RunFromInteraction(string guildId, string userId, string userName, int guess) { - - return await this.Run(ulong.Parse(guildId), ulong.Parse(userId), userName, guess); + return await this.Run(long.Parse(guildId), long.Parse(userId), userName, guess); } - private async Task Run(ulong guildId, ulong userId, string userName, int guess) + private async Task Run(long guildId, long userId, string userName, int guess) { var number = _randomNumberGenerator.Next(1, 100); @@ -73,18 +69,18 @@ namespace Geekbot.Commands.Roll } } - private async Task GetUser(ulong guildId, ulong userId) + private async Task GetUser(long guildId, long userId) { - var user = _database.Rolls.FirstOrDefault(u => u.GuildId.Equals(guildId) && u.UserId.Equals(userId.AsLong())) ?? await CreateNewRow(guildId, userId); + var user = _database.Rolls.FirstOrDefault(u => u.GuildId.Equals(guildId) && u.UserId.Equals(userId)) ?? await CreateNewRow(guildId, userId); return user; } - private async Task CreateNewRow(ulong guildId, ulong userId) + private async Task CreateNewRow(long guildId, long userId) { var user = new RollsModel() { - GuildId = guildId.AsLong(), - UserId = userId.AsLong(), + GuildId = guildId, + UserId = userId, Rolls = 0 }; var newUser = _database.Rolls.Add(user).Entity;