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

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

ただいまの
回答率

90.53%

  • Java

    13745questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6504questions

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

  • Android Studio

    3688questions

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

Listviewの中身を初期化したい

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 1,140

rain_sun

score 4

前提・実現したいこと

Listviewの初期化

発生している問題・エラーメッセージ

読み込み→戻る→読み込み
でListviewの中身が多重化してしまう

該当のソースコード

package 省略

import 省略

public class SqlLoad extends Activity
implements View.OnClickListener{
    ListView lv;
    static String[][] list;
    static String[] Did = new String[2];
    static long id_;
    static String[] Data;
    static List<DataList> Dlist = new ArrayList<>();
    static int sortflg=1;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.sql_load);
       // final boolean[] flg = new boolean[1];
        Button sort = (Button) findViewById(R.id.sort);
        sort.setOnClickListener(this);


        Main();
    }

    @Override
    public void onClick(View v) {
        switch (v.getId()) {
            case sort:
               // Dlist.sort();
                //Log.d("tt","a");
                Collections.sort(Dlist,new SortComparator());
                ListAdapter adapter = new ListAdapter(getApplicationContext(),Dlist);
                if (sortflg < 2){
                    sortflg++;
                }else sortflg = 1;
               // final Toast toast = Toast.makeText(this,"", Toast.LENGTH_SHORT);
                lv.setAdapter(adapter);
                break;
        }
    }


    public void Main(){
        Data = SQLDataGet();
       // lv = new ListView(this);
    //    ArrayList<String> Data = new ArrayList<>(Arrays.asList(SQLDataGet()));
        lv = (ListView)findViewById(R.id.sql_load_main);

        for (String[] aList : list) {
            DataList dataList = new DataList();
            dataList.setTitle(aList[0]);
            dataList.setContent(aList[1]);
            dataList.setDate(aList[2]);
            Dlist.add(dataList);
        }
       // ArrayAdapter<String> adapter = new ArrayAdapter<>(this, android.R.layout.simple_list_item_1, data);

        ListAdapter adapter = new ListAdapter(getApplicationContext(),Dlist);
        lv.setAdapter(adapter);//リスト表示
     //   setContentView(lv,new ViewGroup.LayoutParams(WC,WC));

        lv.setOnItemClickListener(new AdapterView.OnItemClickListener(){
            @Override
            public void onItemClick(AdapterView<?> parent,View view,int position,long id){
                //  ListView listView = (ListView) parent;
                //   String item = (String)listView.getItemAtPosition(position);
                Log.d("id",""+id);
                id_ = id;
                SQLDContentGet();
                SQLDataSet();
                finish();
                // Toast.makeText(getApplicationContext(),item + "clicked",Toast.LENGTH_LONG).show();
            }
        });
    }





    public String[] SQLDataGet(){
        MainActivity main = new MainActivity();
        Context con = this;
        SQLiteDatabase sdb = main.SQLopen(con.getApplicationContext());//MainActivity参照
        final String[] columns = new String[]{"title","content","date"};//列名
       // String where = null;//条件
      //  String[] Args = new String[]{""};//?の置き換え
        @SuppressLint("Recycle") Cursor c = sdb.query("MemoContents", columns, null, null, null, null, null);
        int startPostion = c.getPosition();
        c.moveToFirst();
       // Log.d("DBG",""+c.getCount());
        list = new String[c.getCount()][columns.length];
        String[] Dlist = new String[c.getCount()];
        for (int i = 0; i< list.length; i++){
            for (int j = 0; j < list[1].length; j++){
                list[i][j] = c.getString(j);
                Log.d(i+"-"+j,list[i][j]);//デバッグ
                Dlist[i] = list[i][0];
            }
            c.moveToNext();
        }
        c.moveToPosition(startPostion);

      //  finish();
        return Dlist;
    }

    public void SQLDContentGet(){
        Did[0] = list[(int)id_][0];
        Did[1] = list[(int)id_][1];
    }

    public void SQLDataSet(){
        Intent data = new Intent();
        Bundle bundle = new Bundle();
        data.putExtra("key.data",Did);
        data.putExtras(bundle);
        setResult(RESULT_OK,data);
        finish();
    }

    @Override
    public void finish() {
        super.finish();
    }
}

