Caused by: java.lang.NullPointerException: Attempt to・・・のエラーを解決方法を教えてください！

Question

質問

下に載せてあるコードを実行すると、以下のエラーが表示されました。
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'double android.location.Location.getLatitude()' on a null object reference
at com.myapplication.MainActivity.<init>(MainActivity.java:25)
載せてあるプログラムの説明を簡単にすると、
１、地図の画像をImageViewを用いて張り付ける。
２、その画像の左上、右下の座標をもとにGPS情報を用いて、自分の現在地が表示されるようにdpと緯度経度の計算式を作る。
３、GPS情報の取得
４、計算式、GPS情報から自分の現在地をマーカーのImageViewを貼り付ける。
というプログラムです。

調べてみると、おそらくnullというコードが関係していると思われるのですが、解決方法がわかりません。
プログラム自体にエラーは出ていないのですが、実行すると実機、エミュレーターともにアプリが落ちてしまいます。
どのようにすればエラーがなくなる可能性があるか、教えていただけないでしょうか。

import android.support.annotation.NonNull;
import android.support.v4.content.ContextCompat;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.content.pm.PackageManager;
import android.location.Location;
import android.location.LocationListener;
import android.location.LocationManager;
import android.location.LocationProvider;
import android.support.v4.app.ActivityCompat;
import android.widget.ImageView;
import android.widget.TextView;
import android.content.Intent;
import android.provider.Settings;
import android.util.Log;
import android.widget.Toast;
import android.Manifest;

public class MainActivity extends AppCompatActivity implements LocationListener{

    private Location location1;

    double Lati =location1.getLatitude();
    double Longi =location1.getLongitude();

    double a=340;
    double b=33.104001;
    double c=131.303325;
    double d=300;
    double e=0.001823;
    double f=0.002557;
    double g= location1.getLatitude();
    double h= location1.getLongitude();
    double i=b - g;
    double j=h - c;
    double k=a * i / e;
    double l=d * j / f;


    LocationManager locationManager;

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

        ImageView imageView =findViewById(R.id.image_view);
        imageView.setTranslationX((float) k);
        imageView.setTranslationY((float) l);


        if (ContextCompat.checkSelfPermission(this,
                Manifest.permission.ACCESS_FINE_LOCATION) !=
                PackageManager.PERMISSION_GRANTED) {
            ActivityCompat.requestPermissions(this,
                    new String[]{Manifest.permission.ACCESS_FINE_LOCATION,},
                    1000);
        } else {
            locationStart();

            locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER,
                    1000, 50, this);

        }
    }

    private void locationStart() {
        Log.d("debug", "locationStart()");

        // LocationManager インスタンス生成
        locationManager =
                (LocationManager) getSystemService(LOCATION_SERVICE);

        if (locationManager != null && locationManager.isProviderEnabled(LocationManager.GPS_PROVIDER)) {
            Log.d("debug", "location manager Enabled");
        } else {
            // GPSを設定するように促す
            Intent settingsIntent =
                    new Intent(Settings.ACTION_LOCATION_SOURCE_SETTINGS);
            startActivity(settingsIntent);
            Log.d("debug", "not gpsEnable, startActivity");
        }

        if (ContextCompat.checkSelfPermission(this,
                Manifest.permission.ACCESS_FINE_LOCATION) !=
                PackageManager.PERMISSION_GRANTED) {
            ActivityCompat.requestPermissions(this,
                    new String[]{Manifest.permission.ACCESS_FINE_LOCATION,}, 1000);

            Log.d("debug", "checkSelfPermission false");
            return;
        }

        locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER,
                1000, 50, this);

    }

    // 結果の受け取り

    /**
     * Android Quickstart:
     * https://developers.google.com/sheets/api/quickstart/android
     * <p>
     * Respond to requests for permissions at runtime for API 23 and above.
     *
     * @param requestCode  The request code passed in
     *                     requestPermissions(android.app.Activity, String, int, String[])
     * @param permissions  The requested permissions. Never null.
     * @param grantResults The grant results for the corresponding permissions
     *                     which is either PERMISSION_GRANTED or PERMISSION_DENIED. Never null.
     */
    @Override
    public void onRequestPermissionsResult(
            int requestCode, @NonNull String[] permissions, @NonNull int[] grantResults) {
        if (requestCode == 1000) {
            // 使用が許可された
            if (grantResults[0] == PackageManager.PERMISSION_GRANTED) {
                Log.d("debug", "checkSelfPermission true");

                locationStart();

            } else {
                // それでも拒否された時の対応
                Toast toast = Toast.makeText(this,
                        "これ以上なにもできません", Toast.LENGTH_SHORT);
                toast.show();
            }
        }
    }

    @Override
    public void onStatusChanged(String provider, int status, Bundle extras) {
        switch (status) {
            case LocationProvider.AVAILABLE:
                Log.d("debug", "LocationProvider.AVAILABLE");
                break;
            case LocationProvider.OUT_OF_SERVICE:
                Log.d("debug", "LocationProvider.OUT_OF_SERVICE");
                break;
            case LocationProvider.TEMPORARILY_UNAVAILABLE:
                Log.d("debug", "LocationProvider.TEMPORARILY_UNAVAILABLE");
                break;
        }
    }

    @Override
    public void onLocationChanged(Location location) {

        location1 = location;

    }

    @Override
    public void onProviderEnabled(String provider) {

    }

    @Override
    public void onProviderDisabled(String provider) {

    }
}

