Брозовский Максим #11
Conversation
| private const double OptimalRadius = 1; | ||
| private const double OptimalAngleOffset = 0.5; |
There was a problem hiding this comment.
Можно вынести в какой-нибудь класс конфигурации для настройки layouter'a и передавать в него как настройки
| public static class EnumerableExtensions | ||
| { | ||
| public static string Join(this IEnumerable<string> words, string separator = "") => string.Join(separator, words); | ||
|
|
||
| public static IEnumerable<string> ToLower(this IEnumerable<string> words) => words.Select(w => w.ToLower()); | ||
| } |
There was a problem hiding this comment.
Как будто лишняя оболочка уже существующих методов
|
|
||
| namespace TagCloud.Visualization; | ||
|
|
||
| public class TagCloudImageGenerator(SKSizeI Size, SKTypeface fontFamily ,SKColor background, SKColor foreground) |
There was a problem hiding this comment.
Тоже кажется прикольно было бы иметь конфигурацию
|
Пока всё норм, но давай постараемся сделать максимально настраиваемый генератор, то есть сделать программу такой, чтобы можно было довольно просто ей пользоваться, но при этом можно было создавать очень разные "облака". К примеру, на вход у меня есть файл с каким-то текстом и я хочу создать облако тегов. Я могу просто закинуть файл и получить результат, а могу точечно настроить: указать шрифт, размер текста, размер картинки, исключить местоимения из слов и т.п. штуки. |
TagCloudClient/Program.cs
Outdated
| builder | ||
| .RegisterType<FileReader>().As<IWordsReader>() | ||
| .OnlyIf(_ => Path.GetExtension(settings.FilePath) == ".txt"); | ||
|
|
||
| builder | ||
| .RegisterType<CsvFileReader>().As<IWordsReader>() | ||
| .OnlyIf(_ => Path.GetExtension(settings.FilePath) == ".csv"); | ||
|
|
||
| builder | ||
| .RegisterType<WordFileReader>().As<IWordsReader>() | ||
| .OnlyIf(_ => Path.GetExtension(settings.FilePath) == ".docx"); |
There was a problem hiding this comment.
Точно не уверен, но кажется если взять файл другого расширения, то всё развалится так как не будет ни одного зарегистрированного IWordsReader
| "../../../../TagCloud/WordsFilter/Dictionaries/EnUS.dic", | ||
| "../../../../TagCloud/WordsFilter/Dictionaries/EnUS.aff"); |
There was a problem hiding this comment.
Давай вынесем в пути относительно проекта
TagCloudClient/Options.cs
Outdated
| Required = false, | ||
| HelpText = "Words font")] | ||
| public string FontFamilyName { get; set; } = "Arial"; | ||
| public SKTypeface FontFamily => SKTypeface.FromFamilyName(FontFamilyName); |
There was a problem hiding this comment.
FromFamilyName насколько мне известно может вернуть null, давай как-то сделаем чтобы если шрифт оказался некорректным, чтобы это не ломало работу программы, а просто подставлялся дефолтный
TagCloudClient/Program.cs
Outdated
| builder.RegisterType<TagCloudGenerator>().AsSelf(); | ||
| builder.RegisterType<TagCloudBitmapGenerator>().AsSelf(); | ||
| builder.RegisterType<TagCloudSaver>().AsSelf(); |
There was a problem hiding this comment.
Давай завяжемся на интерфейсах, а не реализациях
@ksamnole