0

私はJavaに本当に慣れていません.Androidは初めてで、すべてがMainActivityクラスにあるときに完全に正常に動作するコードをいくつかまとめました。しかし今は、関数をクリーンアップして独自のクラスに配置し、コードをきれいに見せたいと考えています。これが私がやろうとしていることの例です。IDE でエラーが発生していませんが、MainActivity が setArtist および setAlbum 関数を実行しようとすると、デバイスがクラッシュします。

public class MainActivity extends Activity {

private ListView lv;
String path, selectedFromList;
ArrayAdapter<String> adapter;
public ViewFiller k = new ViewFiller();

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

    lv = (ListView) findViewById(R.id.listView1);

    k.setArtist();
    k.setAlbums();

    adapter = new ArrayAdapter<String>(this,android.R.layout.simple_list_item_1, k.getAlbums());
    lv.setAdapter(adapter);

    ImageButton albumButton = (ImageButton) findViewById(R.id.albumButton);
    ImageButton artistButton = (ImageButton) findViewById(R.id.artistButton);

    albumButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            k.album(lv, adapter);
        }
    });

    artistButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            k.artist(lv, adapter);
        }
    });

}

}

そして、私が問題を抱えているクラスはこちらです...

public class ViewFiller extends MainActivity  {

public ViewFiller(){
    }

ArrayList<String> Albums = new ArrayList<String>();
ArrayList<String> Artist = new ArrayList<String>();
LinkedHashSet<String> hs = new LinkedHashSet<String>();

public ArrayList<String> getAlbums() {
    return Albums;
}

public void setAlbums() {

    String selection = AudioColumns.IS_MUSIC + " != 0";
    String[] projection = { BaseColumns._ID, AudioColumns.ALBUM };
    Cursor c = this.getContentResolver().query(
            MediaStore.Audio.Media.EXTERNAL_CONTENT_URI, projection,
            selection, null, null);

    while (c.moveToNext()) {
        Albums.add(c.getString(1));
    }
    hs.clear();
    hs.addAll(Albums);
    Albums.clear();
    Albums.addAll(hs);
}


public ArrayList<String> getArtist() {
    return Artist;
}

public void setArtist() {

    String selection = AudioColumns.IS_MUSIC + " != 0";
    String[] projection = { BaseColumns._ID, AudioColumns.ARTIST, };
    Cursor c = this.getContentResolver().query(
            MediaStore.Audio.Media.EXTERNAL_CONTENT_URI, projection,
            selection, null, null);

    while (c.moveToNext()) {
        Artist.add(c.getString(1));
    }

    hs.clear();
    hs.addAll(Artist);
    Artist.clear();
    Artist.addAll(hs);
}

public void artist(ListView lv, ArrayAdapter<String> s) {
    lv.setAdapter(null);
    s = new ArrayAdapter<String>(this,
            android.R.layout.simple_list_item_1, Artist);
    lv.setAdapter(s);
}

public void album(ListView lv, ArrayAdapter<String> s) {
    lv.setAdapter(null);
    s = new ArrayAdapter<String>(this,
            android.R.layout.simple_list_item_1, Albums);
    lv.setAdapter(s);

}

}

4

1 に答える 1