変数の初期化をする場所を変えることで、強制終了する。

Question

### 前提・実現したいこと


Android Studio を用いて、スマホアプリを開発しています。
ボタンを押すと、距離を算出して画面に表示するプログラミングを作っており、わからないことがあるのです。
緯度経度を定義している部分（8つ）をリスナの外に出したいのですが、強制終了してしまいます。
リスナの中に定義すると、実行はするのですが、できればリスナ外で定義をしたいです。
どのようにすれば宜しいでしょうか？


### 該当のソースコード

```ここに言語名を入力

public class DistanceActivity extends AppCompatActivity {

    private FusedLocationProviderClient fusedLocationClient;
    TextView home_latitude;
    TextView write_latitude;
    TextView home_longitude;
    TextView write_longitude;
    TextView set_distance;
    EditText assumed_distance;

    TextView now_latitude;
    TextView now_longitude;
    TextView num_distance_to_home;

    String num_distance_to_home2;
    double num_distance_to_home3;
    String set_distance2;
    double set_distance3;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_distance);

        // LocationClientクラスのインスタンスを生成
        fusedLocationClient = LocationServices.getFusedLocationProviderClient(this);

        // 位置情報取得開始
        startUpdateLocation();


　　　　//こちらに以下の8つを置きたい。

        Button register_myHome = findViewById(R.id.register_my_home);
        register_myHome.setOnClickListener(v -> {

            home_latitude = findViewById(R.id.home_latitude);
            write_latitude = findViewById(R.id.now_latitude);
            String text1 = write_latitude.getText().toString();
            home_latitude.setText(text1);

            home_longitude = findViewById(R.id.home_longitude);
            write_longitude = findViewById(R.id.now_longitude);
            String text2 = write_longitude.getText().toString();
            home_longitude.setText(text2);

        });
        home_latitude = findViewById(R.id.home_latitude);
        home_longitude = findViewById(R.id.home_longitude);


        //ボタンを押すと、距離を算出してくれる仕組み
        //distance[0]は2地点間の距離
        Button btn_distance_to_home = findViewById(R.id.btn_distance_to_home);
        btn_distance_to_home.setOnClickListener(v -> {

            //以下の８つ
            String now_latitude2 = now_latitude.getText().toString();
            String now_longitude2 = now_longitude.getText().toString();
            double now_latitude3 = Double.parseDouble(now_latitude2);
            double now_longitude3 = Double.parseDouble(now_longitude2);

            String home_latitude2 = home_latitude.getText().toString();
            String home_longitude2 = home_longitude.getText().toString();
            double home_latitude3 = Double.parseDouble(home_latitude2);
            double home_longitude3 = Double.parseDouble(home_longitude2);
            //以上

            float[] distance =
                    getDistance(now_latitude3, now_longitude3, home_latitude3, home_longitude3);

            String dis = String.valueOf(distance[0]);
            num_distance_to_home = findViewById(R.id.num_distance_to_home);
            num_distance_to_home.setText(dis);
        });
        num_distance_to_home = findViewById(R.id.num_distance_to_home);
    }


    /*
     * 2点間の距離（メートル）、方位角（始点、終点）を取得
     * ※配列で返す[距離、始点から見た方位角、終点から見た方位角]
     */

    public float[] getDistance(double x, double y, double x2, double y2) {
        // 結果を格納するための配列を生成
        float[] results = new float[3];

        // 距離計算
        Location.distanceBetween(x, y, x2, y2, results);

        return results;
    }

}
```

Answer

機能とコードを整理しつつというところで、例えば以下のようなコードにもなり得ます。(とりあえずサンプルとして不要・不明な部分は削除・コメントしています)

GlobalPosition というクラスを新設し、緯度経度の 2 つを保持させます。
この 2 つは、双方個々に対応する TextWatcher を TextView に addTextChangedListener することで、 TextView が変更されると自動的に値が更新されます。
getDistance メソッドを GlobalPosition に移し、自分の座標からパラメータで渡される座標までを計算させるようにすると、 btn_distance_to_home のクリック処理では nowGP.getDistance(homeGP) の結果を表示するだけで済むようになります。