class ListAdapter extends ArrayAdapter<DataList>{
    private LayoutInflater inflater;

    ListAdapter(Context context, List<DataList> objects){
        super(context,0,objects);
        inflater = (LayoutInflater)context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    }

    @SuppressLint("InflateParams")
    @NonNull
    @Override
    public View getView(final int position, View convertView, @NonNull ViewGroup parent){
        if (convertView == null) {
            convertView = inflater.inflate(R.layout.custom_load_list, null);
        }
        final DataList list = this.getItem(position);
        if (list != null){
            TextView title = (TextView) convertView.findViewById(R.id.textView1);
            title.setText(list.getTitle());
            TextView content = (TextView) convertView.findViewById(R.id.textView2);
            content.setText(list.getContent());
            TextView date = (TextView) convertView.findViewById(R.id.textView3);
            date.setText(list.getDate());
        }
        return convertView;
    }
}

class SortComparator implements Comparator<DataList>{
    private int a=SqlLoad.sortflg;
    private int b;
    @Override
    public int compare(DataList d1, DataList d2) {
        switch (a){
            case 1:
               b = Long.parseLong(d1.date) > Long.parseLong(d2.date) ? -1 : 1;
            break;
            case 2:
                a=0;
                b = Long.parseLong(d1.date) < Long.parseLong(d2.date) ? -1 : 1;
            break;
        }
        Log.d("TestSort",""+a);
        return b;
    }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • yona

    2017/02/17 12:08

    多重化とは具体的にどのような状態ですか?

    キャンセル

  • rain_sun

    2017/02/17 12:17

    同じデータがListveiw内に複数配置されている状態です

    キャンセル

回答 4

checkベストアンサー

+1

// 表示するリスト要素をクリアする
Dlist.clear();

for (String[] aList : list) {
    DataList dataList = new DataList();
    dataList.setTitle(aList[0]);
    dataList.setContent(aList[1]);
    dataList.setDate(aList[2]);
    Dlist.add(dataList);
}
  1. clear()追加
  2. for文実行後のDlistの中身をデバックしてみてください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

安易にstaticを使ってはいけません、staticにしている意味はなんでしょうか?
特に理由が無いならフィールド変数のstaticを外すといいですよ。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/17 12:44

    同.java内の他のクラスに渡す為だと思います
    修正しておきました

    キャンセル

+1

BA後ですが一応。

無駄な変数や処理が多すぎです。
処理を変えていくのはいいですが、
適当なタイミングで最適化していかないと
化け物みたいなコードが出来上がるだけですよ。

見た感じでは、
各メソッドの引数、戻り値を見直すことで、
クラス(インスタンス)変数は3つにまで減らすことが出来ると思います。

ListView lv;
List<DataList> Dlist;
static int sortflg;


また、List<DataList>の代わりにListAdapterを保持すれば、
ListView用の変数も必要なくなります。
なぜかは、ArrayAdapterの使い方を調べてください。

あと、今後もここなどで、他人にコードを見せるのであれば、
「java 命名規約」で少し調べてみることをお勧めします。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/17 13:14

    命名規約なんてものがあったのですか…
    ありがとうございます

    キャンセル

  • 2017/02/17 13:52

    規約じゃなくて規則でしたね。
    まぁ、検索には引っかかってくれると思いますが。

    全てのコードが一定の規則に従って記述されている、というのは、
    読む側にとって、読みやすい、とてもありがたいことです。
    早めに癖をつけておきましょう。

    キャンセル

0

初期化すりゃいいんじゃないの?

Dlist = new ArrayList<>(); //初期化
for (String[] aList : list) {
    :
    Dlist.add(dataList);
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/17 12:04

    試したのですが
    Dlistを初期化するとListviewの中身が空になってしまいます

    キャンセル

  • 2017/02/17 12:44 編集

    下でaddしてるから空にならないんじゃ?

    キャンセル

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

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

関連した質問

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

  • Java

    13745questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6504questions

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

  • Android Studio

    3688questions

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