質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.63%

Kotlinで作ったAndroidアプリが落ちる

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 607

matsumo2002

score 11

問題

作成したメモアプリが唐突に落ちます。
メモを保存してHOMEに帰ってきたとき(MainActivityに復帰したとき)落ちることが多いです。
アプリを再起動して同じ操作をもう一度行うと、上手く行ったり、また他のところで落ちてしまったりと、不安定な状態です。
どのタイミングで落ちるのかわからないので、どこがおかしいのかわからず、手も足も出ません...。
下にコードを載せますので、おかしいところをご指摘していただけるとありがたいです。

参考にしたサイト

ここのサイトを参考に作っています。データの保存に関しては自分でやってみたかったので、自分で実装しています。ここが怪しいかもしれません。

コード

MainActivity.kt

package com.example.simplememo

import android.content.DialogInterface
import android.content.Intent
import android.support.v7.app.AppCompatActivity
import android.os.Bundle
import android.support.v7.app.AlertDialog
import android.widget.ArrayAdapter
import android.widget.Toast
import kotlinx.android.synthetic.main.activity_main.*
import java.io.BufferedReader
import java.io.File
import java.io.IOException

class MainActivity : AppCompatActivity() {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        var map = StartUp()

        HomeMemoList.setOnItemClickListener { parent, view, position, id ->
            map = StartUp()

            var Title = map[position+1]
            Toast.makeText(this,"編集:${Title}",Toast.LENGTH_SHORT).show()

            val intent= Intent(this,CreateMemoActivity::class.java)
            intent.putExtra("Title",Title)
            startActivity(intent)
        }

        HomeMemoList.setOnItemLongClickListener { parent, view, position, id ->
            var Title = map[position+1]
            AlertDialog.Builder(this)
                .setTitle("削除")
                .setMessage("このメモを削除しますか?")
                .setPositiveButton("はい",DialogInterface.OnClickListener { dialog, which ->
                    try {
                        val del = File(Title)
                        del.delete()
                        CountDownFile()
                        Toast.makeText(this,"${Title}を削除しました",Toast.LENGTH_SHORT).show()
                        map = StartUp()
                    }
                    catch (e:IOException){
                        Toast.makeText(this,"ERROR:${Title}を削除できません\n${e}",Toast.LENGTH_SHORT).show()
                        map = StartUp()
                    }
                })
                .setNegativeButton("いいえ",null)
                .show()
            return@setOnItemLongClickListener true
        }

        NewCreateButton.setOnClickListener {
            val intent= Intent(this,CreateMemoActivity::class.java)
            intent.putExtra("Title","新しいメモ")
            startActivity(intent)
        }
    }

    override fun onResume() {
        super.onResume()

        StartUp()
    }

    fun CountMemo():Int{
        val readFile = File(applicationContext.filesDir,"SimpleMemoCount.txt")
        var count:Int
        if(readFile.exists()){
            count=readFile.bufferedReader().use(BufferedReader::read)
        }
        else count=0
        return count
    }

    fun CountAllMemo():Int{
        val readFile = File(applicationContext.filesDir,"SimpleMemoAllCount.txt")
        var count:Int
        if(readFile.exists()){
            count=readFile.bufferedReader().use(BufferedReader::read)
        }
        else count=0
        return count
    }

    fun CountDownFile() {
        val file="SimpleMemoCount.txt"
        var Count:Int
        val readFile= File(applicationContext.filesDir, file)

        if(readFile.exists()) {
            Count=readFile.bufferedReader().use(BufferedReader::read)
            Count-=1
        }
        else Count=1

        File(applicationContext.filesDir,file).writer().use{
            it.write(Count)
        }
    }

    fun StartUp():Map<Int,String>{
        val count = CountMemo()
        val allcount = CountAllMemo()
        var memobody: ArrayList<String?> = arrayListOf()
        var Title:String
        var map= mapOf<Int,String>()
        val mutable=map.toMutableMap()

        if(count!=0){
            var flag=0
            for(i in 0..allcount){
                Title = "SimpleMemo" + (i+1) + ".txt"
                val readFile=File(application.filesDir,Title)
                if(readFile.exists()){
                    memobody.add(readFile.bufferedReader().use(BufferedReader::readText))
                    flag++
                    mutable += flag to Title
                }
            }
        }

        val adapter=ArrayAdapter<String>(this,android.R.layout.simple_list_item_1,memobody)
        HomeMemoList.adapter=adapter

        return mutable
    }
}


