Калентьев Илья tg: @m1nus0ne #22
Conversation
| [TestCaseSource(typeof(ColorConverterTestData), nameof(ColorConverterTestData.RightCases))] | ||
| public Color Converter_ShouldConvertStringToColor_WhenStringInRightFormat(string hexString) | ||
| { | ||
| if (ColorConverter.TryConvert(hexString, out var color)) |
There was a problem hiding this comment.
Можно было проверить возвращаемый bool, а не кидать пустую ошибку
|
|
||
| public class ColorConverterTestData | ||
| { | ||
| public static IEnumerable<TestCaseData> RightCases |
There was a problem hiding this comment.
такие данные удобнее хранить в классе теста
| } | ||
|
|
||
| [Test] | ||
| public void DrawOneTundredRectangles() |
| if (TagCloudServicesFactory.ConfigureServiceAndTryGet<TagCloudGenerator>(option, out var generator)) | ||
| generator.Generate(); | ||
| else | ||
| throw new Exception("Can't configure service"); |
There was a problem hiding this comment.
Если введены неверные аргументы, лучше вывести об этом сообщение и завершить исполнение с не нулевым кодом, а не кидать Exception
There was a problem hiding this comment.
Результат генерации получился странным. Крупные слова разбросаны по всему облаку, а так быть не должно
|
|
||
| private void SaveToFile(Bitmap bitmap) | ||
| { | ||
| var pathToFile = @$"{path}\{name}"; |
There was a problem hiding this comment.
Сейчас это не так важно, потому, что код запуститься только на windows из-за зависимостей, но на будущее, для обьединения путей стоит использовать Path.Combine или хотя бы Path.PathSeparator
| var rgbValue = hexString | ||
| .Replace("#", "") | ||
| .Chunk(2) | ||
| .Select(chars => Convert.ToInt32(new string(chars), 16)) | ||
| .ToArray(); |
There was a problem hiding this comment.
rgb коды могут задаваться 3 hex цифрами
| [Option('d', "destination", HelpText = "Set destination path.", Default = @"..\..\..\Results")] | ||
| public string DestinationPath { get; set; } | ||
|
|
||
| [Option('s', "source", HelpText = "Set source path.", Default = @"..\..\..\Results\text.txt")] |
There was a problem hiding this comment.
Попробуй посмотреть описание любой часто используемой консольной программы, сдеалнное тобой описание не информативно
| if (options.ColorScheme == "gray") | ||
| builder.RegisterType<RandomColorSelector>().As<IColorSelector>().SingleInstance(); | ||
| if (options.ColorScheme == "random") | ||
| builder.RegisterType<GrayScaleColorSelector>().As<IColorSelector>().SingleInstance(); | ||
| else if (ColorConverter.TryConvert(options.ColorScheme, out var color)) | ||
| builder.Register(provider => new ConstantColorSelector(color)).As<IColorSelector>().SingleInstance(); | ||
| else | ||
| builder.Register(provider => new ConstantColorSelector(Color.Black)).As<IColorSelector>().SingleInstance(); |
There was a problem hiding this comment.
эти условия можно заменить полиморфизмом
| @@ -0,0 +1,7 @@ | |||
| namespace TagCloud.TextHandlers; | |||
|
|
|||
| public class TextData | |||
There was a problem hiding this comment.
Этот класс не стоило вводить, можно заменить его на ValueTuple
No description provided.