Agliullov D. A. Lab work 1 Base #2

Closed
d.agliullov wants to merge 14 commits from Lab1_Base into main
Owner
No description provided.
d.agliullov added 13 commits 2023-02-09 16:01:43 +04:00
d.agliullov requested review from eegov 2023-02-09 16:03:54 +04:00
d.agliullov added 1 commit 2023-02-09 16:19:18 +04:00
eegov approved these changes 2023-02-11 18:18:18 +04:00
eegov left a comment
Collaborator

Зачтено с понижением балла(ов)

Зачтено с понижением балла(ов)
@ -0,0 +97,4 @@
nameof(vmodel));
}
model.Status = orderStatus;
model.DateCreate = vmodel.DateCreate;
Collaborator

Требуется сменить только статус, зачем все поля передавать?

Требуется сменить только статус, зачем все поля передавать?
@ -0,0 +44,4 @@
{
return;
}
PastryId = model.PastryId;
Collaborator

Не требуется обновлять все данные, только статус

Не требуется обновлять все данные, только статус
@ -0,0 +34,4 @@
_logger.LogInformation("Загрузка изделий для заказа");
foreach (var el in _logicP.ReadList(null) ?? new())
{
comboBoxPastry.Items.Add(el.PastryName);
Collaborator

У comboBoxPastry есть DataSource

У comboBoxPastry есть DataSource
@ -0,0 +173,4 @@
private DataGridView dataGridView;
private Button buttonCreateOrder;
private Button buttonTakeOrderInWork;
private Button button2;
Collaborator

У элементов форм, с которыми работаем в логике должны быть логичные имена

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +174,4 @@
private Button buttonCreateOrder;
private Button buttonTakeOrderInWork;
private Button button2;
private Button button3;
Collaborator

У элементов форм, с которыми работаем в логике должны быть логичные имена

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +175,4 @@
private Button buttonTakeOrderInWork;
private Button button2;
private Button button3;
private Button button4;
Collaborator

У элементов форм, с которыми работаем в логике должны быть логичные имена

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +107,4 @@
var operationResult = _orderLogic.FinishOrder(new OrderBindingModel
{
Id = id,
Status = orderStatus
Collaborator

А зачем зедсь статус передавать, он должен в логике определеяться

А зачем зедсь статус передавать, он должен в логике определеяться
eegov closed this pull request 2023-02-11 18:18:46 +04:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: d.agliullov/ISEbd-21_Agliullov.D.A._Confectionery#2
No description provided.