CreateMemoActivity.kt

package com.example.simplememo

import android.support.v7.app.AppCompatActivity
import android.os.Bundle
import android.support.v7.app.AlertDialog
import android.view.MenuItem
import android.widget.TextView
import android.widget.Toast
import kotlinx.android.synthetic.main.activity_create_memo.*
import java.io.BufferedReader
import java.io.File

class CreateMemoActivity : AppCompatActivity() {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_create_memo)

        var Title:String=intent.getStringExtra("Title")

        if(Title!="新しいメモ")LordMemo(Title)

        supportActionBar?.setDisplayHomeAsUpEnabled(true)
        supportActionBar?.title=Title

        register.setOnClickListener {
            if(Title=="新しいメモ")Title = CountUpFile()
            File(applicationContext.filesDir,Title).writer().use{
                it.write(body.text.toString())
            }
            Toast.makeText(applicationContext,"${Title}を保存しました",Toast.LENGTH_SHORT).show()
            finish()
        }
    }

    override fun onOptionsItemSelected(item: MenuItem?): Boolean {
        when(item?.itemId){
            android.R.id.home->finish()
            else -> {
                return super.onOptionsItemSelected(item)
            }
        }
        return true
    }

    fun CountUpFile(): String {
        val file="SimpleMemoCount.txt"
        var Count:Int
        val readFile= File(applicationContext.filesDir, file)

        if(readFile.exists()) {
            Count=readFile.bufferedReader().use(BufferedReader::read)
            Count+=1
        }
        else Count=1

        File(applicationContext.filesDir,file).writer().use{
            it.write(Count)
        }

        val file2="SimpleMemoAllCount.txt"
        var Count2:Int
        val readFile2= File(applicationContext.filesDir, file2)

        if(readFile2.exists()) {
            Count2=readFile2.bufferedReader().use(BufferedReader::read)
            Count2+=1
        }
        else Count2=1

        File(applicationContext.filesDir,file2).writer().use{
            it.write(Count2)
        }

        return "SimpleMemo"+Count2+".txt"
    }

    fun LordMemo(Title:String){
        val readFile=File(applicationContext.filesDir,Title)
        if(readFile.exists()){
            var text=readFile.bufferedReader().use(BufferedReader::readText)
            body.setText(text, TextView.BufferType.NORMAL)
        }
        else {
            AlertDialog.Builder(this)
                .setTitle("読み込みエラー")
                .setMessage("指定されたファイルが見つかりませんでした\nMemo:${Title}")
                .setPositiveButton("OK",null)
                .show()
            finish()
        }
    }
}

エラーログ(追記)

2019-06-12 00:35:16.647 5223-5223/com.example.simplememo E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.example.simplememo, PID: 5223
    java.lang.NullPointerException
        at java.io.File.<init>(File.java:282)
        at com.example.simplememo.MainActivity$onCreate$2$1.onClick(MainActivity.kt:41)
        at android.support.v7.app.AlertController$ButtonHandler.handleMessage(AlertController.java:167)
        at android.os.Handler.dispatchMessage(Handler.java:105)
        at android.os.Looper.loop(Looper.java:251)
        at android.app.ActivityThread.main(ActivityThread.java:6589)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:767)


MainActivity.kt:41はval del = File(Title)の部分でした

回答よろしくおねがいします

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2019/06/11 23:34

    落ちるってことはエラーログが出てますよね。それも載せた方が回答つきますよ。

    キャンセル

  • matsumo2002

    2019/06/12 00:39

    指摘ありがとうございます。
    エラーを追記しました

    キャンセル

  • Akashic

    2019/06/12 04:17

    なんどか動かしてみましたが、再現できませんでした。

    レイアウトのXMLも貼ってもらえますか?

    あと、初期化(キャッシュを消して、登録されているファイルが無い状態)から、発生時までの操作も詳しく書いてもらえますか(毎回発生ではないとのことなので、難しいかもしれませんが..)

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/06/12 13:05

    NullPointerExceptionならif(Title == null)returnとか入れればとりあえず落ちなくなりそうですね。Titleがnullになっていることがまずいならだめですが。

    キャンセル

回答 1

check解決した方法

0

回答者様の言うとおりにすれば治りました

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.63%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る