Сохранение данных из List<string> в txt

[nomerasd]

многопоточный парсер html.
данные парсятся регулярными выражениями и заносятся в List <string>.
примерный вид List:
1
12
123
1234
...
то есть, каждая новая строка отличается новым добавленным последним элементом.
данные из List, периодически сохраняются в txt.
проблема в том, что в txt записываются все строки из List. отсюда получается множество дублей.
пробовал записывать последнюю строку из List - var x = vfail[vfail.Count - 1];
не помогает. почему-то все равно записывается с дублями. возможно из-за многопоточности, но почему тогда lock не срабатывает.
в общем, как решить данную проблему?

код:
List<string> vfail = new List<string>(); - глобально в проекте.
основной метод:

PHP код:

// good парсим регуляркой
foreach (Match m in good)
                    {
                        result = result + m.Value + "\r\n";
                        lock (vfail)
                        vfail.Add(result);
                    } 


по таймеру срабатывает метод:

PHP код:

lock (vfail)
                if (vfail.Count > 0)
                {
                    var sw = new StreamWriter("ALL.txt", true);
                    foreach (var akk in vfail)
                    sw.Write(akk);
                    sw.Close();
                    vfail.Clear();
                } 


строки в List имеет вид:

PHP код:

YKY.RU\r\n
YKY.RU\r\nYKY.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\nYKY.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\nYKY.RU\r\nYLC.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\nYKY.RU\r\nYLC.RU\r\nYLF.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\nYKY.RU\r\nYLC.RU\r\nYLF.RU\r\nYKY.RU\r\n
YKY.RU\r\nYKY.RU\r\nYLC.RU\r\nYKY.RU\r\nYLC.RU\r\nYLF.RU\r\nYKY.RU\r\nYLC.RU ... 


с каждым последующим добавлением в строку: домен\r\n
вот он мне все эти строки и сохраняет в файл.
вот пример отчёта файла txt -> url

P.S.
периодическая чистка List производится - vfail.Clear();
сам проект как есть - url
можно конечно реализовать и без List, в методе main: спарсить данные, сформировать result, открыть txt, записать result, закрыть txt. Но тогда резко увеличивается время работы программы, т.к. много страниц для парсинга.

[ds.Dante]

почему-то все равно записывается с дублями.

При записи в файл старый текст остаётся в result-е.

Сообщение от nomerasd

можно конечно реализовать и без List, в методе main: спарсить данные, сформировать result, открыть txt, записать result, закрыть txt. Но тогда резко увеличивается время работы программы, т.к. много страниц для парсинга.

Вы пробовали? Если дотнет кеширует запись в файл и если это соответствует спецификациям ECMA по CLI, то можно писать как есть. Файл остаётся открытым; если одновременно нужно читать его, по таймеру делаем Flush().

Если не кеширует, весь текст нужно аккумулировать в одной строке, не в коллекции. Причём result должен быть типа StringBuilder, а не string, иначе на каждой итерации у вас впустую будет создаваться новый экземпляр.

lock (vfail)

Не рекомендуется лочить публичные поля, лучше создать отдельный объект с минимальной видимостью.
https://msdn.microsoft.com/en-us/library/c5kehkcz.aspx
http://stackoverflow.com/questions/6...-is-a-bad-idea

[Alex11223]

Для парсинга HTML обычно проще взять библиотеку для парсинга HTML, чем регексп. HtmlAgilityPack например.

[nomerasd]

Сообщение от ds.Dante

При записи в файл старый текст остаётся в result-е.

СПАСИБО тебе, добрый человек!!!

[nomerasd]

Сообщение от Alex11223

HtmlAgilityPack например.

в архиве есть и с разбором DOM. просто мне показалось, что регулярками быстрее парсится на объёмах.