Method QuestionAndAnswerAsync has a Cognitive Complexity of 117 (exceeds 20 allowed). Consider refactoring.

Open

        public async Task<string> QuestionAndAnswerAsync(string answer, string userId)
        {
            // method to get user's details, user's accesstoken, user's task mail details and list or else appropriate message will be send
            var userAndTaskMailDetailsWithAccessToken = await GetUserAndTaskMailDetailsAsync(userId);
            _logger.Info("Task Mail Bot Module, Is task mail question text : " + userAndTaskMailDetailsWithAccessToken.QuestionText);

• Read up
  Read up

Method QuestionAndAnswerAsync has 201 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public async Task<string> QuestionAndAnswerAsync(string answer, string userId)
        {
            // method to get user's details, user's accesstoken, user's task mail details and list or else appropriate message will be send
            var userAndTaskMailDetailsWithAccessToken = await GetUserAndTaskMailDetailsAsync(userId);
            _logger.Info("Task Mail Bot Module, Is task mail question text : " + userAndTaskMailDetailsWithAccessToken.QuestionText);

File TaskMailRepository.cs has 388 lines of code (exceeds 250 allowed). Consider refactoring.

Open

using Promact.Core.Repository.OauthCallsRepository;
using Promact.Erp.DomainModel.Models;
using System;
using System.Threading.Tasks;
using System.Linq;

Method StartTaskMailAsync has 47 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public async Task<string> StartTaskMailAsync(string userId)
        {
            // method to get user's details, user's accesstoken, user's task mail details and list or else appropriate message will be send
            var userAndTaskMailDetailsWithAccessToken = await GetUserAndTaskMailDetailsAsync(userId);
            _logger.Info("Task Mail Bot Module, Is task mail question text : " + userAndTaskMailDetailsWithAccessToken.QuestionText);

Method GetUserAndTaskMailDetailsAsync has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        private async Task<UserAndTaskMailDetailsWithAccessToken> GetUserAndTaskMailDetailsAsync(string slackUserId)
        {
            UserAndTaskMailDetailsWithAccessToken userAndTaskMailDetailsWithAccessToken = new UserAndTaskMailDetailsWithAccessToken();
            var user = await _userRepository.FirstOrDefaultAsync(x => x.SlackUserId == slackUserId);
            if (user != null)

Avoid deeply nested control flow statements.

Open

                                    if (answer != null)
                                    {
                                        taskDetails.Description = answer;
                                        userAndTaskMailDetailsWithAccessToken.QuestionText = nextQuestion.QuestionStatement;
                                        taskDetails.QuestionId = nextQuestion.Id;

Avoid deeply nested control flow statements.

Open

                                    if (sendEmailConfirmationConvertResult)
                                    {
                                        taskDetails.SendEmailConfirmation = confirmation;
                                        userAndTaskMailDetailsWithAccessToken.QuestionText = _stringConstant.ThankYou;
                                        switch (confirmation)

Avoid deeply nested control flow statements.

Open

                                    if (taskMailHourConvertResult)
                                    {
                                        decimal totalHourSpented = 0;
                                        // checking range of hours
                                        if (hour > 0 && hour <= Convert.ToInt32(_stringConstant.TaskMailMaximumTime))

Avoid deeply nested control flow statements.

Open

                                    if (taskMailStatusConvertResult)
                                    {
                                        // if previous question was status of task and it was not null/wrong value then answer will ask next question
                                        taskDetails.Status = status;
                                        userAndTaskMailDetailsWithAccessToken.QuestionText = nextQuestion.QuestionStatement;

Avoid deeply nested control flow statements.

Open

                                    if (sendEmailConfirmationConvertResult)
                                    {
                                        taskDetails.SendEmailConfirmation = confirmation;
                                        switch (confirmation)
                                        {

Avoid deeply nested control flow statements.

Open

                                    if (answer != null)
                                    {
                                        // if previous question was comment of task and answer was not null/wrong value then answer will ask next question
                                        taskDetails.Comment = answer;
                                        userAndTaskMailDetailsWithAccessToken.QuestionText = nextQuestion.QuestionStatement;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            case QuestionOrder.YourTask:
                                {
                                    // if previous question was description of task and it was not null/wrong vale then answer will ask next question
                                    if (answer != null)
                                    {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            case QuestionOrder.Comment:
                                {
                                    if (answer != null)
                                    {
                                        // if previous question was comment of task and answer was not null/wrong value then answer will ask next question

• Read up
  Read up