Czemu to niedziała ?

[DrexonPl]

tak jak w temacie

 

aplikacja na androida:

package com.example.drexplayer;

import java.io.IOException;
import android.media.MediaPlayer;
import android.os.Bundle;
import android.view.View;
import android.widget.AdapterView;
import android.widget.AdapterView.OnItemClickListener;
import android.widget.ArrayAdapter;
import android.widget.ListView;
import android.app.Activity;

public class MusicInternet extends Activity implements OnItemClickListener {
    
    MediaPlayer mediaPlayer;
    ListView l;
    
    String musicName[]={
            "Avicii - Levels",
            "Kamil Bednarek - Cisza",
            "LMFAO - Sorry for Party Rocking"
            };

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_music_internet);
        
        ArrayAdapter<String> adapter=new ArrayAdapter<String>(getApplicationContext(), android.R.layout.simple_list_item_1, musicName);
        l = (ListView) findViewById(R.id.listView1);
        l.setAdapter(adapter);
          
        l.setOnItemClickListener(this);
    }

    @Override
    public void onItemClick(AdapterView<?> adapter, View arg1, int position, long arg3) {
        String item = adapter.getItemAtPosition(position).toString();
        
        if(item == "Avicii - Levels") {
            try {
                mediaPlayer.setDataSource("http://drex-radio.cba.pl/LMFAO%20-%20Sorry%20for%20Party%20Rocking.mp3");
            } catch (IllegalArgumentException e) {
                e.printStackTrace();
            } catch (SecurityException e) {
                e.printStackTrace();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            try {
                mediaPlayer.prepare();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            mediaPlayer.start();
        }
        
        if(item == "Kamil Bednarek - Cisza") {
            try {
                mediaPlayer.setDataSource("http://drex-radio.cba.pl/Kamil%20Bednarek%20-%20Cisza.mp3");
            } catch (IllegalArgumentException e) {
                e.printStackTrace();
            } catch (SecurityException e) {
                e.printStackTrace();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            try {
                mediaPlayer.prepare();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            mediaPlayer.start();
        }
        
        if(item == "LMFAO - Sorry for Party Rocking") {
            try {
                mediaPlayer.setDataSource("http://drex-radio.cba.pl/Avicii%20-%20Levels.mp3");
            } catch (IllegalArgumentException e) {
                e.printStackTrace();
            } catch (SecurityException e) {
                e.printStackTrace();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            try {
                mediaPlayer.prepare();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            mediaPlayer.start();
        }
    }
}

ps: proszę o poprawienie mojego tematu(z "niedziała" na "nie działa")

[Jaca777]

Strasznie nieczytelny kod:

Po 1 - zamiast pierdyliardów instrukcji warunkowych, używaj switch.

Po 2 - zamiast pierdyliardów trykaczów, używaj poprostu catch(Exception e);

Po 3 - Nie pisz kodu wymagającego wykonananie poprzedniej metody, w polu poza blokiem trykacza, w którym się ta metoda wykonuje.

Kiedy to poprawisz, znacznie ułatwisz nam zadanie.

 

@Edit Problem jest taki - adapter.getItemAtPosition(position).toString(); najpewniej nie jest przeciążane, i nie zwraca Ci treści tego, co jest w tym obiekcie na liście.

[DrexonPl]

#up

dzięki, ale problem nadal nie rozwiązany.

[Jaca777]

 

@Edit Problem jest taki - adapter.getItemAtPosition(position).toString(); najpewniej nie jest przeciążane, i nie zwraca Ci treści tego, co jest w tym obiekcie na liście.

Czyli musisz poszukać innej metody, niż toString.

[DrexonPl]

dobra problem rozwiązany wszystko działa !!

 

zostawie tutaj naprawiony fragment:

 

            Uri uri = Uri.parse("http://drex-radio.cba.pl/Kamil%20Bednarek%20-%20Cisza.mp3");
            mediaPlayer.setAudioStreamType(AudioManager.STREAM_MUSIC);
            try {
                mediaPlayer.setDataSource(this, uri);
            } catch (IllegalArgumentException e1) {
                e1.printStackTrace();
            } catch (SecurityException e1) {
                e1.printStackTrace();
            } catch (IllegalStateException e1) {
                e1.printStackTrace();
            } catch (IOException e1) {
                e1.printStackTrace();
            }
            try {
                mediaPlayer.prepare();
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            }
            mediaPlayer.start();

[Gość]

Jeszcze tak ode mnie : Stringi porwnujemy za pomoc .equals a nie ==