androidStudioで天気予報のアプリを作りたい

Question

#androidStudioで天気予報のアプリを作りたいのですが、jsonで取得したデータが出力出来ません。
####[参照]天気予報のアプリ

###【activity_main.xml】

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

    <TextView
        android:id="@+id/messageTextView"
        android:text="Hello World!!"
        android:textSize="32sp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        android:layout_marginLeft="101dp"
        android:layout_marginTop="81dp" />

    <Button
        android:id="@+id/mybottun"
        android:onClick="changeTextView"
        android:text="ラベルを変更"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        android:layout_marginLeft="138dp"
        android:layout_marginTop="160dp" />

</androidx.constraintlayout.widget.ConstraintLayout>

###【AndroidManifest.xml】

?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.example.ddd">
    <!-- ネット接続を許可する -->
    <uses-permission android:name="android.permission.INTERNET" />
    <application
        android:allowBackup="true"
        android:icon="@mipmap/ic_launcher"
        android:label="@string/app_name"
        android:roundIcon="@mipmap/ic_launcher_round"
        android:supportsRtl="true"
        android:theme="@style/AppTheme">

        <activity android:name=".MainActivity">
            <intent-filter>

                <action android:name="android.intent.action.MAIN" />

                <category android:name="android.intent.category.LAUNCHER" />

            </intent-filter>
        </activity>
    </application>

</manifest>

###【MainActivity.java】

package com.example.ddd;

import androidx.appcompat.app.AppCompatActivity;
import java.net.MalformedURLException;
import java.net.URL;
import android.os.Bundle;
import android.view.View;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity {

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

    public void changeTextView(View view) {
        // 非同期処理(AsyncHttpRequest#doInBackground())を呼び出す
        try {
            new AsyncHttpRequest(this).execute(new URL("http://api.openweathermap.org/data/2.5/forecast?id=1850147&units=metric&cnt=1&appid={MY APEKEY}"));
        } catch (MalformedURLException e) {
            e.printStackTrace();
        }
    }
}

###【AsyncHttpRequest.java】

package com.example.ddd;

import android.app.Activity;
import android.os.AsyncTask;
import android.widget.TextView;
import com.example.ddd.R;
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.URL;
/**
 * 非同期処理を行うクラス.
 */
public  final class AsyncHttpRequest extends AsyncTask<URL, Void, String> {
    private int TODAY_FORCAST_INDEX = 0;
    private Activity mainActivity;

    public AsyncHttpRequest(Activity activity) {
        // 呼び出し元のアクティビティ
        this.mainActivity = activity;
    }

    /**
     * 非同期処理で天気情報を取得する.
     * @param urls 接続先のURL
     * @return 取得した天気情報
     */
    @Override
    protected String doInBackground(URL... urls) {

        final URL url = urls[0];
        HttpURLConnection con = null;

        try {
            con = (HttpURLConnection) url.openConnection();
            con.setRequestMethod("GET");
            // リダイレクトを自動で許可しない設定
            con.setInstanceFollowRedirects(false);
            con.connect();

            final int statusCode = con.getResponseCode();
            if (statusCode != HttpURLConnection.HTTP_OK) {
                System.err.println("正常に接続できていません。statusCode:" + statusCode);
                return null;
            }

            // レスポンス(JSON文字列)を読み込む準備
            final InputStream in = con.getInputStream();
            String encoding = con.getContentEncoding();
            if(null == encoding){
                encoding = "UTF-8";
            }
            final InputStreamReader inReader = new InputStreamReader(in, encoding);
            final BufferedReader bufReader = new BufferedReader(inReader);
            StringBuilder response = new StringBuilder();
            String line = null;
            // 1行ずつ読み込む
            while((line = bufReader.readLine()) != null) {
                response.append(line);
            }
            bufReader.close();
            inReader.close();
            in.close();

            // 受け取ったJSON文字列をパース
            JSONObject jsonObject = new JSONObject(response.toString());
            JSONObject todayForcasts = jsonObject.getJSONArray("weather").getJSONObject(TODAY_FORCAST_INDEX);

            return "今日の天気は" + todayForcasts.getString("main");
        } catch (IOException e) {
            e.printStackTrace();
            return null;
        } catch (JSONException e) {
            e.printStackTrace();
            return null;
        } finally {
            if (con != null) {
                con.disconnect();
            }
        }
    }

    /**
     * 非同期処理が終わった後の処理.
     * @param result 非同期処理の結果得られる文字列
     */
    @Override
    protected void onPostExecute(String result) {
        TextView tv = mainActivity.findViewById(R.id.messageTextView);
        tv.setText(result);
    }
}

###【ターミナル】

iMac215-2011:ddd admin$ curl "http://api.openweathermap.org/data/2.5/forecast?id=1850147&units=metric&cnt=1&appid="MyAppID" | jq '.'
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   590  100   590    0     0   2163      0 --:--:-- --:--:-- --:--:--  2169
{
  "cod": "200",
  "message": 0,
  "cnt": 1,
  "list": [
    {
      "dt": 1607860800,
      "main": {
        "temp": 10.04,
        "feels_like": 6.37,
        "temp_min": 10.04,
        "temp_max": 10.54,
        "pressure": 1010,
        "sea_level": 1010,
        "grnd_level": 1006,
        "humidity": 78,
        "temp_kf": -0.5
      },
      "weather": [
        {
          "id": 802,
          "main": "Clouds",
          "description": "scattered clouds",
          "icon": "03n"
        }
      ],
      "clouds": {
        "all": 43
      },
      "wind": {
        "speed": 4.05,
        "deg": 97
      },
      "visibility": 10000,
      "pop": 0,
      "sys": {
        "pod": "n"
      },
      "dt_txt": "2020-12-13 12:00:00"
    }
  ],
  "city": {
    "id": 1850147,
    "name": "Tokyo",
    "coord": {
      "lat": 35.6895,
      "lon": 139.6917
    },
    "country": "JP",
    "population": 0,
    "timezone": 32400,
    "sunrise": 1607809343,
    "sunset": 1607844516
  }
}
iMac215-2011:ddd admin$ 

