Reuploading fixed qoutation #286
No Assignees
protuberanets
protuberanets
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: tech/sendico#286
Reference in New Issue
Block a user
Delete Branch "SEND031"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
есть мелкие замечания
@@ -0,0 +96,4 @@if (name != null && name.isNotEmpty) {final parts = name.split(RegExp(r'\s+'));if (parts.length == 1) {вне зависимости от условия всегда выполняется "firstName = parts.first;". Чего условие тогда так составлено? Можно просто всегда присвоение делать, а в зависимости от некоторых доп условий делать остальное. Перерисуй код так, чтобы в нем не было повторов. Одно присвоение - строго один раз.
снова неоптимально. Обрати внимание, попросил перерисовать так, чтобы каждое присвоение было строго один раз. Условия можно перестроить так, чтобы флоу кода был более естественный и без повторных проверок того, что ты уже знаешь
@@ -0,0 +54,4 @@);if (intent == null) return;final lastIntentDto = _lastIntent?.toDTO().toJson();final isSameIntent = lastIntentDto != null &&зачем так сложно? у котировки должен быть ключ идемпотентности, нет смысла глубоко так сравнивать
@@ -0,0 +86,4 @@return timeLeft <= Duration.zero;}QuoteStatusType get quoteStatusType {именование: ты возвращаешь смысл, а не название типа, поэтому должно быть "QuoteStatusType get quoteStatus", а не "QuoteStatusType get quoteStatusType"
@@ -0,0 +140,4 @@_logger.warning('Failed to get quotation', e, st);_setResource(_quotation.copyWith(data: null,error: e is Exception ? e : Exception(e.toString()),где-то должна быть функция toException, или что-то такое, на нее лучше опираться, а не повторять код
@@ -0,0 +155,4 @@_setResource(Resource(data: null, isLoading: false, error: null));}void _syncAutoRefresh() {признак плохого дизайна: появление ручных синхронизаций - всегда проблема.
@@ -0,0 +1,11 @@String formatQuoteDuration(Duration duration) {Предпочтительно использовать готовые пакеты форматирования: https://pub.dev/packages/duration
Pull request closed