Пибд-23 Тихоненков Алексей Лабораторная №1 #2

Closed
YourDax wants to merge 6 commits from Lab1 into main
Owner
No description provided.
YourDax added 1 commit 2023-10-11 03:15:38 +04:00
YourDax added 1 commit 2023-10-11 03:26:02 +04:00
YourDax requested review from eegov 2023-10-11 03:26:42 +04:00
YourDax added 1 commit 2023-10-11 03:30:12 +04:00
YourDax changed title from Пибд-23 Тихоненков Алексей Лабораторная №1 to WIP: Пибд-23 Тихоненков Алексей Лабораторная №1 2023-10-11 13:27:16 +04:00
YourDax changed title from WIP: Пибд-23 Тихоненков Алексей Лабораторная №1 to Пибд-23 Тихоненков Алексей Лабораторная №1 2023-10-11 13:27:48 +04:00
YourDax removed review request for eegov 2023-10-12 20:49:15 +04:00
YourDax added 2 commits 2023-10-12 20:52:21 +04:00
YourDax requested review from eegov 2023-10-12 20:55:22 +04:00
eegov approved these changes 2023-10-15 18:10:38 +04:00
eegov left a comment
Owner

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

Зачтено с понижением балла(ов)
@ -0,0 +72,4 @@
if (AntiAircraftGun == null) return;
_startPosX = x;
_startPosY = y;
if (x + _zenitWidth >= _pictureWidth || y + _zenitHeight>= _pictureHeight)
Owner

Не учтены все условия, при которых объект может выйти за границы

Не учтены все условия, при которых объект может выйти за границы
@ -0,0 +106,4 @@
break;
// вправо
case DirectionType.Right:
if (_startPosX + AntiAircraftGun.Step < _pictureWidth-110)
Owner

Что за 110?

Что за 110?
@ -0,0 +5,4 @@
/// <summary>
/// Поле-объект для прорисовки объекта
/// </summary>
private DrawingAntiAirCraftGun? _drawing;
Owner

Класс DrawingAntiAirCraftGun, а объект _drawing?

Класс DrawingAntiAirCraftGun, а объект _drawing?
@ -0,0 +10,4 @@
{
InitializeComponent();
}
private void Move_Click(object sender, EventArgs e)
Owner

Не используемый метод

Не используемый метод
@ -0,0 +39,4 @@
private void pictureBoxZenit_Click(object sender, EventArgs e)
Owner

Пустых методов быть не должно

Пустых методов быть не должно
eegov closed this pull request 2023-10-15 18:10:49 +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: YourDax/PIbd-23_Tikhonenkov_A.E_AntiAircraftGun#2
No description provided.