0

Добрый день!

Вопрос такой!
Пытаюсь записать все элементы коллекции в файл json, но почему-то записываеться только последний элемент коллекции в файл!

Вот код, который я использую для вывода в консоль и запись их в файл:

        foreach (var m in listHashRepo)
        {
            Console.ForegroundColor = System.ConsoleColor.Cyan;
            Console.WriteLine(m);

            JObject wcomm = new JObject(new JProperty("url", m));
            File.WriteAllText("files.json", wcomm.ToString());
        }

В файле записываеться только последний элемент:

{"url": "e92bc448678ef499cfeabb3df93ea7adad47f747"}

А хотелось, чтобы он записывал всё. Примерно так:

{"url": "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345"}
{"url": "89fsd898c23993d9571cmvfnjh450cnklfhijrwi"}
{"url": "43j24j5390fsnto0g775jli43omh5oh632ji5p5p"}
{"url": "e92bc448678ef499cfeabb3df93ea7adad47f747"}

Помогите разобраться, в чем может быть проблема?

5
  • Неудивительно, вы же переписываете файл заново на каждой итерации. Пожалуйста, поместите ваш код не в виде картинки, а в виде текста, и дайте пример того, какой выходной формат файла вы хотите.
    – VladD
    16 фев 2017 в 13:37
  • @VladD Поправил вопрос! 16 фев 2017 в 13:43
  • Но то, что вы хотите, это же не JSON! JSON-формат выглядел бы как-то так: [ {"url": "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345"}, {"url": "89fsd898c23993d9571cmvfnjh450cnklfhijrwi"} ].
    – VladD
    16 фев 2017 в 13:44
  • @VladD Да, не поставил скобки, извиняюсь. 16 фев 2017 в 13:45
  • Цикл foreach предназначен только для перебора (чтения), а не для записи!!! 23 фев 2020 в 12:36

2 ответа 2

3

Вообще-то, чтобы сериализовать коллекцию, можно написать просто

File.WriteAllText("files.json", hashes, Formatting.Indented);

(не в цикле!) Это произведёт такой JSON:

[
  "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345",
  "89fsd898c23993d9571cmvfnjh450cnklfhijrwi",
  "43j24j5390fsnto0g775jli43omh5oh632ji5p5p",
  "e92bc448678ef499cfeabb3df93ea7adad47f747"
]

Если вам нужно ещё и "url", это делается немного сложнее:

File.WriteAllText(
    "files.json",
    JsonConvert.SerializeObject(
        hashes.Select(hash => new { url = hash }),
        Formatting.Indented));

Результат будет такой:

[
  {
    "url": "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345"
  },
  {
    "url": "89fsd898c23993d9571cmvfnjh450cnklfhijrwi"
  },
  {
    "url": "43j24j5390fsnto0g775jli43omh5oh632ji5p5p"
  },
  {
    "url": "e92bc448678ef499cfeabb3df93ea7adad47f747"
  }
]

Из комментариев: если у вас на самом деле две последовательности, их можно объединить при помощи Zip:

var hashes = new[]
{
    "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345",
    "89fsd898c23993d9571cmvfnjh450cnklfhijrwi",
    "43j24j5390fsnto0g775jli43omh5oh632ji5p5p",
    "e92bc448678ef499cfeabb3df93ea7adad47f747"
};
var urls = new[]
{
    "https://github.com/VladislavsGeidans/wikicar.git",
    "https://github.com/VladislavsGeidans/1.git",
    "https://github.com/VladislavsGeidans/2.git",
    "https://github.com/VladislavsGeidans/3.git",
};

File.WriteAllText(
    "files.json",
    JsonConvert.SerializeObject(
        hashes.Zip(urls, (hash, url) => new { hash, url }),
        Formatting.Indented));

Результат:

[
  {
    "hash": "24gfd5ff7f9fd5gs98uf349k31u6g2134io6h345",
    "url": "https://github.com/VladislavsGeidans/wikicar.git"
  },
  {
    "hash": "89fsd898c23993d9571cmvfnjh450cnklfhijrwi",
    "url": "https://github.com/VladislavsGeidans/1.git"
  },
  {
    "hash": "43j24j5390fsnto0g775jli43omh5oh632ji5p5p",
    "url": "https://github.com/VladislavsGeidans/2.git"
  },
  {
    "hash": "e92bc448678ef499cfeabb3df93ea7adad47f747",
    "url": "https://github.com/VladislavsGeidans/3.git"
  }
]
7
3

Нужен некоторый анализ, различающий запись первого объекта и последующих.

File.Delete("files.json");
foreach (var m in listHashRepo)
{
  Console.ForegroundColor = System.ConsoleColor.Cyan;
  Console.WriteLine(m);

  JObject wcomm = new JObject(new JProperty("url", m));
  if (!File.Exists("files.json"))  
    File.WriteAllText("files.json", "[" + wcomm.ToString());
  else
    File.AppendAllText("files.json", "," + wcomm.ToString());
}
File.AppendAllText("files.json", "]");

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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