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

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

新規登録して質問してみよう
ただいま回答率
85.50%
SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

Q&A

解決済

3回答

2116閲覧

Listにデータを適切に追加できない

hkCgIVNAqvin1AQ

総合スコア14

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

0グッド

0クリップ

投稿2017/12/31 04:36

SQLiteからインスタンスへデータを戻し
それをリストにセットしようと考えています。

リストが存在しない場合は、リストを作り
存在する場合は、存在するリストに直接インスタンスを
入れたいと考えています。
if (list != null && list.size() != 0) {}を付けなかったときは
常にリストサイズが1となり、登録データが一つしか
取り出せません。

if (list != null && list.size() != 0) {}を付けた場合は
Caused by: java.lang.NullPointerException: Attempt to invoke interface method 'boolean java.util.List.add(java.lang.Object)' on a null object reference
となりアプリが落ちます。

どのようにすれば、アプリは落ちなくなるでしょうか
教えてください。よろしくお願いします。

プログラム
if (cursor.moveToNext()) {
MapDiary mapDiary = new MapDiary();
mapDiary.setLatitude(cursor.getDouble(cursor.getColumnIndex("Latitude"))); //緯度
mapDiary.setLongitude(cursor.getDouble(cursor.getColumnIndex("Longitude"))); //経度

#####################
if (list != null && list.size() != 0) {
this.list = new ArrayList();
}
#####################

this.list.add(mapDiary);
Log.d("ListSize", "ListSize"+this.list.size());
}

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

swordone

2017/12/31 07:21

addしているところがないのにNPEでaddで落ちていたり、情報が足りなさすぎます。コード全体を掲載してください。
guest

回答3

0

これはwhile文等の繰り返し構文の中に記載されていますか?
もしそうでないなら最初の1行しかデータは取れません。

また、クイズをしているわけではないので、「ダメです」とか「できません」ではなく、具体的にどうなったかをコメントしてください。
さらに、コードを記載するときはもっと大きく抜粋してください。

投稿2017/12/31 07:37

yona

総合スコア18155

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

if(cursor.moveToNext())の前にif(list == null || list.size() != 0){this.list = new ArrayList();}を追加
if(cursor.moveToNext())while(cursor.moveToNext())に変更
if (list != null && list.size() != 0) を削除
質問から読み取れる範囲だとこれしか回答できないです。

質問文にコードを書くときは読みやすくするため、バッククォート(Shift+@)三つで囲ってください。または、質問文を入力する欄の上にある"</>"というボタンを押し、"コード"と書かれているところにコードを入力してください。

投稿2018/01/02 02:50

編集2018/01/02 03:12
moredeep

総合スコア1507

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2018/01/02 03:03

最後の一軒の実保持の意味なら list.size() != 0 は無駄な処理のきが・q・ # moveToLast() があるじゃない
moredeep

2018/01/02 03:13

失礼しました。listに値が入っていたらリセットしてから実行という意味かな?と思ったのですが、位置が悪いですね。少し変更してみました。
hkCgIVNAqvin1AQ

2018/01/02 10:37

if(cursor.moveToNext())をwhile(cursor.moveToNext())に変更が解決策でした。 言われてみれば確かにといった感じです! ありがとうございました。
guest

0

リストが存在しない場合は、リストを作り

なんだから、こうでは?

java

1if (list == null) { 2 list = new ArrayList(); 3}

投稿2017/12/31 04:45

hichon

総合スコア5737

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

hkCgIVNAqvin1AQ

2017/12/31 05:49

すみません、それではダメでした
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問