Т.е мне нужно 2 метода по получению ссылки и ссылок. Проблема в том, что просто нельзя сделать 1 метод,из которого вернуть коллекцию и из него взять первый элемент(если нужна 1ссылка). На это уйдет больше времени, чем вернуть 1 ссылку. В голове сейчас только реализации паттерна шаблонный метод, но я не знаю как его туда поставить.
Вот проблемный код
public async Task<string> GetWorkingLink(string htmlContent)
{
IDocument angleDocument = await _publicHttp.Context.OpenAsync(html => html.Content(htmlContent));
foreach (var linksElement in angleDocument.QuerySelector("#b_results")?.QuerySelectorAll("cite"))
{
string workingLink = linksElement.TextContent;
var httpResponse = await _publicHttp.Client.GetAsync(workingLink);
if (httpResponse.StatusCode != HttpStatusCode.OK)
{
continue;
}
var siteHtml = await httpResponse.Content.ReadAsStringAsync();
var filmNamesCollection = await new SiteParser().GetPopularFilmsName(siteHtml, 5);
if (filmNamesCollection.Count() == 5)
return workingLink;
}
return System.Configuration.ConfigurationManager.
ConnectionStrings["BackUpLink"].ConnectionString;
}
Итак, проблема тут
if (filmNamesCollection.Count() == 5)
return workingLink;
Мне нужно вернуть ссылку, либо добавить в коллекцию эту ссылку, взависимости какой метод выберут. Как мне это сделать без дубликации кода?
Что мне нужно, как я это вижу сейчас
public async Task<string> GetWorkingLink(string htmlContent)
{
IDocument angleDocument = await _publicHttp.Context.OpenAsync(html => html.Content(htmlContent));
foreach (var linksElement in angleDocument.QuerySelector("#b_results")?.QuerySelectorAll("cite"))
{
string workingLink = linksElement.TextContent;
var httpResponse = await _publicHttp.Client.GetAsync(workingLink);
if (httpResponse.StatusCode != HttpStatusCode.OK)
{
continue;
}
var siteHtml = await httpResponse.Content.ReadAsStringAsync();
var filmNamesCollection = await new SiteParser().GetPopularFilmsName(siteHtml, 5);
if (filmNamesCollection.Count() == 5)
return workingLink;
}
return System.Configuration.ConfigurationManager.
ConnectionStrings["BackUpLink"].ConnectionString;
}
public async Task<IEnumerable<string>> GetWorkingLinks(string htmlContent)
{
List<string> links = new List<string>();
IDocument angleDocument = await _publicHttp.Context.OpenAsync(html => html.Content(htmlContent));
foreach (var linksElement in angleDocument.QuerySelector("#b_results")?.QuerySelectorAll("cite"))
{
string workingLink = linksElement.TextContent;
var httpResponse = await _publicHttp.Client.GetAsync(workingLink);
if (httpResponse.StatusCode != HttpStatusCode.OK)
{
continue;
}
var siteHtml = await httpResponse.Content.ReadAsStringAsync();
var filmNamesCollection = await new SiteParser().GetPopularFilmsName(siteHtml, 5);
if (filmNamesCollection.Count() == 5)
links.Add(workingLink);
}
return links;
}
Разница
if (filmNamesCollection.Count() == 5)
return workingLink;
}
return System.Configuration.ConfigurationManager.
ConnectionStrings["BackUpLink"].ConnectionString;
if (filmNamesCollection.Count() == 5)
links.Add(workingLink);
}
return links;
Я хочу сделать это без дубляжа кода, не повторяя пол метода, ради 2ух строк
yield
возвращение, тогда у вас будет некий контроль над тем, что получаете. То есть, самое базовое, будет нечто такое, если надо чисто 1-й или все, без просчитывания всей коллекции, то это один из вариантов решения. Только учтите, дляasync
там немного иначе (почитайте проIAsyncEnumerable
из C#8)..csproj
версию языка (<LangVersion>latest</LangVersion>
или прям версию<LangVersion>8.0</LangVersion>
), а также, дляIAsyncEnumerable
установить через NuGet пакетMicrosoft.Bcl.AsyncInterfaces
. Также для сборки может потребоваться пакетMicrosoft.Net.Compilers
. Если у вас Visual Studio 2017, то новый код будет подсвечивать ошибками, но проект успешно соберется. Ну а так, пора обновляться до .NET6!.