h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-01-14 20:13:22 +08:00
Code Issues Releases Wiki Activity

Changed internal closedChannel list to store channels names instead of channel objects

Browse Source 
This is for efficiency purposes; it's more efficient to just store
the names than store the whole object.
This commit is contained in:
Joseph-Ramos-CMU 2020-12-16 10:13:50 -05:00
parent 1b702b2b1f
commit 7b169c4f62
1 changed files with 19 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
osu.Game/Online/Chat/ChannelManager.cs
View File

@ -33,8 +33,8 @@ namespace osu.Game.Online.Chat
private readonly BindableList<Channel> availableChannels = new BindableList<Channel>();
private readonly BindableList<Channel> joinedChannels = new BindableList<Channel>();
// Keeps a list of closed channels. More recently closed channels appear at higher indeces
private readonly BindableList<Channel> closedChannels = new BindableList<Channel>();
// Keeps a list of closed channel identifiers
private readonly BindableList<string> closedChannels = new BindableList<string>();
// For efficiency purposes, this constant bounds the number of closed channels we store.
// This number is somewhat arbitrary; future developers are free to modify it.
@ -424,7 +424,7 @@ namespace osu.Game.Online.Chat
}
// insert at the end of the closedChannels list
closedChannels.Insert(closedChannels.Count, channel);
closedChannels.Insert(closedChannels.Count, channel.Name);
if (channel.Joined.Value)
{
@ -445,13 +445,24 @@ namespace osu.Game.Online.Chat
return;
}
Channel lastClosedChannel = closedChannels.Last();
string lastClosedChannelName = closedChannels.Last();
closedChannels.RemoveAt(closedChannels.Count - 1);
closedChannels.Remove(lastClosedChannel);
// types did not work with existing enumerable interfaces funcitons like Contains
for (int i = 0; i < joinedChannels.Count; i++)
{
// If the user already joined the channel, try the next
// channel in the list
if (joinedChannels[i].Name == lastClosedChannelName)
{
JoinLastClosedChannel();
return;
}
}
// If the user already joined the channel, try the next
// channel in the list
if (joinedChannels.IndexOf(lastClosedChannel) >= 0)
Channel lastClosedChannel = AvailableChannels.FirstOrDefault(c => c.Name == lastClosedChannelName);
if (lastClosedChannel == null)
{
JoinLastClosedChannel();
}