<?xml version="1.0" encoding="utf-8"?>
<android.support.constraint.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">

    <FrameLayout
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        >


        <ImageView
            android:layout_width="340dp"
            android:layout_height="300dp"
            android:src="@drawable/tizu4"
            app:layout_constraintEnd_toEndOf="parent"
            app:layout_constraintLeft_toLeftOf="parent"
            app:layout_constraintRight_toRightOf="parent"
            app:layout_constraintStart_toStartOf="parent"
            app:layout_constraintTop_toTopOf="parent" />


        <ImageView
            android:layout_width="15dp"
            android:layout_height="15dp"
            android:id="@+id/image_view"
            android:src="@drawable/azaz"
            />

    </FrameLayout>
</android.support.constraint.ConstraintLayout>

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.sousaku1.nakahara.myapplication">

    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>

    <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>

Comments

[Stan_Dma]

I'm afraid the variable 'location1' has no value set on your code.

[m.ts10806]

タイトルには要件を書いてください。

[sen10]

修正、アドバイスありがとうございます。
次回から要件を書くようにしたいと思います。

[sen10]

Thank you for your answer.
I would like to refer to your answer.

[m.ts10806]

質問は編集できるので適宜ご対応いただけたらと

[sen10]

返信ありがとうございます。
質問の編集、できるんですね！参考になりました！

[m.ts10806]

はい。そのためのこの「追記、修正依頼」です。

[m.ts10806]

別件で、質問者さんが良ければそれで良いですが、エラー文のパッケージ名に名前らしきものが含まれた文言が出力されています。これは問題ないですか？
もし問題ありでしたら質問削除依頼を運営に直接だしてください(回答がついた質問は質問から削除依頼が出せないのと、編集では編集履歴から確認できるので)

[sen10]

返信ありがとうございます。
本当ですね。あまり問題はないですが、一応編集だけしておきたいと思います。
助かります！

Answer 1

    private Location location1;

    double Lati =location1.getLatitude();
    double Longi =location1.getLongitude();

    double a=340;
    double b=33.104001;
    double c=131.303325;
    double d=300;
    double e=0.001823;
    double f=0.002557;
    double g= location1.getLatitude();
    double h= location1.getLongitude();
    double i=b - g;
    double j=h - c;
    double k=a * i / e;
    double l=d * j / f;

クラスのメンバ変数は、そのクラスのインスタンス生成時に初期化されます。
location1は初期化宣言が無いため、自動的にnullに初期化されます。
この状態でLatiやLongiの初期化をlocation1のメソッドを使って初期化しようとしましたが、
location1はnullなので、NullPointerExceptionで落ちます。
なのでここでそのような初期化は不可能です。

Comments

[sen10]

回答ありがとうございます。今、このように書き換えてみたのですが、やはりエラーがでてしまいます。
public class MainActivity extends AppCompatActivity implements LocationListener{

private Location location1;

//double Lati = location1.getLatitude();
//double Longi = location1.getLongitude();
double Lati;
double Longi;

/* double a=360;
double b=33.098;
double c=131.2063;
double d=225;
double e=0.0217;
double f=0.041;
double g=location1.getLatitude();
double h=location1.getLongitude();
double g=location1.getLatitude();
double h=location1.getLongitude();
double i=b - g;
double j=h - c;
double k=a * i / e;
double l=d * j / f;*/

LocationManager locationManager;

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

if (ContextCompat.checkSelfPermission(this,
Manifest.permission.ACCESS_FINE_LOCATION) !=
PackageManager.PERMISSION_GRANTED) {
ActivityCompat.requestPermissions(this,
new String[]{Manifest.permission.ACCESS_FINE_LOCATION,},
1000);
} else {
locationStart();

locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER,
1000, 50, this);

}
double a=360;
double b=33.098;
double c=131.2063;
double d=225;
double e=0.0217;
double f=0.041;
double g=location1.getLatitude();
double h=location1.getLongitude();
double i=b - g;
double j=h - c;
double k=a * i / e;
double l=d * j / f;

ImageView imageView =findViewById(R.id.image_view);
imageView.setTranslationX((float) k);
imageView.setTranslationY((float) l);

}

エラーの原因は簡単に言うと、計算式のlocation1.getLatitude();に数値が反映されていないことなのでしょうか。どのようにコードをかけばエラーがとれますか？

[sen10]

解決することができました！
swordoneさんのアドバイスのおかげでエラーの原因がわかりました！
本当にありがとうございました！！

Answer 2

エラーの原因を簡単に言うと、プログラムの流れ的には

    Location location1 = null;
    double g = location1.getLatitude();
    double h = location1.getLongitude();

と同じことになっているということです。location1がnullですから、これを参照すればNullPointerExceptionです。計算式に数値が反映されていないとか、そういった難しい要因は一切存在しません。

Location型の値を得ることができるのは、LocationManager#requestLocationUpdates()によってセットしたリスナーが呼び出されたときでです。

    @Override
    public void onLocationChanged(Location location) {

        location1 = location;

        double a = 360;
        double b = 33.098;
        double c = 131.2063;
        double d = 225;
        double e = 0.0217;
        double f = 0.041;
        double g = location1.getLatitude();
        double h = location1.getLongitude();
        double i = b - g;
        double j = h - c;
        double k = a * i / e;
        double l = d * j / f;


        ImageView imageView = findViewById(R.id.image_view);
        imageView.setTranslationX((float) k);
        imageView.setTranslationY((float) l);
    }

のように、onLocationChanged()の引数として渡されたLocation型を用いれば解決するのではないでしょうか。

Comments

[sen10]

回答ありがとうございます！
先程、私もその方法で解決することができました。質問を読んでいただき本当にありがとうございます！