AndroidでのHttpClientを使うにあたっての注意点か何か

[利崇山崎]

先日はお世話になりました。ブログにも出てしまい、何かやらないとということで
何やらやっている山崎です。

開発で一つ非常に引っかかっている部分があり、何か意見をいただけたらいいのですが・・・
AndroidでのApacheのHttpClientクラスに関して何か使うにあたって注意点みたいなものはあるでしょうか？

いろいろ横着してましてHttpClient4.0のほうを使っているのですが
接続が安定せず、かなりの確率で
　Socket is not connected
というエラーを吐いてしまっています。・・・これが毎度ならわかるんですが、出たりでなかったりするのが問題でありまして。

同時接続処理がおかしくなる原因と睨んで、
Clientのインスタンスの管理をsingletonにしまして、使用するメソッドをすべてsynchronizedで一本化したのですが
やっぱり発生してしまうのです

UIのメインスレッドからの分岐はAsyncTackクラスを使って、あとは普通に同期的に通信するように設計しています。

同様の問題を抱えている方とか、解決法を知っているという方はいませんでしょうか？
何か糸口だけでも教えていただきたいです

Javaのプログラミング流儀に不慣れでネットーワーク系の慣例的な実装がよくわからす、さらに参考書の記述もいろいろで混乱気味ですｗ

[tyamaz]

自己解決しそうな感じで・・・テストサーバーにWindowsXPを使ってたんですがこいつのソケットがなにやらおかしいようです。
HttpClient4系とWindowsのSocket実装が相性悪いのかもという・・・

[tyamaz]

わかっている範囲で自己レス。
SDK同梱のHttpClientは動いているスレッドが何らかの原因でエラーで落ちた場合その影響をうけてさらにエラーを誘発して接続に不具合が出る
模様
エラー処理を細やかにすれば発生は抑えられる・・・か？対処療法ですけどｗ

[tyamaz]

さらに自己レス＋コード晒し
2010-04-05のAndroidの会の懇親会で色々相談に乗ってくれた方々ありがとうございます。
前回の自己レスよりも多少進展があったので・・・さらに自己レスしておきます。
状況だけ・・・
USBデバッグ接続でアプリを実機に転送して立ち上げる場合に一発目のエラーが発生しやすい
エラーが発生すると以後Socket is not connected が頻発する

runtimeerrorとかでシステム側までエラーが登りきった場合
以後Socket is not connected が頻発する

アプリのタスクを完全に殺しても、エラーが頻発する。
挙動としてはエラーがエラーを助長させる雰囲気
エラーは開発中のアプリのみで発生する。
他のアプリには影響が無い。
本体の電源を入れ直すと回復する

現在は立ち上げの際にエラーが出たら一旦接続を解除して、また接続するということでなんだかエラーが収まっているのでなんとかできているみたいですが
やはりたまにこの状況になりますね。

一応参考までにコード晒しておきます
public enum Connection{
INSTANCE;

private HttpClient client = null;

private HttpClient getClient(){
if(client == null){
client = new DefaultHttpClient();

client.getParams().setIntParameter("http.connection.timeout", 5000);
client.getParams().setIntParameter("http.socket.timeout",
5000);

client.getParams().setParameter(ClientPNames.COOKIE_POLICY,
CookiePolicy.BROWSER_COMPATIBILITY);
}
return client;
}

synchronized public byte[] get(String url, List<SendParam>
sendParams) throws Exception{
List<String> paramString = new ArrayList<String>();
for(SendParam param : sendParams){
switch(param.getPartType()){
case FILE:
break;
case STRING:
paramString.add(param.getName() + "=" +
param.getBody());
break;
}
}

if(!paramString.isEmpty()){
url = url + "?" + StringUtils.join(paramString, "&");
}

HttpGet get = new HttpGet(url);
byte[] result = new byte[0];

try{
HttpEntity he = doRequest(get);
result = EntityUtils.toByteArray(he);
}
catch(Exception e){
Log.e("Connection.get", e.getMessage());
}
finally{
if(get != null && !get.isAborted()){
get.abort();
}
}
return result;
}

synchronized private HttpEntity doRequest(HttpUriRequest method)
throws Exception{
HttpResponse response = null;
try{
response = getClient().execute(method);
StatusLine statusLine = response.getStatusLine();
if(statusLine.getStatusCode() != HttpURLConnection.HTTP_OK)
{
throw new Exception("error");
}
}
catch(Exception e){
throw e;
}
HttpEntity he = response.getEntity();

return he;
}

}

このgetメソッドを
Activity内のインナークラスとして作成したAsyncTaskのdoInBackgroundメソッド内で使っています。
Javaのコードはあまり得手ではないので、変なコードかもしれないですけど、こんな感じです。SendParamは単なる入れ物的クラスです。