Egor_Petrushin/PIbd-22_Petrushin_E.A._SelfPropelledArtilleryUnit_Base
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

PIbd-22_Petrushin_E.A._Lab7 #12

Closed
Egor_Petrushin wants to merge 3 commits from PIbd-22_Petrushin_E.A._Lab7 into PIbd-22_Petrushin_E.A._Lab6
pull from: PIbd-22_Petrushin_E.A._Lab7
merge into: Egor_Petrushin:PIbd-22_Petrushin_E.A._Lab6
Conversation 0 Commits 3 Files Changed 10 +190 -51
Egor_Petrushin commented 2023-12-06 09:43:30 +04:00
Owner
Copy Link
No description provided.
Egor_Petrushin added 3 commits 2023-12-06 09:43:31 +04:00
eegov approved these changes 2023-12-10 21:12:44 +04:00
eegov left a comment
Collaborator
Copy Link

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

Зачтено с понижением балла(ов)
SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit/Program.cs
@ -15,0 +27,4 @@
{
string currentTime = DateTime.Now.ToString("yyyy-MM-dd_HH-mm-ss");
string logFileName = $"logs/spaulog-{currentTime}.txt";
services.AddSingleton<FormSPAUCollection>().AddLogging(option =>
eegov commented 2023-12-10 21:11:04 +04:00
Collaborator
Copy Link

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

Настройку логера следует выносить в отдельный конфигурационный файл, чтобы ее можно было менять без пересборки проекта
SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit/ReadFileExeption.cs
@ -0,0 +8,4 @@
namespace SelfPropelledArtilleryUnit
{
[Serializable]
internal class ReadFileExeption : ApplicationException
eegov commented 2023-12-10 21:12:08 +04:00
Collaborator
Copy Link

Не требовалось делать

Не требовалось делать
SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit/SPAUGenericStorage.cs
@ -117,3 +117,3 @@
if (data.Length == 0)
{
return false;
throw new ReadFileExeption("Невалиданя операция, нет данных для сохранения");
eegov commented 2023-12-10 21:12:36 +04:00
Collaborator
Copy Link

Требовалось заменить класс Exception на его более подходящих наследников (стандартных, а не своих)

Требовалось заменить класс Exception на его более подходящих наследников (стандартных, а не своих)
SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit/SPAUNotFoundException.cs
@ -0,0 +8,4 @@
namespace SelfPropelledArtilleryUnit
{
[Serializable]
internal class SPAUNotFoundException : ApplicationException
eegov commented 2023-12-10 21:11:52 +04:00
Collaborator
Copy Link

Не требовалось делать

Не требовалось делать
eegov closed this pull request 2023-12-10 21:12:49 +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: Egor_Petrushin/PIbd-22_Petrushin_E.A._SelfPropelledArtilleryUnit_Base#12
No description provided.
Delete Branch "PIbd-22_Petrushin_E.A._Lab7"

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?