Как преобразовать List в строку?

Question

[Alex Lav]

Подскажите пожалуйста, уже неделю ломаем голову почему из list вытягивается в строку только первую запись NickName=9813281328787132213,
Остальные записи игнорируются., а должно быть NickName=9813281328787132213,NickName2=199124912441923121,NickName3=14421412412412412

using System.Collections.Generic;
using System.Linq;

namespace Oxide.Plugins
{
    [Info("GameSib", "Drop Dead", "1.0.0")]
    public class GameSib : RustPlugin
    {
        public Dictionary<ulong, string> playerlist = new Dictionary<ulong, string>();
        const int secondstoupdate = 120;
        const string serverid = "169";
        const string api = "https://site.ru/playerinfo.php";
        const string args = "?idserver={ID}&update_time={secondstoupdate}&ip={IP}&port={PORT}&qport={QPORT}&online={ONLINE}&slots={SLOTS}&players_list={NICNAME_SteamID}";

        void OnServerInitialized()
        {
            InvokeHandler.Instance.InvokeRepeating(SendAPI, 20f, secondstoupdate);

            foreach (var player in BasePlayer.activePlayerList)
                OnPlayerConnected(player);
        }

        void Unload()
        {
            InvokeHandler.Instance.CancelInvoke(SendAPI);
        }

        void OnPlayerConnected(BasePlayer player)
        {
            if (player.IsReceivingSnapshot)
            {
                NextTick(() => OnPlayerConnected(player));
                return;
            }

            if (!playerlist.ContainsKey(player.userID))
                playerlist.Add(player.userID, player.displayName);
        }

        void OnPlayerDisconnected(BasePlayer player)
        {
            if (playerlist.ContainsKey(player.userID))
                playerlist.Remove(player.userID);
        }

        void SendAPI()
        {
            var playerlists = DictToString(playerlist);
            string link = api + args.Replace("{ID}", serverid).Replace("{secondstoupdate}", secondstoupdate.ToString()).Replace("{IP}", ConVar.Server.ip).Replace("{PORT}", ConVar.Server.port.ToString()).Replace("{QPORT}", ConVar.Server.queryport.ToString())
            .Replace("{ONLINE}", BasePlayer.activePlayerList.Count.ToString()).Replace("{SLOTS}", ConVar.Server.maxplayers.ToString()).Replace("{NICNAME_SteamID}", playerlists);

            Puts(link);
            webrequest.Enqueue(link, null, (code, response) =>
            {

            }, this, Core.Libraries.RequestMethod.POST);
        }

        public static string DictToString(Dictionary<ulong, string> dictionary)
        {
            if (dictionary == null)
                return "null";
            if (dictionary.Count < 1) return "null";

            var items = from kvp in dictionary
                        select kvp.Key + "=" + kvp.Value;

            return "{" + string.Join(",", items) + "}";
        }
    }
}

Comments

[Сергей Соловьев]

А точно игнорируются? Может этот словарь и изначально содержал только 1 пользователя

[Alex Lav]

Сергей Соловьев, нет их там несколько, а выводит только одного.

[Сергей Соловьев]

Alex Lav, если вопрос про API, это это действительно странное поведение - это прямое нарушение API/контракта.
Тут скорее проблема на вашей стороне:
- Может в BasePlayer.activePlayerList только 1 пользователь
- Может

if (player.IsReceivingSnapshot)
{
        NextTick(() => OnPlayerConnected(player));
        return;
}

постоянно срабатывает
- Может вашей библиотеке для отправки запросов поплохело
- Этот public class GameSib : RustPlugin на каждого пользователя создается

Этот маленький кусок кода мало что говорит

[#]

Alex Lav, точно формулируйте задачу. если цель именно такова, с небольшой правкой, более чем достаточен код от Nik Faraday

[#]

... при попытке воспроизвести код, требует плагин rust.. это глюк? если нет - какой именно? ))

Answer 1

[Nik Faraday]

Зачем нам весь ваш код?

Вот небольшой метод для решения вашей задачи:

public string ListConcat(ICollection<Player> players)
{
     string list = string.Empty;
     foreach (Player player in players)
          list += player.Name;
}

А вообще посмотрите в сторону метода string.Concat

Comments

[#]

только по задумке автора, надо еще запятую вставлять ))

[#]

а еще и "=" и id

[Nik Faraday]

#, ну, там буквально два слова дописать, думаю автор справится ))

[OwDafuq]

Имхо, лучше уж string.Join, а не Concate

[Nik Faraday]

OwDafuq, верно. Как вы видите, вариантов для решения есть очень много

Answer 2

[#]

using System.Runtime.CompilerServices;
using System.Text;

public static class ca2
{
    public static void Main()
    {
        var d = new Dictionary<long, string>();
        var s = "";
        var sb = new StringBuilder();
        var rnd = new Random();
        var cnt = rnd.Next(8, 12);
        for (int i = 0; i < cnt; i++) d.Add(rnd.Next(), $"Name {i}");
        foreach (var v in d) sb.Append($"{v.Value} = {v.Key}, ");
        s = sb.ToString();
        s = s.Substring(0, s.Length - 2);
        s.print();
    }
    public static void print(this string s) => Console.WriteLine(s);
}

... если интересует производительность StringBuilder впереди планеты всей...