Почему не вызывается обратный вызов BeginGetResponse? - PullRequest
Новая
       58

Почему не вызывается обратный вызов BeginGetResponse?

1 голос
/ 06 октября 2011

Вот мой код: 

namespace RequestApi
{
    public partial class MainPage : PhoneApplicationPage
    {
        private BackgroundWorker bw;
        private string ans;
        private JObject ansJson;
        private static ManualResetEvent allDone = new ManualResetEvent(false);
        // Constructor
        public MainPage()
        {
            InitializeComponent();

        }

        private void button1_Click(object sender, RoutedEventArgs e)
        {
            string url = "http://192.168.0.43:8182/Account/SignIn";
            CreateRequest(url);

            userId.Text = ansJson.Value<int>("user_id").ToString();

        }


        private void CreateRequest(string url)
        {
            Debug.WriteLine("CreateRequest");
            HttpWebRequest req = (HttpWebRequest)HttpWebRequest.Create(url);
            req.ContentType = "application/json";
            req.Method = "POST";
            req.BeginGetRequestStream(new AsyncCallback(RequestCallback), req);
            allDone.WaitOne();
        }

        private  void RequestCallback(IAsyncResult aresult)
        {
            Debug.WriteLine("RequestCallback");
            HttpWebRequest req = (HttpWebRequest)aresult.AsyncState;
            Stream postStream = req.EndGetRequestStream(aresult);

            string obj = "{ 'username': 'test_2@aragast.com', 'password': 'a123456' }";
            JObject json = JObject.Parse(obj);
            string s = JsonConvert.SerializeObject(json);
            byte[] postdata = System.Text.Encoding.Unicode.GetBytes(s);

            postStream.Write(postdata, 0, postdata.Length);
            postStream.Close();

            req.BeginGetResponse(new AsyncCallback(ResponseCallback), req);


        }

        private  void ResponseCallback(IAsyncResult aresult)
        {
            Debug.WriteLine("ResponseCallback");
            HttpWebRequest req = (HttpWebRequest)aresult.AsyncState;
            HttpWebResponse resp = (HttpWebResponse)req.EndGetResponse(aresult);
            StreamReader reader = new StreamReader(resp.GetResponseStream());
            string response = reader.ReadToEnd();
            Debug.WriteLine(response);
            JObject responseJson = JObject.Parse(response);
            ansJson = responseJson;
            Debug.WriteLine("ansJson from responseCallback {0}", ansJson);
            reader.Close();
            resp.Close();
            allDone.Set();

        }
    }
}

Когда я отлаживаю приложение, оно вводит CreateRequest, затем вводит RequestCallback, но затем останавливается и никогда не вводит ResponseCallback; вместо этого он пытается использовать userId.Text, чтобы присвоить значению asnJson значение null, поскольку оно не вводит ResponseCallback. Когда я делаю неправильно, и почему он никогда не входит в ResponseCallback?

Ответы [ 2 ]

2 голосов
/ 06 октября 2011

Ваш ManualResetEvent создан с true в качестве аргумента, поэтому он уже имеет сигнал для начала. Это означает, что вызов allDone.Wait() будет немедленно продолжен ... так что CreateRequest завершится, и вы немедленно попытаетесь использовать переменную asnJson, которая, как вы сказали, null , Я подозреваю, что это убьет приложение, поэтому у вас никогда не будет возможности получить ответ.

Теперь, решение для этого - , а не , чтобы изменить конструктор ManualResetEvent - вы все равно не должны так ждать в потоке пользовательского интерфейса! Вы заблокируете его и удалите весь смысл Windows Phone 7, сделав все асинхронным для начала.

Вместо этого ваши обратные вызовы должны использовать диспетчер для обратного вызова в потоке пользовательского интерфейса, когда они закончили. Обратите внимание, что вы также должны иметь надежную обработку ошибок на случай, если с запросом что-то пойдет не так.

1 голос
/ 06 октября 2011

Не совсем связано с ответом на вопрос, но блок ответов, кажется, единственное место, которое соответствует рекомендации, которую я собираюсь дать ... Вы должны иметь соответствующую защиту ресурсов для своих потоков, используя блоки using как например:

Оригинальный код: 

        Stream postStream = req.EndGetRequestStream(aresult);

        string obj = "{ 'username': 'test_2@aragast.com', 'password': 'a123456' }";
        JObject json = JObject.Parse(obj);
        string s = JsonConvert.SerializeObject(json);
        byte[] postdata = System.Text.Encoding.Unicode.GetBytes(s);

        postStream.Write(postdata, 0, postdata.Length);
        postStream.Close();

Новый код ( также убедитесь, что ваша кодировка верна для запроса: ваш веб-сервис действительно ожидает UTF-16? Чаще всего веб-серверы используют UTF-8 (Encoding.UTF8) ): 

        using (Stream postStream = req.EndGetRequestStream(aresult))
        {
            string obj = "{ 'username': 'test_2@aragast.com', 'password': 'a123456' }";
            JObject json = JObject.Parse(obj);
            string s = JsonConvert.SerializeObject(json);
            byte[] postdata = System.Text.Encoding.Unicode.GetBytes(s);

            postStream.Write(postdata, 0, postdata.Length);
        }

Оригинальный код: 

        HttpWebResponse resp = (HttpWebResponse)req.EndGetResponse(aresult);
        StreamReader reader = new StreamReader(resp.GetResponseStream());
        string response = reader.ReadToEnd();
        Debug.WriteLine(response);
        JObject responseJson = JObject.Parse(response);
        ansJson = responseJson;
        Debug.WriteLine("ansJson from responseCallback {0}", ansJson);
        reader.Close();
        resp.Close();

Новый код: 

        using (HttpWebResponse resp = (HttpWebResponse)req.EndGetResponse(aresult))
        using (StreamReader reader = new StreamReader(resp.GetResponseStream()))
        {
            string response = reader.ReadToEnd();
            Debug.WriteLine(response);
            JObject responseJson = JObject.Parse(response);
            ansJson = responseJson;
            Debug.WriteLine("ansJson from responseCallback {0}", ansJson);
        }

Я бы также рекомендовал окружать ваши операции req и resp блоками try..catch, чтобы что-то могло обрабатывать исключительные условия - в противном случае это будет связано с обработчиком исключений AppDomain (также может быть перехвачен событием UnhandledException ).

...