AJAX : How can I set response type other than 200 ("OK") in Server Response

[Sagar Rohankar]

Hi all,

From my servlet I'm setting

response.sendError(100); 

i.e other than 200 ("OK") message ! But while checking at client side using AJAX like

 if (req.readyState == 4) { // Complete
                if (req.status == 200) { // OK response   
                        }
                alert("ERROR"); // This is not working !
                       

The alert stating "ERROR", is not executing ? Whats the problem ?

[Bear Bibeault]

Because of the wonky formatting of your posted code (is that really how you write your code?) I can't tell what's going on. it looks like your alert is outside any conditional that checks the error code, so it should display always. If not, then you're not getting to that block of code at all.

[Sagar Rohankar]

Sorry for the formatting !

Here is AJAX 'call back handler' code

function processStateChange() { 
            if (req.readyState == 4) { // Complete
                if (req.status == 200) { // OK response
              
               // Some things to write on HTML
                         
                }else  // 1
                     {
                           // NOT "OK" response
                           alert("I`m not OK RESPONSE !");
                      }   
              }
           }

else block at line 1 , is not executing , Im expecting it because , In my Server side Servlet action , I wrote

 
           PrintWriter out = response.getWriter();
           out.print("<root>Hi</root>");
           response.sendError(100); 
           out.flush();
           out.close();
           return null;
           

So where I'm wrong ?

[Anirvan Majumdar]

You need to set the response status to something else other than 200 incase of an error.

response.setStatus(500);

That should work in your case.
PS: your code formatting is still awfully wonky. Bugs resulting from lack of proper indentation can give you awful nightmares in the future.

[Sagar Rohankar]

Originally posted by Anirvan Majumdar:
You need to set the response status to something else other than 200 incase of an error.

response.setStatus(500);

Ok, Thanks for the help ! I ll try !

Originally posted by Anirvan Majumdar:


PS: your code formatting is still awfully wonky.

Can you please let me know whats the wonky thing about my above code ?

I put only a part of code , which is of interest !

[Anirvan Majumdar]

Since you insist, let me try highlighting the issue with your indentation.
This is how your code appears to be:
----------------------------------------

function processStateChange() { 
            if (req.readyState == 4) { // Complete
                if (req.status == 200) { // OK response
              
               // Some things to write on HTML
                         
                }else  // 1
                     {
                           // NOT "OK" response
                           alert("I`m not OK RESPONSE !");
                      }   
              }
           }

A much better way of writing the code above would be:
-----------------------------

   function processStateChange() { 
      // Complete
      if (req.readyState == 4) { 
         // OK response
         if (req.status == 200) { 
            // Some things to write on HTML
         }else  {
            // NOT "OK" response
            alert("I`m not OK RESPONSE !");
         }
      }
   }

Now you might argue that there's not much to it, in which case, I guess there's nothing much to say. Putting in a little bit of effort to structure your code not only helps you minimise errors, but also helps those who try and go through the code to help you fix an error.