AndroidStudioでhttpURLConnectionのConnect()メソッドで例外処理になってしまう

Question

いつもありがとうございます。
このプログラムをAndroidStudioで実行すると例外処理になってしまいます。
connect()メソッドから例外処理になってしまいます。
このサンプルプログラムはもともとボタンを押されるとhttpURLConnectionで接続されるプログラムなのですが、
起動時にhttpURLConnectionで値を持ってきたいです。
例外処理の原因を教えて頂けませんでしょうか。
よろしくお願いいたします。

package com.websarva.wings.android.asyncsample;

import android.annotation.SuppressLint;
import android.os.AsyncTask;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.AdapterView;
import android.widget.ListView;
import android.widget.SimpleAdapter;
import android.widget.TextView;

import org.json.JSONArray;
import org.json.JSONException;
import org.json.JSONObject;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.URL;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

public class WeatherInfoActivity extends AppCompatActivity {

	@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_weather_info);


		//画面部品ListViewを取得
		ListView lvCityList = findViewById(R.id.lvCityList);
		//SimpleAdapterで使用するListオブジェクトを用意。
		List<Map<String, String>> cityList = new ArrayList<>();
		//都市データを格納するMapオブジェクトの用意とcityListへのデータ登録。
		Map<String, String> city = new HashMap<>();
		//PlaceGetter place = new PlaceGetter();

		http h = new http();
		URL url = null;
		String s = null;

		try {
			url = new URL("https://www.yahoo.co.jp/");
			s = h.doInBackground(url);
		} catch (MalformedURLException e) {
			e.printStackTrace();
		}
		city.put("name", s);
		city.put("id", "270000");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "神戸");
		city.put("id", "280010");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "豊岡");
		city.put("id", "280020");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "京都");
		city.put("id", "260010");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "舞鶴");
		city.put("id", "260020");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "奈良");
		city.put("id", "290010");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "風屋");
		city.put("id", "290020");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "和歌山");
		city.put("id", "300010");
		cityList.add(city);
		city = new HashMap<>();
		city.put("name", "潮岬");
		city.put("id", "300020");
		cityList.add(city);
		//SimpleAdapterで使用するfrom-to用変数の用意。
		String[] from = {"name"};
		int[] to = {android.R.id.text1};
		//SimpleAdapterを生成。
		SimpleAdapter adapter = new SimpleAdapter(WeatherInfoActivity.this, cityList, android.R.layout.simple_expandable_list_item_1, from, to);
		//ListViewにSimpleAdapterを設定。
		lvCityList.setAdapter(adapter);
		//ListViewにリスナを設定。
		lvCityList.setOnItemClickListener(new ListItemClickListener());
	}

package com.websarva.wings.android.asyncsample;
import android.os.AsyncTask;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.ProtocolException;
import java.net.URL;

public final class http extends AsyncTask<URL, Void, String> {

    @Override
    protected String doInBackground(URL... urls) {
        // 取得したテキストを格納する変数
        final StringBuilder result = new StringBuilder();
        // アクセス先URL
        final URL url = urls[0];

        HttpURLConnection con = null;
        try {
            // ローカル処理
            // コネクション取得
            con = (HttpURLConnection) url.openConnection();
            con.setConnectTimeout(10000);
            con.connect();

            // HTTPレスポンスコード
            final int status = con.getResponseCode();
            if (status == HttpURLConnection.HTTP_OK) {
                // 通信に成功した
                // テキストを取得する
                final InputStream in = con.getInputStream();
                final String encoding = con.getContentEncoding();
                final InputStreamReader inReader = new InputStreamReader(in, encoding);
                final BufferedReader bufReader = new BufferedReader(inReader);
                String line = null;
                // 1行ずつテキストを読み込む
                while((line = bufReader.readLine()) != null) {
                    result.append(line);
                }
                bufReader.close();
                inReader.close();
                in.close();
            }

        } catch (MalformedURLException e1) {
            e1.printStackTrace();
        } catch (ProtocolException e1) {
            e1.printStackTrace();
        } catch (IOException e1) {
            e1.printStackTrace();
        } finally {
            if (con != null) {
                // コネクションを切断
                con.disconnect();
            }
        }
        return result.toString();
    }
}

Comments

[hoshi-takanori]

AsyncTask の使い方が間違ってます。doInBackground を直接呼んではいけません。h.execute() すると別のスレッドで doInBackground が実行され、それが終了するとメインスレッドで onPostExecute が呼ばれます。
Android ではメインスレッドでは通信は行えませんし、メインスレッドでないと GUI をいじれないので、doInBackground で通信した結果を onPostExecute で受け取って GUI を更新する必要があります。
あと、http というクラス名はどうかと思います。クラス名の最初の文字は大文字にすべきですし、Http だけだと何のことかよく分からないので、せめて HttpTask とかにすることをお勧めします。

[keicha_hrs]

AsyncTaskは次期APIレベルで非推奨になるので、今のうちに別の方法を検討した方がいいのではないかと思います。（ベストは開発言語をKotlinに変えてCoroutineを用いることだろうが）

[urncus]

hoshi-takanori様
ありがとうございます。解決しました！

Answer 1

エラー内容が不明ですが、以下の点をご確認出来ますでしょうか。

１、AndroidManifest.xmlファイルに、インターネットアクセスのパーミッションを記載している？

// 以下を記載済みかどうか
<uses-permission android:name="android.permission.INTERNET" />

２、AsyncTaskの実行方法が違うのでは？

// 誤
s = h.doInBackground(url);

// 正
s = h.execute(url);

AsyncTaskのドキュメント