PIbd-14 Pruidze I.K Cruiser_Simple #1

Closed
Delphy wants to merge 3 commits from Lab1 into start
Owner
No description provided.
Delphy added 3 commits 2024-03-03 17:45:03 +04:00
eegov approved these changes 2024-03-16 17:45:42 +04:00
eegov left a comment
Owner

Проверено

Проверено
@ -0,0 +21,4 @@
public bool Deckhouse { get; private set; } /// салон на верхней палубе
public double Step => Speed * 100 / Weight;
public int[] values = { 0, 0, 0 };
Owner

Публичные поля - нарушение инкапсуляции

Публичные поля - нарушение инкапсуляции
@ -0,0 +25,4 @@
public void Init(int speed, double weight,
Color mainc, Color addtc, bool pad,
bool hangar, bool deckhouse, int p, int d_coordinates, int h)
Owner

Что за p, d_coordinates, h?? По заданию не требовалось, раз
Переменные должны иметь логические название, два

Что за p, d_coordinates, h?? По заданию не требовалось, раз Переменные должны иметь логические название, два
@ -0,0 +128,4 @@
#endregion
private PictureBox pictureBox1;
Owner

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

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +20,4 @@
InitializeComponent();
}
//private DrawingCanvas_Load(object sender, EventArgs e);
Owner

Закомментированного кода быть не должно

Закомментированного кода быть не должно
@ -0,0 +28,4 @@
private int? _startPosY;
public void Init(Class_CruiserEntity CruiserEntity)
Owner

Дополнительное задание в итоговый код не включается

Дополнительное задание в итоговый код не включается
eegov closed this pull request 2024-03-16 17:45: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: Delphy/PIbd-14_Pruidze_I.K_Cruiser_Simple#1
No description provided.