Salastil/KfChatDotNet
1
0
Fork 0
mirror of https://github.com/barelyprofessional/KfChatDotNet.git synced 2026-05-02 04:22:04 -04:00
Code Issues Packages Projects Releases Wiki Activity

Original TruncateBytes extension method wasn't working right, seemed to include an extra character which would put us over the limit. Replaced it and new method seems to be working as expected albeit not as nice looking.

Browse Source
This commit is contained in:
barelyprofessional
2025-02-09 16:23:35 +08:00
parent f0fc79c142
commit 42e6589e2a
2 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
KfChatDotNetBot/ChatBot.cs
View File

@@ -302,8 +302,6 @@ public class ChatBot
if (lengthLimitBehavior == LengthLimitBehavior.TruncateExactly)
{
// ReSharper disable once ReplaceSubstringWithRangeIndexer
// The range indexer is a fucking piece of shit that does not work.
// TrimEnd in case you end up truncating on a space (happened during testing) as Sneedchat will trim it
messageTracker.Message = messageTracker.Message.TruncateBytes(lengthLimit).TrimEnd();
}
@@ -311,6 +309,7 @@ public class ChatBot
messageTracker.Status = SentMessageTrackerStatus.WaitingForResponse;
messageTracker.SentAt = DateTimeOffset.UtcNow;
_logger.Debug($"Message is {messageTracker.Message.Utf8LengthBytes()} bytes");
_sentMessages.Add(messageTracker);
await KfClient.SendMessageInstantAsync(messageTracker.Message);
return messageTracker;

18
KfChatDotNetBot/Extensions.cs
View File

@@ -115,10 +115,18 @@ public static class Extensions
public static string TruncateBytes(this string s, int limitBytes)
{
return Encoding.UTF8.GetString(
Encoding.UTF8.GetBytes(s)
.Take(limitBytes)
.ToArray()
).TrimEnd();
if (string.IsNullOrEmpty(s) || limitBytes <= 0)
{
return string.Empty;
}
if (s.Utf8LengthBytes() <= limitBytes)
{
return s;
}
var bytes = Encoding.UTF8.GetBytes(s);
var charCount = Encoding.UTF8.GetCharCount(bytes, 0, limitBytes);
return s.Substring(0, charCount);
}
}