Написал небольшой парсер, как найти ошибку?

Question

[Anton2001]

Есть код который должен парсить ссылку на самую новую (по времени) новость .
Так как там бываю "горячие" новости которые остаются вверху , парсить просто первую ссылку не вариант .
Есть такой код , он работает в таймере на 3 минуты , но из 7 новых новостей вернул только 5 .
Перебрал весь код , подскажите куда копать ?

public string GetLink()
        {
            WebClient client = new WebClient();
            client.Encoding = Encoding.UTF8;
            string result = client.DownloadString("https://crypto.ru/novosti/");
            DateTime dt = DateTime.MinValue;
            string link = null;
            for (int i = 0; i < 4; i++)
            {
                string item = Pars(result, "class=\"posts-grid-news__item-info", "</a>", i);
                string data = Pars(item, "posts-grid-news__item-date\">", "<", i);

                string tempLink = Pars(item, "href=\"", "\"", 0);

                string tempTime = data.Split(' ')[2];
                string tempDate = data.Split(' ')[0] + " " + data.Split(' ')[1];

                data = tempTime + " " + tempDate;
                DateTime tempDT = DateTime.Parse(data);

                if (tempDT > dt)
                {
                    dt = tempDT;
                    link = tempLink;
                }
            }
            return link;
        }

Comments

[freeExec]

Парсить через поиск текста - ну такое, когда есть нормальная возможность обрабатывать xml как данные.
А так твой вопрос сродни: я написал заклинание, работает норм, но в лунную ночь выдаёт 5 из 7, что не так?

[Anton2001]

freeExec, мне интересно , может я просто как-то с сверкой времени/даты накосячил

[Myclass]

Что то вообще не понимаю, как может быть быть 'вернул 5 из 7', если функция возвращает только один единственный link. Это первое, второе вопрос про 5, в теле функции цикл на 5 итерации. Ничего не напоминает? Хотя всё равно - поставленный вопрос мне непонятен.

[Anton2001]

Myclass,
Постараюсь обьяснить , софт по таймеру вызывает функцию GetLink()
Функция должна отдавать ссылку на самую новую новость ,
Потом таймер сохраняет ссылку на неё в txt файле .
Софт работал сутки , за это время на сайте вышло 7 новостей но софт сохранил только 5 . Те функция GetLink() посчитала только 5 новостей новыми с момента когда софт был запущен

[Myclass]

Anton2001, ну исходя из описанной логики, должна каждые три минуты одна новость сохраниться. Ты не там ищешь. Эта функция работает. Не работает та наверное, где идёт проверка - была-ли эта новость в списке сохранённых или нет и надо ли её сохранять.

[Anton2001]

Myclass, хм , об этом не думал даже ... спасибо , попробую копать .
Хотя не совсем понимаю как это может быть тут , сейчас примерный код напишу

string link = null;
string tempLink = GetLink();
if( link != tempLink)
{
link = tempLink;
// записываем ссылку 
}

[Myclass]

Anton2001, ну логика записи не имеет ошибки. Хотя ещё один вариант, что парсирование каких то новостей и правда оплачивается и не возвращается линк. Короче понаставь всяких выводов на консоли, каждые ли три минуты ты получаешь новость , если нет, то парс, если да, но иногда старый линкольн, хотя новый уже должен был быть, то парс, если да, но сохранение не работает, потому что например на тот файл в который ты хочешь писать - его например ексклюзиано другая программа открывает итд.

[Anton2001]

Myclass, сам сайт может выпускать всего пару новых новостий в день , слишком долго чтоб прямо тестить )
Пробовал свою вёрстку подставлять , вроде отрабатывает нормально ...

Меня берут сомнения , правильно ли я сравниваю время ?
Просто с датами особо не работал никогда , и правильна ли эта логика не уверен
DateTime dt = DateTime.MinValue;

data = tempTime + " " + tempDate;
DateTime tempDT = DateTime.Parse(data);

if (tempDT > dt)
{
dt = tempDT;
link = tempLink;
}

Можно ли вообще оператором больше / меньше её сравнивать ...

[Myclass]

Anton2001,а не было такой идеи, что новости при сравнении не между мобой сравнивают а со списком последних сообщений. Последнего часа, или последнии сто...
Не могу понять, почему нельзя функцию мориторитб и не писать в файл. Это первое, а второе - функцию можно стимулировать и понять, работает ли запись. Думай немного ьольше вокруг да около.

[Myclass]

на вопрос не ответил.

Можно ли вообще оператором больше / меньше её сравнивать ...

Нет, я так никогда не делал. Всегда только через DateTime.Compare, если просто сравнить надо (пример от микрософта)

DateTime date1 = new DateTime(2009, 8, 1, 0, 0, 0);
DateTime date2 = new DateTime(2009, 8, 1, 12, 0, 0);
int result = DateTime.Compare(date1, date2);
string relationship;

if (result < 0)
   relationship = "is earlier than";
else if (result == 0)
   relationship = "is the same time as";
else
   relationship = "is later than";

Console.WriteLine("{0} {1} {2}", date1, relationship, date2);

и. если надо знать разницу между двумя датами, то

long TicksBetween = Convert.ToDateTime( pDateEnd ).Ticks - Convert.ToDateTime( pDateBegin ).Ticks;

// Convert.ToInt32( TimeSpan.FromTicks( TicksBetween ).TotalMilliseconds );
...
// Convert.ToInt32( TimeSpan.FromTicks( TicksBetween ).TotalDays );