他にあるという緯度経度を用いる処理でも、 GlobalPosition に入れるか GlobalPosition をパラメータとして使用すれば、見た目ある程度は整理されるのではないでしょうか。
```java
package com.teratail.q374330;

import androidx.appcompat.app.AppCompatActivity;

import android.location.Location;
import android.os.Bundle;
import android.text.*;
import android.widget.*;

//import com.google.android.gms.location.*;

public class DistanceActivity extends AppCompatActivity {

  //private FusedLocationProviderClient fusedLocationClient;

  private TextView homeLatitudeTextView;
  private TextView homeLongitudeTextView;
  private TextView nowLatitudeTextView;
  private TextView nowLongitudeTextView;
  private TextView distanceToHomeTextView;

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_distance);

    // 各View取得
    homeLatitudeTextView = findViewById(R.id.home_latitude);
    homeLongitudeTextView = findViewById(R.id.home_longitude);
    nowLatitudeTextView = findViewById(R.id.now_latitude);
    nowLongitudeTextView = findViewById(R.id.now_longitude);
    distanceToHomeTextView = findViewById(R.id.num_distance_to_home);

    //GlobalPosition オブジェクト設定
    GlobalPosition homeGP = new GlobalPosition(1,2);
    homeLatitudeTextView.addTextChangedListener(homeGP.getLatitudeTextWatcher());
    homeLongitudeTextView.addTextChangedListener(homeGP.getLongitudeTextWatcher());

    GlobalPosition nowGP = new GlobalPosition(3,4);
    nowLatitudeTextView.addTextChangedListener(nowGP.getLatitudeTextWatcher());
    nowLongitudeTextView.addTextChangedListener(nowGP.getLongitudeTextWatcher());

    // LocationClientクラスのインスタンスを生成
    //fusedLocationClient = LocationServices.getFusedLocationProviderClient(this);

    // 位置情報取得開始
    //startUpdateLocation();

    Button registerMyHomeButton = findViewById(R.id.register_my_home);
    registerMyHomeButton.setOnClickListener(v -> {
      homeLatitudeTextView.setText(nowLatitudeTextView.getText().toString());
      homeLongitudeTextView.setText(nowLongitudeTextView.getText().toString());
    });

    //ボタンを押すと、距離を算出してくれる
    Button distanceToHomeButton = findViewById(R.id.btn_distance_to_home);
    distanceToHomeButton.setOnClickListener(v -> {
      distanceToHomeTextView.setText(""+nowGP.getDistance(homeGP));
    });
  }
}

class GlobalPosition {
  private double latitude; //緯度
  private double longitude; //経度

  GlobalPosition() {}

  //テスト用
  GlobalPosition(double latitude, double longitude) {
    this.latitude = latitude;
    this.longitude = longitude;
  }

  double getLatitude() { return latitude; }
  double getLongitude() { return longitude; }

  /*
   * to までの距離（メートル）
   */
  float getDistance(GlobalPosition to) {
    // 結果を格納するための配列を生成
    float[] results = new float[3];

    // 距離計算
    Location.distanceBetween(latitude, longitude, to.latitude, to.longitude, results);

    return results[0];
  }

  private double getDouble(Editable editable) {
    try {
      return Double.parseDouble(editable.toString());
    } catch(NumberFormatException e) {
      return 0.0;
    }
  }

  TextWatcher getLatitudeTextWatcher() {
    return new TextWatcher() {
      @Override
      public void beforeTextChanged(CharSequence charSequence, int i, int i1, int i2) {} //ignore
      @Override
      public void onTextChanged(CharSequence charSequence, int i, int i1, int i2) {} //ignore
      @Override
      public void afterTextChanged(Editable editable) {
        latitude = getDouble(editable);
      }
    };
  }

  TextWatcher getLongitudeTextWatcher() {
    return new TextWatcher() {
      @Override
      public void beforeTextChanged(CharSequence charSequence, int i, int i1, int i2) {} //ignore
      @Override
      public void onTextChanged(CharSequence charSequence, int i, int i1, int i2) {} //ignore
      @Override
      public void afterTextChanged(Editable editable) {
        longitude = getDouble(editable);
      }
    };
  }
}
```

前提・実現したいこと

該当のソースコード

関連した質問