Andronov Alexander #178
Andronov Alexander #178
Conversation
| { | ||
| public static AppOptions CreateOptions(Options clOptions, IConfiguration defaultConfig) | ||
| { | ||
| var tagCloudOptions = CreateTagCloudOptions(clOptions, defaultConfig); |
|
|
||
| var result = textLoader.Load(inp) | ||
| .Then(text => tagCloud.CreateCloud(text)) | ||
| .Then(image => imageStorage.Save(image, outp)) |
| public static class DependencyInjection | ||
| { | ||
| public static IServiceCollection AddDomain | ||
| (this IServiceCollection services, |
| } | ||
|
|
||
| services.AddScoped<IWordsFilter, LengthFilter>(); | ||
| services.AddScoped<IWordsFilter, MorphologicalFilter>(); |
There was a problem hiding this comment.
В итоге же всё равно добавится оба типа? В чем смысл switch?
| { | ||
| var dict = new Dictionary<T, int>(); | ||
|
|
||
| foreach (var item in items) |
There was a problem hiding this comment.
Попробуй обойтись Linq, в таком случае сама надобность в интерфейсе и классе может отпасть
|
|
||
| var parsed = result.Select(ParseResult).ToArray(); | ||
|
|
||
| var filtered = parsed.Where(x => partsSpeech.HasFlag(x.partSpeech)).Select(x => x.word).ToArray(); |
There was a problem hiding this comment.
HasFlag принимает тип Enum. Можешь сказать, чем это чревато и можно ли сделать лучше?
| }; | ||
| } | ||
|
|
||
| (string word, PartSpeech partSpeech) ParseResult(string result) |
| public enum PartSpeech | ||
| { | ||
| None = 0, | ||
| Adjective = 1, |
There was a problem hiding this comment.
Я бы воспользовался смещением / степенью
|
|
||
| var dict = builder.Build(words); | ||
|
|
||
| dict.Should().BeEquivalentTo(correctDict); |
There was a problem hiding this comment.
Как уже говорил, можно было впихнуть это в linq. Думается, что интерфейс, класс и тесты избыточны для подсчета частотности.
| @@ -0,0 +1,32 @@ | |||
| using FluentAssertions; | |||
|
|
|||
| namespace TagCloud.Tests | |||
There was a problem hiding this comment.
Маловато тестов.
Как минимум, я бы хотел увидеть, что у тебя правильно определяются части речи, парочку тестов на Result
No description provided.