chtzsch/Pibd-21_Ievlewa_M.D._SpeedBoat_Normal
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Иевлева_1ая лабораторная #1

Closed
chtzsch wants to merge 2 commits from lab_01 into main
pull from: lab_01
merge into: chtzsch:main
Conversation 0 Commits 2 Files Changed 18 +713 -79
chtzsch commented 2023-09-19 19:19:42 +04:00
Owner
Copy Link
No description provided.
chtzsch added 2 commits 2023-09-19 19:19:42 +04:00
eegov approved these changes 2023-10-15 14:11:47 +04:00
eegov left a comment
Collaborator
Copy Link

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

Зачтено с понижением балла(ов)
speed_Boat/speed_Boat/FormSpeedBoat.cs
@ -0,0 +15,4 @@
/// <summary>
/// Поле-объект для прорисовки объекта
/// </summary>
private SpeedBoatMovement? _tankerMovement;
eegov commented 2023-10-15 14:06:34 +04:00
Collaborator
Copy Link

Почему класс SpeedBoatMovement, а объект от класса _tankerMovement?

Почему класс SpeedBoatMovement, а объект от класса _tankerMovement?
speed_Boat/speed_Boat/SpeedBoat.cs
@ -0,0 +7,4 @@
namespace SpeedBoatLab
{
public class SpeedBoat
eegov commented 2023-10-15 14:09:20 +04:00
Collaborator
Copy Link

В названии класса должен присутствовать префикс Entity

В названии класса должен присутствовать префикс Entity
speed_Boat/speed_Boat/SpeedBoatMovement.cs
@ -0,0 +53,4 @@
/// <summary>
/// Проверка на вместимость обьекта в рамки сцены
/// </summary>
if ((widthBoat >= screenWidth) || (heightBoat >= screenHeight))
eegov commented 2023-10-15 14:10:46 +04:00
Collaborator
Copy Link

Сперва все запомнили, а потом проверка?

Сперва все запомнили, а потом проверка?
speed_Boat/speed_Boat/SpeedBoatMovement.cs
@ -0,0 +55,4 @@
/// </summary>
if ((widthBoat >= screenWidth) || (heightBoat >= screenHeight))
{
Console.WriteLine("проверка не пройдена, нельзя создать объект в этих размерах");
eegov commented 2023-10-15 14:10:11 +04:00
Collaborator
Copy Link

Это просто класс, неизвестно, где он будет использоваться, в консоль или куда-то еще ничего не пишется, максимум - в логи

Это просто класс, неизвестно, где он будет использоваться, в консоль или куда-то еще ничего не пишется, максимум - в логи
speed_Boat/speed_Boat/SpeedBoatMovement.cs
@ -0,0 +68,4 @@
/// </summary>
public void SetPosition(int x, int y)
{
if ((x + widthBoat > screenWidth) || (y + heightBoat > screenHeight))
eegov commented 2023-10-15 14:11:26 +04:00
Collaborator
Copy Link

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

Не учтены все условия, при которых объект может выйти за границы
eegov closed this pull request 2023-10-15 14:13:09 +04:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
eegov
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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: chtzsch/Pibd-21_Ievlewa_M.D._SpeedBoat_Normal#1
No description provided.
Delete Branch "lab_01"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?