Савенко Дмитрий #25
Савенко Дмитрий #25
Conversation
| private readonly ITagCloudLayouter _layouter; | ||
| private readonly ITagCloudRenderer _renderer; | ||
|
|
||
| public TagCloudGenerator(IWordProcessor wordProcessor, ITagCloudLayouter layouter, ITagCloudRenderer renderer) |
There was a problem hiding this comment.
Да давай воспользуемся PrimaryCocstructor'ом, чтобы этот код каждый раз не править: я сейчас накину идей на подключение допзависимостей - сейчас это придется править в 3 местах, а так будет всего в одном.
|
|
||
| public void GenerateCloud(string inputFilePath, string outputFilePath, Size imageSize) | ||
| { | ||
| var words = File.ReadAllLines(inputFilePath); |
There was a problem hiding this comment.
Давай эту штуку в отдельный интерфейс, типа ITextReadear: таким образом мы уже закладываем точку расширения на Исходный текст: поддерживать разные форматы файлов.
| var tags = wordFrequencies | ||
| .OrderByDescending(pair => pair.Value) | ||
| .Select(pair => new Tag(pair.Key, pair.Value)) | ||
| .ToList(); |
There was a problem hiding this comment.
Вот этот подсчёт статистики, как мне кажется, тоже не обязанность этого класса - т.е. может эту обязанность переложить на какой-то отдельный интерфейс?
|
|
||
| foreach (var tag in tags) | ||
| { | ||
| var size = new Size(tag.Frequency * 10, tag.Frequency * 10); |
There was a problem hiding this comment.
Почему на 10 умножаем?)
Как-то это надо пояснить - либо в отдельный метод, либо 10 в константу с понятным именем.
|
|
||
| public interface ITagCloudGenerator | ||
| { | ||
| void GenerateCloud(string inputFilePath, string outputFilePath, Size imageSize); |
There was a problem hiding this comment.
Должна быть возможность задать цвета, шрифт и размер изображения.
Это по заданию обязательный пункт - пока я вижу поддержку размера изображения. Хотелось бы хотя бы простенькую поддержку цветов (задний фон, фон шрифта, массив наименований шрифтов - если не задано, то какие-то дефолтные значения).
@SlavikGh0st