2

У меня есть следующий код

string urlAddress = $"https://api.coingecko.com/api/v3/coins/list";

WebRequest req = WebRequest.Create(urlAddress);
WebResponse resp = req.GetResponse();
Stream stream = resp.GetResponseStream();
StreamReader sr = new(stream);
string s = sr.ReadToEnd();

Dictionary<string, string> symbolId = new();

string[] symbol = Regex.Matches(s, "(?<=\"symbol\":\")[^\"]+").Select(x => x.Value.ToUpper()).ToArray();
string[] id = Regex.Matches(s, "(?<=\"id\":\")[^\"]+").Select(x => x.Value).ToArray();

for (int i = 0; i < symbol.Length; i++)
    if (!symbolId.ContainsKey(symbol[i]))
        symbolId.Add(symbol[i], id[i]);

StreamWriter sw = new(@"C:\Users\hem12\source\repos\PNL_Now\symbol_id.json");
var options = new JsonSerializerOptions { WriteIndented = true };
sw.Write(JsonSerializer.Serialize(symbolId, options));

В данном коде я получаю достаточно большой список (10251 строчек в JSON). Словарь содержит правильное количество объектов, но вот при записи в файл все останавливается на 10094 строчке, при чем строчка не дописывается до конца. JsonSerializer.Serialize() работает корректно, вся проблема лишь в StreamWriter, который не может полностью записать данные в файл. С чем это связано и как это решить?

9
  • Могу предоставить полный код если нужно
    – Miclell
    Commented 2 фев 2022 в 18:30
  • А зачем вам тут вообще Stream, не уж то 10к строчек, это прям зависоны при сериализации десериализации?
    – EvgeniyZ
    Commented 2 фев 2022 в 18:39
  • А что тогда вместо Stream можно использовать? Зависонов то при сериализации десериализации нет. Тем более, оказывается, что данный глюк возникает при любом количестве объектов с словаре
    – Miclell
    Commented 2 фев 2022 в 18:42
  • 1
    Да просто File.WriteAllText("file", JsonSerializer.Serialize(symbolId)); и обратно symbolId = JsonSerializer.Deserialize<Dictionary<string, string>>(jsonString);, вот вам две строки, без стримов, с загрузкой и выгрузкой JSON данных. Ну и да, с Regex тоже ерундой страдаете, создали класс с нужными свойствами и работаете. А, ну и WebRequest является устаревшим классом в C#! Используйте HttpClient.
    – EvgeniyZ
    Commented 2 фев 2022 в 18:56
  • 1
    Ну раз пишет "файл занят", значит не все они "неактивны", верно? Ну а так, вот вся реализация вашего кода, не пойму правда зачем там словарь, но дело ваше. Так что подумайте, верно ли вы реализовали все у себя, или нет...
    – EvgeniyZ
    Commented 2 фев 2022 в 19:30

1 ответ 1

1

Помылся - закрой душ.


using (StreamWriter sw = new(...)) {
  ...
}
5
  • а почему мой вариант не работает?
    – Miclell
    Commented 2 фев 2022 в 18:54
  • @Miclell Хм, потому что Вы не закрыли.
    – user176262
    Commented 2 фев 2022 в 18:56
  • по фактам, спасибо:)
    – Miclell
    Commented 2 фев 2022 в 18:58
  • 1
    @aepot, спасибо, стало намного понятнее
    – Miclell
    Commented 2 фев 2022 в 22:10

Сайт временно доступен в режиме «только для чтения», и публикация новых ответов невозможна.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками .