PIbd-23 Firsov Kirill LabWork 01 #4

Closed
Firsov_Kirill wants to merge 7 commits from Laba1 into main
No description provided.
Firsov_Kirill added 2 commits 2023-10-13 09:17:19 +04:00
eegov approved these changes 2023-10-15 18:17:33 +04:00
eegov left a comment
Owner

Имеются конфликты пула

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

Имеются конфликты пула Зачтено с понижением балла(ов)
@ -0,0 +1,29 @@
using System;
Owner

Название проекта не соответствует указаному в задании

Название проекта не соответствует указаному в задании
@ -0,0 +54,4 @@
_pictureWidth = width;
_pictureHeight = height;
if (_tractorWidth > _pictureHeight || _tractorHeight > _pictureHeight)
Owner

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

Сначала выполняются проверки и только потом, если они пройдены успешно, запоминаются данные
@ -0,0 +56,4 @@
_pictureHeight = height;
if (_tractorWidth > _pictureHeight || _tractorHeight > _pictureHeight)
{
_pictureWidth = 2 * _tractorWidth;
Owner

Здесь нет корректировок, только возврат false, если в границы не вписываемся

Здесь нет корректировок, только возврат false, если в границы не вписываемся
@ -0,0 +74,4 @@
_startPosX = x;
_startPosY = y;
if ((x + _tractorWidth > _pictureWidth) || (y + _pictureHeight > _pictureHeight))
Owner

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

Не учтены все условия, при которых объект может выйти за границы
@ -0,0 +77,4 @@
if ((x + _tractorWidth > _pictureWidth) || (y + _pictureHeight > _pictureHeight))
{
Random random = new();
x = random.Next(0, _pictureWidth);
Owner

Ну изменили x и что дальше? _startPosX хранит старое значение то

Ну изменили x и что дальше? _startPosX хранит старое значение то
@ -0,0 +78,4 @@
{
Random random = new();
x = random.Next(0, _pictureWidth);
x = random.Next(0, _pictureHeight);
Owner

Дважды меняем x

Дважды меняем x
@ -0,0 +6,4 @@
namespace RPP_FirstLaba_Tractor
{
public class EntityTractor
Owner

Не заданы свойства-признаки

Не заданы свойства-признаки
eegov closed this pull request 2023-10-15 18:17: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: Firsov_Kirill/PIbd-23_Firsov_Kirill_Tractor_basis#4
No description provided.