Comments

[dodox86]

ソースコードはマークダウン記法を用いて読みやすくするようにしてください。インデントも崩れ、読みづらいとそれだけで回答は離れがちになります）
https://teratail.com/help/question-tips#questionTips3-5-1
https://teratail.com/questions/238564
あたりを参考にしてソースコードを適切に表示するようにしてください。

更に「AsyncHttpRequest」は自作のクラスだと思いますが、MainActivityとパッケージが違うのではないですか？それぞれのソースファイルが配置されているディレクトリ構造も示しましょう。(質問文中への追記、修正でお願いします）

[shinmo]

ご返事、又ご指摘ありがとうございます。
読みやすいように修正しました。（まだ読みづらかったら教えて下さい）
ご指摘どうりAsyncHttpRequest.javaのパッケージを指定したらエラーがなくなりました。ありがとうございました。
申し訳ありませんがディレクトリ構造の示し方がいまいちわからなかったのですが添付したものの事でしょうか？

[dodox86]

「ディレクトリ構造を示してください」と書いたのは、質問当初の「シンボルが見つからない」エラーがパッケージ指定が合っていないのが原因だとも思ったので、パッケージ指定の構造と合っているかどうかを閲覧者、回答者の皆さんが判断できるように、との意図で指摘したものです。そのエラーが解決したのであれば、それで結構です。

[dodox86]

参照されているサイトの記事は2018年6月でまぁ、最近と言えば最近なのですが、「Weather Hacks」のサービスは今はどうもそのままのURLでは使えないようです。ステータスコード302はエラーというよりリダイレクトで、コンテンツの移動を示しています。livedoor内で移動したか、サービス自体が停止しているのでは。Web APIを使うときはいきなりプログラムに組み込むのではなく、curlやwgetなどでまず使い方や結果を確認しましょう。

$ curl http://weather.livedoor.com/forecast/webservice/json/v1?city=270000
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>302 Found</title>
</head><body>
<h1>Found</h1>
<p>The document has moved <a href="http://www.livedoor.com/?city=270000">here</a>.</p>
</body></html>

[shinmo]

dodox86さんご指摘ありがとうございます。curlの使い方を確認し結果も出るところまで行ったのですが（別のURLを使いました。）実行すると、エラーは出ず、ボタンを押下するとテキストが消えます。
jsonをパースするのが間違えているのか、取りだす配列が間違えているのかわかりません。
ご教授お願いします。

[dodox86]

[2020/12/13 12:02]の編集履歴を見て：
AndroidManifest.xmlの提示の際の記法に間違いがあって、表示がひどく崩れています。
正: ```xml
誤: ``xml

それと、一応指摘しておくと、レスポンスで得たJSONのデータは画像で貼ると閲覧した方や回答しようとする方にとって扱いづらいです。検証できません。

[shinmo]

dodox86さんご指摘ありがとうございます！
表示の修正と画像ではなくコードとして入力致しました。
お忙しい中だと思いますが宜しくお願い致します。

Answer 1

ご提示のコードではパース処理が足りないです。まず、質問文中に提示されたHTTPレスポンス中のJSON文字列を抜粋すると、およそ以下のようになります。

{
  "list": [
    {
      "main": {
      },
      "weather": [
        {
          "id": 802,
          "main": "Clouds",
          "description": "scattered clouds",
          "icon": "03n"
        }
      ]
    }
  ]
}

ここで、最上位の"list"は配列、その中の"weather"も配列なので、"main"の値を取り出したいのであれば、例えば以下のようなコードになります。

String content = "{...}" // JSON文字列
JSONObject jsonContent = new JSONObject(content);
JSONArray array = jsonContent.getJSONArray("list");
JSONObject todayForcasts = array.getJSONObject(0);
JSONArray items  = todayForcasts.getJSONArray("weather");

// "main": "Clouds" の値をresultに
String result = items.getJSONObject(0).getString("main");
Log.d(TAG, result);

対象のJSON文字列が想定外のものであった場合の例外の捕捉など、エラー系の処理は省いていますので適切に対処してください。

Comments

[shinmo]

dodox86さんご指摘ありがとうございます
1点わからないのですが、『"{...}" // JSON文字列』の"{}"の中にcurlで取得した文字列を全て入れるという認識で合っているでしょうか？

[dodox86]

「curl等を使ってみて」と提言したのは、Web APIを初めて使うような場合はそういったコマンドを使って動作確認した方が理解が早まるはずだからです。ですので、

> curlで取得した文字列を全て入れるという認識で合っているでしょうか？

curlコマンドで正しく取得できた場合のJSON文字列と言うのは、ご提示のコードの以下の部分、response.toString() に相当するであろうから、

> // 受け取ったJSON文字列をパース
> JSONObject jsonObject = new JSONObject(response.toString());

curlコマンドではなくプログラムで取得する場合はそれを使うことでパースできるでしょう、ということです。

[shinmo]

dodox86さんご指摘ありがとうございました。
無事表示することが出来ました。本当に助かりました！
何度もご返答頂き心より感謝致します。