Hello Pedro,

from my point of view it doesn't make sense to wrap each statement
into a try..catch. In you sample if httpclient.execute() fails, then
I'm not sure that response.getEntity().getContent() makes sense, since
there was no response.

I would rewrite your sample as:

try {
  httpPost.setEntity(new UrlEncodedFormEntity(nameValuePairs));
  response = httpclient.execute(httpPost);
  content = response.getEntity().getContent();
} catch(UnsupportedEncodingException e) {
  // Some error handling
} catch(ClientProtocolException e) {
  // Some error handling
} catch(IOException e) {
  // Some error handling
} catch(IllegalStateException e) {
  // Some error handling
}

Thus you have a one set of catch blocks per scenario not per
individual step. Using catch(Exception e) is not a good practice,
since you can miss something (usually run-time exception) you should
handle on development stage.

-- 
You received this message because you are subscribed to the Google
Groups "Android Developers" group.
To post to this group, send email to android-developers@googlegroups.com
To unsubscribe from this group, send email to
android-developers+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/android-developers?hl=en

Reply via email to