Quizlet - CQFDKS #15
Quizlet - CQFDKS #15
Conversation
| { | ||
| if (!questionDAO.QuestionIsExist(question)) | ||
| { | ||
| if (question.AnswerIndex >= 0 && question.AnswerIndex <= 4) |
There was a problem hiding this comment.
A 4 egy magic konstans
| } | ||
| else | ||
| { | ||
| AnsiConsole.Write(new Markup($"[bold red]\n Error: The good answer index is lower then 0 or higher then 4!\n\n[/]")); |
| { | ||
| public class QuestionController | ||
| { | ||
| private IQuestionDAO questionDAO = new QuestionDAO(); |
There was a problem hiding this comment.
Dependency Inversion?
| { | ||
| public class ScoreController | ||
| { | ||
| private IScoreDAO scoreDAO = new ScoreDAO(); |
There was a problem hiding this comment.
Szintén lehetne dependency inversion
| { | ||
| try | ||
| { | ||
| var count = scoreDAO.GetScores().Where(x => x.Email == score.Email && x.Category == score.Category).Count(); |
There was a problem hiding this comment.
Dupla linq szűrés ugyan arra az adatra? FirstOrDefault() Null-t ad vissza ha nincs és akkor a try-catch se kell, meg a Count() alapú számolás. A (Count > 0) amúgy is helyettesíthető lenne egy Any() hívással.
|
|
||
| public void EvaluateAnswer(List<Question> questions, List<string> answeredQuestions) | ||
| { | ||
| for (int i = 0; i < answeredQuestions.Count(); i++) |
There was a problem hiding this comment.
A Count() minden esetben egy foreach-el a háttérben számol. Az olyan kollekciók, mint a Lista rendelkezik egy Count property-vel, amit érdemes lenne használni.
| subText.DisplayText("Main Menu"); | ||
| var choice1 = AnsiConsole.Prompt(new SelectionPrompt<string>().Title(" Choose a functionality!").PageSize(6).AddChoices(functionalities)); | ||
|
|
||
| switch (choice1) |
There was a problem hiding this comment.
Ez a blokk ismétlődik egy párszor. Erre egy metódus jó lenne.
| public static IdGenerator generator = new IdGenerator(0); | ||
|
|
||
| public static int GenerateID() | ||
| { |
There was a problem hiding this comment.
A CreateId() a IdGen-ben long-ot ad vissza, így ez túcsordul mint állat, főleg hogy ez idő alapon generál ahogy néztem a forráskódját. Vagyis az ID amit ezzel generálsz nem a legjobb megoldás. Ennél akár egy Random.Shared.Next() is jobb lenne. De felmerül a kérdés: A szimpla növekményes ID miért nem jó?
|
|
||
| namespace Quizlet.Utilities | ||
| { | ||
| public class MainTitle |
There was a problem hiding this comment.
Ez egy értelmetlen osztály, A FigletText-et akár magában is létrehozhatná string-ből, akkor több értelme lenne.
| { | ||
| private Rule Text { get; set; } | ||
|
|
||
| public SubTitle(Rule text) |
There was a problem hiding this comment.
Szintén, mint a FigletText-es osztály
No description provided.