2

Метод справляется со своей задачей, только мне не нравится как это всё записано, уверен можно сделать красивее, и увеличить читабельность кода.

public int GetStatusCode(string url)
{
    int statusCode = 0; // хочу избавиться от объявления здесь.
    try
    {
        HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url);
        request.Method = WebRequestMethods.Http.Head;
        request.AllowAutoRedirect = false;
        request.Accept = @"*/*";
        HttpWebResponse response = (HttpWebResponse)request.GetResponse();
        statusCode = (int)response.StatusCode;
        response.Close();
    }
    catch (WebException ex)
    {
        if (ex.Response == null)
            throw;
        statusCode = (int)((HttpWebResponse)ex.Response).StatusCode;
    }
    return statusCode;
}

Что-то на подобии этого не предлагайте:

public int GetStatusCode(string url, int statusCode = 0)

Оптимизированный код должен работать на версии .Net 4.0, спасибо.

6
  • а что вас не устраивает?
    – Monomax
    1 авг 2018 в 8:48
  • @VladimirT в коде есть комментарий, объявление int statusCode 1 авг 2018 в 8:52
  • А что вам мешает? удалите строку с status code и возвращайте в блоке try: return (int)response.StatusCode; 1 авг 2018 в 8:57
  • @Kirill'luridSNK'F. вы дали комментарий когда я дал ответ)))
    – Monomax
    1 авг 2018 в 8:58
  • почему бы не сделать метод асинхронным?
    – tym32167
    1 авг 2018 в 9:01

3 ответа 3

4

Пример асинхронного вызова

public async Task<int> GetStatusCode(string url)
{
    try
    {
        HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url);
        request.Method = WebRequestMethods.Http.Head;
        request.AllowAutoRedirect = false;
        request.Accept = @"*/*";

        using (var response = (HttpWebResponse)await request.GetResponseAsync())
        {
            return (int)response.StatusCode;
        }
    }
    catch (WebException ex)
    {
        if (ex.Response == null)
            throw;
        return (int)((HttpWebResponse)ex.Response).StatusCode;
    }
}

Ну, или, например, так

async Task<int> GetStatus(string uri)
{
    var handler = new HttpClientHandler()
    {
        AllowAutoRedirect = false
    };

    using (var cl = new HttpClient(handler))
    {
        cl.DefaultRequestHeaders.Accept.Add(new MediaTypeWithQualityHeaderValue(@"*/*"));
        var response = await cl.SendAsync(new HttpRequestMessage(HttpMethod.Head, uri),
             HttpCompletionOption.ResponseHeadersRead);
        return (int)response.StatusCode;
    }
}
2
  • Спасибо, а я как раз не знал как обернуть в using - прямо читаете мысли, теперь понял где была ошибка IDisposable. 1 авг 2018 в 10:07
  • @SergeyUsick пожалуйста :)
    – tym32167
    1 авг 2018 в 10:11
4

WebResponse реализует интерфейс IDisposable, поэтому желательно его диспозить. Проще всего это сделать, обернув код в using.

public int GetStatusCode(string url)
{
    try
    {
        HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url);
        request.Method = WebRequestMethods.Http.Head;
        request.AllowAutoRedirect = false;
        request.Accept = @"*/*";
        using (HttpWebResponse response = (HttpWebResponse)request.GetResponse())
        {
            return (int)response.StatusCode;
        }
    }
    catch (WebException ex)
    {
        if (ex.Response == null)
            throw;
        return (int)((HttpWebResponse)ex.Response).StatusCode;
    }
}
1

Всего лишь перенести return:

public int GetStatusCode(string url)
{

  try
  {
    HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url);
    request.Method = WebRequestMethods.Http.Head;
    request.AllowAutoRedirect = false;
    request.Accept = @"*/*";
    HttpWebResponse response = (HttpWebResponse)request.GetResponse();
    var rescode = (int)response.StatusCode;
    response.Close();
    return rescode;
  }
  catch (WebException ex)
  {
    if (ex.Response == null)
        throw;
    return (int)((HttpWebResponse)ex.Response).StatusCode;
  }

}
4
  • response.Close() после returnне сработает. 1 авг 2018 в 8:58
  • после return (int)response.StatusCode; выполнится response.Close(); ? 1 авг 2018 в 8:58
  • @Kirill'luridSNK'F. поправил, согласен с вами.
    – Monomax
    1 авг 2018 в 9:00
  • @SergeyUsick комментарий ранее.
    – Monomax
    1 авг 2018 в 9:00

Ваш ответ

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

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