ПИбд-21, Лёвушкина А.А., лаб 1 #1

Closed
AnnaLioness wants to merge 2 commits from containerShip_base into main
Owner
No description provided.
AnnaLioness added 2 commits 2023-09-15 14:23:37 +04:00
eegov approved these changes 2023-09-23 09:08:29 +04:00
eegov left a comment
Collaborator

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

Зачтено с понижением балла(ов)
@ -0,0 +44,4 @@
// TODO: Продумать проверки
_pictureWidth = width;
_pictureHeight = height;
if (_pictureWidth < _shipWidth || _pictureHeight < _shipHeight)
Collaborator

Сначала выполняются проверки и только потом, если они пройдены успешно, запоминаются данные

Сначала выполняются проверки и только потом, если они пройдены успешно, запоминаются данные
@ -0,0 +54,4 @@
return true;
}
/*public bool Init(EntityContainerShip entcon, int wid, int hei)
Collaborator

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

Закомментированного кода быть не должно
@ -0,0 +71,4 @@
public void SetPosition(int x, int y)
{
// TODO: Изменение x, y
_startPosX = Math.Min(x, _pictureWidth - _shipWidth);
Collaborator

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

Не учтены все условия, при которых объект может выйти за границы
@ -0,0 +128,4 @@
#endregion
private System.Windows.Forms.PictureBox pictureBox1;
Collaborator

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

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +10,4 @@
namespace Lab1ContainersShip
{
public partial class Form1 : Form
Collaborator

Имя класса не соответствует указанному в задании

Имя класса не соответствует указанному в задании
@ -0,0 +48,4 @@
Random random = new Random();
_drawningShip = new DrawingContainerShip();
EntityContainerShip _entitycont = new EntityContainerShip();
/*_entitycont.Init(random.Next(100, 300),
Collaborator

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

Закомментированного кода быть не должно
eegov closed this pull request 2023-09-23 09:08:42 +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: AnnaLioness/PIbd21.LyovushkinaA.A.Container_ship.Base#1
No description provided.