Кашин Александр #19
Кашин Александр #19
Conversation
TagCloud/Client/ConsoleClient.cs
Outdated
| private readonly App _app; | ||
| private readonly ISettingsProvider _settingsProvider; | ||
|
|
||
| public ConsoleClient(App app, ISettingsProvider settingsProvider) |
There was a problem hiding this comment.
Хотелось бы, чтобы клиенты были в отдельном запускаемом проекте (для каждого свой), а весь код облака тегов был доступен в виде ограниченного набора классов из библиотеки
TagCloud/TagPositioner/Circular/CircularCloudLayouterExtensions.cs
Outdated
Show resolved
Hide resolved
| _wordsReader = wordsReader; | ||
| _settingsProvider = settingsProvider; | ||
| } | ||
| public IEnumerable<string> Process() |
There was a problem hiding this comment.
Довай отрефакторим проект (расставим пробельные символы и переносы строк верно), у райдера есть специальная команда для этого, если ты в нем работаешь
|
|
||
| public class AppSettingsProvider : IAppSettingsProvider | ||
| { | ||
| public AppSettings AppSettings { get; set; } = null!; |
There was a problem hiding this comment.
В целом в рамках приложения как будто логично сделать настройки статичными, либо явно передавать их параметром в код библиотеки (облака тегов), потому что сейчас их можно в любой момент из любого места пререопределить
There was a problem hiding this comment.
Решил на вход в библиотеку принимать настройки и сетить в провайдеры. Сами провайдеры сделал internal.
| graphics.DrawString(tag.Word, font, brush, tag.Location); | ||
| } | ||
|
|
||
| bitmap.Save(appSettingsProvider.AppSettings.SavePath, System.Drawing.Imaging.ImageFormat.Png); |
There was a problem hiding this comment.
Вот этот метод не выглядит гибким, на мой взгляд, svg сложно будет в текущий код добавить
There was a problem hiding this comment.
Выделил в отдельный класс для сохранения
No description provided.