0

Здравствуйте! Извиняюсь, если вдруг повторюсь. Есть метод:

private string GetContent(string proxy, string address)
{
     var proxyClient = HttpProxyClient.Parse(proxy);
     var request = new HttpRequest();
     request.UserAgent = Http.ChromeUserAgent();
     request.Proxy = proxyClient;
     HttpResponse response = request.Get(address);
     return response.ToString();
}

А также метод, возвращающий предыдущий:

public string GeneralGetContent(string address)
{
     var proxies = File.ReadLines("Proxy_list.txt").ToList();
     var numProxies = proxies.Count;
     for (int t = 0;t < numProxies; t++)
     {
         try
         {
            return GetContent(proxies[goodProxyIndex], address);
         }
         catch (HttpException ex)
         {
            goodProxyIndex = (goodProxyIndex + 1) % numProxies;

         }
            throw new Exception("Не осталось хороших прокси");
     }
}

Компилятор выдает ошибку: GeneralGetContent(string): не все ветви кода возвращают значение.

Но ведь мне и не нужно, чтобы в блоке catch был return, потому что при неработающем прокси, в блоке catch он поменяется и будет вызван заново. Подскажите пожалуйста, как это обойти и убрать ошибку. Код был взят отсюда

14
  • 1
    ошибку выдает GetProxContent а ты привел GeneralGetContent и GetContent
    – Grundy
    21 ноя 2017 в 7:19
  • Извиняюсь что ввел в заблуждение вас, ошибся. Ошибку выдает: GeneralGetContent 21 ноя 2017 в 7:21
  • Так ведь у вас метод GeneralGetContent должен возвращать string, а слова return в нем ни одного( напишите в конце метода return ""; и усё 21 ноя 2017 в 7:21
  • 2
    @СергейПолушин за такие советы расстреливать надо. 21 ноя 2017 в 7:22
  • 1
    @МаксимМолчанов так у вас код из чужого ответа? Мда, вы даже скопировать код без ошибки не смогли... 21 ноя 2017 в 7:26

1 ответ 1

2

В случае, если файл Proxy_list.txt пустой, proxies.Count будет 0 и ты не войдешь в цикл.

Для этого случая у тебя нет return.


Как подсказывает @PavelMayorov скорее всего выброс исключения должен быть вне цикла

public string GeneralGetContent(string address)
{
    ...
    for (int t = 0;t < numProxies; t++)
    {
        try{
         return ...;
        }catch(...){...}
    }

    throw new Exception("Не осталось хороших прокси");
}
3
  • Да, помогло! Спасибо вам большое! Извиняюсь за тупость( 21 ноя 2017 в 7:34
  • @МаксимМолчанов, скорее невнимательность. Нужно быть чуть внимательнее, даже когда копируешь код из ответов.
    – Grundy
    21 ноя 2017 в 7:35
  • Хорошо, стыдно очень. извиняюсь еще раз. 21 ноя 2017 в 7:40

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.