Шестопалов Андрей #33
Conversation
| <Project Sdk="Microsoft.NET.Sdk"> | ||
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>net48</TargetFramework> |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| { | ||
| public static string GetFileNameFromUser() | ||
| { | ||
| while (true) |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
TagsCloudContainer/App.cs
Outdated
| { | ||
| while (true) | ||
| { | ||
| Console.Write("Введите размер изображения (ширина высота): "); |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
TagsCloudContainer/App.cs
Outdated
| public static string GetFontNameFromUser() | ||
| { | ||
| Console.Write("Введите название шрифта: "); | ||
| return Console.ReadLine() ?? "Arial"; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| { | ||
| public static bool FontExists(this string fontName) | ||
| { | ||
| return FontFamily.Families.Any(f => f.Name.Equals(fontName, StringComparison.OrdinalIgnoreCase)); |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
TagsCloudContainer/App.cs
Outdated
| return Console.ReadLine() ?? "Arial"; | ||
| } | ||
|
|
||
| public static Color GetColorFromUser(string prompt) |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
TagsCloudContainer/Program.cs
Outdated
| { | ||
| public static void Main() | ||
| { | ||
| var fileName = App.GetFileNameFromUser(); |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
|
||
| public class Program | ||
| { | ||
| public static void Main() |
There was a problem hiding this comment.
Давай инициализацию контейнера вынесем в отдельный метод, а лучше класс, а тут оставим только запуск облака тэгов
|
|
||
| namespace TagsCloudContainer; | ||
|
|
||
| public class CircularCloudLayouter |
There was a problem hiding this comment.
Не нравится что этот класс отвечает и за генерацию последовательности прямоугольников, и за размещение текста по ним. Кажется это две разные ответственности, которые нужно разделить.
TagsCloudContainer/FileParser.cs
Outdated
| { | ||
| public static Dictionary<string, int> Parse(string wordsPath) | ||
| { | ||
| var excludedWordsPath = Path.Combine("..", "..", "..", "excluded_words.txt"); |
There was a problem hiding this comment.
Что если пользователь захочет дополнительно какие-то слова исключать, передавая их при запуске. Давай сделаем какой-нибудь класс с интерфейсом, суть которого получить все слова, которые надо исключить
No description provided.