Не все ветви кода возвращают значение

@alya770 · Регистрация: 10.12.2016

Author24 — интернет-сервис помощи студентам

Здравствуйте, у меня есть такой кусочек кода

C#

/*•   метод bool Remove(ResearchTeam rt) для удаления элемента со значением
         *  rt из словаря Dictionary<TKey, ResearchTeam>; 
         * если в словаре нет элемента rt, метод возвращает значение false; */
private static Dictionary<TKey, ResearchTeam> TKeyResearchTeam;
        public bool Remove(ResearchTeam rt)
        {
            ResearchTeam researcht = new ResearchTeam();
            Dictionary<TKey, ResearchTeam>.KeyCollection keyColl = TKeyResearchTeam.Keys;
            if (TKeyResearchTeam.ContainsValue(rt))
            {
                foreach(TKey t in keyColl)
                {
                    TKeyResearchTeam.TryGetValue(t, out researcht);
                    if(rt == researcht)
                    {
                       return TKeyResearchTeam.Remove(t);          
                    }
                    else
                    {
                        continue;
                    }
                }
            }
            else
            {
                return false;
            }
        }

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

@Bretbas · 07.10.2017, 15:45

alya770, Ошибка в том, что не все ветви возвращают значение

Тоесть есть места в коде, где нет return. К примеру, что если, при выполнении программы, будет всегда происходить так, что она будет заходить всегда в этот блок:

C#

            if (TKeyResearchTeam.ContainsValue(rt))
            {
                foreach(TKey t in keyColl)
                {
                    TKeyResearchTeam.TryGetValue(t, out researcht);
                    if(rt == researcht)
                    {
                       return TKeyResearchTeam.Remove(t);          
                    }
                    else
                    {
                        continue;
                    }
                }
            }

А в этом блоке, всегда будет заходить в:

C#

else
{
    continue;
}

В этом случае ниразу не будет вызван return, и программа не сможет выйти из метода и вернуть bool
Компилятор на стадии компиляции может это проверить и предостеречь Вас, так как C# является безопасным управляемым языком. Вам нужно переделать Ваш метод, чтобы во всех блоках было точных возврат из метода, чтобы при любом раскладе return был

@alya770 · 07.10.2017, 16:33 **[ТС]**

Если я убираю этот блок, все равно та же самая ошибка

@and09reyy · 07.10.2017, 17:06

попробуйте вот так

C#

private static Dictionary<TKey, ResearchTeam> TKeyResearchTeam;
        public bool Remove(ResearchTeam rt)
        {
            ResearchTeam researcht = new ResearchTeam();
            Dictionary<TKey, ResearchTeam>.KeyCollection keyColl = TKeyResearchTeam.Keys;
            if (TKeyResearchTeam.ContainsValue(rt))
            {
                foreach (TKey t in keyColl)
                {
                    TKeyResearchTeam.TryGetValue(t, out researcht);
                    if (rt == researcht)
                    {
                        return TKeyResearchTeam.Remove(t);
                    }
                    else
                    {
                        continue;
                    }
                }
            }
            
           return false;
            
        }

@Bretbas · 07.10.2017, 19:02

alya770,

Сообщение от alya770

Если я убираю этот блок, все равно та же самая ошибка

Его и не надо убирать. Надо просто перепроверить код, чтобы во всех случаях return был. Я же это хотел до Вас донести, а Вы так и не поняли...

@alya770 0 / 0 / 1 Регистрация: 10.12.2016 Сообщений: 36
	07.10.2017, 16:33 [ТС]	3
	Если я убираю этот блок, все равно та же самая ошибка 0

@Bretbas Каждому свое 533 / 219 / 81 Регистрация: 05.08.2013 Сообщений: 1,614
	07.10.2017, 19:02	5
	alya770, Сообщение от alya770 Если я убираю этот блок, все равно та же самая ошибка Его и не надо убирать. Надо просто перепроверить код, чтобы во всех случаях `return` был. Я же это хотел до Вас донести, а Вы так и не поняли... 0