Зубакин Ренат #170
Зубакин Ренат #170
Conversation
| public Result<Color> GetTagColor(Tag tag) | ||
| { | ||
| var color =Color.FromArgb(Random.Next(256), Random.Next(256), Random.Next(256)); | ||
| return Result.Result.Ok(color); |
There was a problem hiding this comment.
Неудобно так писать, да?
Порешать проблему может implicit operator в резалте, тогда шарп сам будет оборачивать в резалт
TagsCloud/App/App.cs
Outdated
| layouter.CreateTagCloud(wordsProvider.GetWords().GetValueOrThrow()); | ||
| painter.DrawCloud(layouter.GetTagsCollection().GetValueOrThrow(),layouter.GetImageSize().GetValueOrThrow()); |
There was a problem hiding this comment.
Посмотри как тут устроен метод PrepareFileToSend вот хотелось бы что-то типа такого увидеть
| public Result<Dictionary<string, int>> GetWords() | ||
| { | ||
| if (!File.Exists(filename)) | ||
| throw new FileNotFoundException(filename); |
There was a problem hiding this comment.
Тогда при подходе который я вышел показал, здесь можно будет не бросать ошибку, а вернуть result с ошибкой, которая выведется на консоль и пользователю будет понятно, что пошло не так
TagsCloud/App/App.cs
Outdated
| if (!result.IsSuccess) | ||
| { | ||
| Console.WriteLine(result.Error); | ||
| } |
There was a problem hiding this comment.
Можно написать еще один экстеншн метод в резалт, который при наличии ошибки будет ее печатать
There was a problem hiding this comment.
точнее даже не так, передавать в метод действие, которое необходимо сделать с ошибкой, если она есть
| public Result<Color> GetTagColor(Tag tag) | ||
| { | ||
| var color =Color.FromArgb(Random.Next(256), Random.Next(256), Random.Next(256)); | ||
| return Result.Ok(color); |
There was a problem hiding this comment.
ты же добавил implicit operator, можно просто
return color;
TagsCloud/Program.cs
Outdated
| var options = CommandLine.Parser.Default.ParseArguments<Options>(args).Value; | ||
| var container = ContainerConfig.Configure(options); |
There was a problem hiding this comment.
А если с аргументами что-то не так?
| using (Graphics g = Graphics.FromImage(new Bitmap(1, 1))) | ||
| { | ||
| var sizeF = g.MeasureString(tag.Key, tag.Value); | ||
| rectangle.Size = new Size((int)Math.Ceiling(sizeF.Width), (int)Math.Ceiling(sizeF.Height)); |
There was a problem hiding this comment.
Как будто отдельный метод напрашивается... Возможно даже extension
@69raccoon96