Из трех человек, успешно закончивших тестовое задание - двое сделали этот баг (включили "Apply Now"). Что забавно, эти двое - добавили еще по одному багу каждый (разные баги). Один - добавил try-catch, который пожирал exception и прятал его в логах (я НЕ просил добавить exception handling). Другой решил щегольнуть тем, как он ловко добавляет прослойку (которую я НЕ просил добавлять):
public static class HtmlExtensions
{
private static HtmlWeb _web = new HtmlWeb();
public static HtmlDocument GetHtmlFromWebSite(this string url) => _web.Load(url);
public static HtmlNode SelectNode(this HtmlDocument document, string pnodePath) => document.DocumentNode.SelectSingleNode(pnodePath);
}
Увы, он не только добавил ненужный код, но и добавил трудноуловимый mutli-treading bug из-за использования статического поля для хранения _web instance.
Bugs in test tasks
Что забавно, эти двое - добавили еще по одному багу каждый (разные баги).
Один - добавил try-catch, который пожирал exception и прятал его в логах (я НЕ просил добавить exception handling).
Другой решил щегольнуть тем, как он ловко добавляет прослойку (которую я НЕ просил добавлять):
public static class HtmlExtensions { private static HtmlWeb _web = new HtmlWeb(); public static HtmlDocument GetHtmlFromWebSite(this string url) => _web.Load(url); public static HtmlNode SelectNode(this HtmlDocument document, string pnodePath) => document.DocumentNode.SelectSingleNode(pnodePath); }Увы, он не только добавил ненужный код, но и добавил трудноуловимый mutli-treading bug из-за использования статического поля для хранения _web instance.