Uh oh!
There was an error while loading. Please reload this page.
add simple database UI - #43
Conversation
| @code { | ||
| private string SetName { get; set; } = String.Empty; | ||
| private ActivityInfo Info { get; set; } | ||
| private string _setName { get; set; } = String.Empty; |
There was a problem hiding this comment.
Оч странно что у тебя название поля - глагол,
Как я понял из логики, было бы неплохо назвать _selectedName
| private List<string> _usernames { get; set; } = new List<string>(); | ||
| private StudyGroupResponse _studyGroup; | ||
| private bool StatsIsVisible { get; set; } = false; | ||
| private bool _statsIsVisible { get; set; } = false; |
| private string _username { get; set; } = String.Empty; | ||
| private string _groupName { get; set; } = String.Empty; | ||
| private List<string> _usernames { get; set; } = new List<string>(); | ||
| private StudyGroupResponse _studyGroup; | ||
| private bool StatsIsVisible { get; set; } = false; | ||
| private bool _statsIsVisible { get; set; } = false; |
| @@ -9,12 +10,22 @@ namespace Kysect.GithubActivityAnalyzer.WebDemo.Server.Controllers | |||
| [Route("[controller]")] | |||
SonarCloud Quality Gate failed. |
FrediKats
commented
May 6, 2021
@ALEXXXANDRO ты очень плохой человек! Я говорил, что не стоит в один пр лить добавления и переименования. Сейчас половина пул реквеста - это ренеймы. И нужно внимательно вчитываться, чтобы понять, где добавленная логика, а где ренейм и нужно скипать. |
ALEXXXANDRO
commented
May 6, 2021
@FrediKats Может я не так что то понял, но я у сергея эсхучаковича консультировался в тот раз, показывал ветки. Он сказал, что наименее проблемным будет запихать в одну. |
| public List<(string Username, ActivityInfo Activity)> GetActivityFromUserCash(IEnumerable<string> usernames, bool isParallel) | ||
| { | ||
| if (!isParallel) | ||
| { | ||
| return usernames | ||
| .Select(username => (username, GetActivityFromUserCash(FindByUsername(username)))) | ||
| .ToList(); | ||
| } | ||
| List<(string, ActivityInfo)> result = usernames | ||
| .AsParallel() | ||
| .Select(username => (username, GetActivityFromUserCash(FindByUsername(username)))) | ||
| .ToList(); | ||
| return result; | ||
| } | ||
| } |
There was a problem hiding this comment.
Параллельно делать запросы в базу вместо того чтобы сделать один - это странно. Мне кажется, что это все можно переписать так, чтобы выполнялся один SQL запрос с использованием Join. Но я думаю, что на это можно отдельную таску завести, это не критично.
Добавлена возможность начать взаимодействовать с DB из UI (добавил DI, поменял конфигурацию и т.д)
В качестве демонстрации на текущий момент есть возможность доставать TotalActivity двумя способами - из DB и из гитхаба(страница TestPage).