Conversation
…й poinGenerator, добавил возможность его использования
| private const int minContentWordLength = 4; | ||
| private const int minWordsCount = 2; | ||
|
|
||
| public Dictionary<string, int> Filter(Dictionary<string, int> rawCounts, int totalWords) |
There was a problem hiding this comment.
Добавь возможность отфильтровать слова по "черному списку". Давай черный список не хардкодить, а читать из файла
| if (string.IsNullOrWhiteSpace(name)) | ||
| return spiral; | ||
|
|
||
| return name.ToLowerInvariant() switch |
There was a problem hiding this comment.
Нехорошо. Избавься от свича и завязки на конкретные строки - код становится хрупким. "Запрещено использовать операторы if, switch, ?: и прочие условные операторы, если их можно заменить полиморфизмом."
|
|
||
| public static class Program | ||
| { | ||
| public static void Main(string[] args) |
There was a problem hiding this comment.
Нет общего интерфейса для "запуска генерации", нет слоя, изолированного от ввода-вывода. "Организуй код так, чтобы было легко сделать оба вида клиентов — и клиента командной строки, и GUI приложение с вводом параметров и интерактивным просмотром"
…ном файле, также при запуске можно указать дополнительный файл с черным списком
… убрал swich в PointGeneratorFactory
|
|
||
| foreach (var ch in trimmed) | ||
| { | ||
| if (!char.IsLetter(ch)) |
There was a problem hiding this comment.
Как быть со словами типа "can't", "co-owner" и т.д.?
2 participants
No description provided.