Яценко Ирина #174
Яценко Ирина #174
Conversation
| { | ||
| public class Result<T> | ||
| { |
There was a problem hiding this comment.
Чаще всего используют вот эту реализацию https://github.com/kontur-courses/fp/blob/master/ErrorHandling/Result.cs при выполнении этого задания. У тебя немного она отличается. Она заставляет тебя больше писать try catch. В целом я больше за твой вариант, т.к похожий я использую у себя в команде
| public Result(T value, string error) | ||
| { | ||
| Value = value; | ||
| Error = error; | ||
| } | ||
|
|
||
| public static Result<T> Success(T value) => new Result<T>(value, null); | ||
| public static Result<T> Failure(string error) => new Result<T>(default, error); |
There was a problem hiding this comment.
Добавить конструктор по умолчанию добавили, но использовать не стала, а надобы
Я бы основной конструктор сделал бы private, т.к есть статичные методы которые выступают гарантией, что его вызывать нужно только вот так
| namespace TagsCloudVisualization | ||
| { | ||
| public class CircularCloudLayouter : ICircularCloudLayouter |
There was a problem hiding this comment.
Не вижу использование в этом проекте патерна, да и ошибки вот прямо в классе выкидываются
| if (cloud.Rectangles.Count == 0) | ||
| return AddToCenterPosition(rectangleSize); |
There was a problem hiding this comment.
Немного странный метод, т.к. первая точка которую ты возвращаешь это всегда центр
|
|
||
|
|
|
|
||
| public Result<Bitmap> DrawWordsCloud(string filePath, VisualizingSettings visualizingSettings) | ||
| { | ||
| if (filePath == null) |
There was a problem hiding this comment.
не уверен что все что в происходит в методе безопасно и не выкинет ошибку
@Pasha0666