на кнопки не работают в андроид студии - PullRequest
Новая
       9

на кнопки не работают в андроид студии

0 голосов
/ 01 октября 2018

для приложения, которое я создаю, оно не позволяет мне указать, что делать с кнопкой, и вместо этого выдает ошибки.Приложение помогает переключаться между тремя музыкальными песнями.

код не работает 

    package com.example.atvan.babysleeper;

import android.media.MediaPlayer;
import android.os.Handler;
import android.os.Message;
import android.provider.MediaStore;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.SeekBar;
import android.widget.TextView;

import com.example.atvan.babysleep.R;

public class MainActivity extends AppCompatActivity {

    Button playBtn;
    Button one;
    Button two;
    Button three;
    SeekBar positionBar;
    SeekBar volumeBar;
    TextView elapsedTimeLabel;
    TextView remainingTimelabel;
    MediaPlayer mp;
    int totalTime;

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

        playBtn = (Button) findViewById(R.id.playBtn);
        elapsedTimeLabel = (TextView) findViewById(R.id.elapsedTimeLabel);
        remainingTimelabel = (TextView) findViewById(R.id.remainingTimeLabel);
        one = (Button) findViewById(R.id.one);
        two = (Button) findViewById(R.id.two);
        three = (Button) findViewById(R.id.three);


        //Media player

        public void setOne(View view){
            mp = MediaPlayer.create(this, R.raw.music3);
        }
        public void setTwo(View view){
            mp = MediaPlayer.create(this, R.raw.music2);
        }
        public void setThree(View view){
            mp = MediaPlayer.create(this, R.raw.music1);

        }


        mp.setLooping(true);
        mp.seekTo(0);
        mp.setVolume(0.5f,0.5f);
        totalTime = mp.getDuration();

        //position Bar
        positionBar = (SeekBar) findViewById(R.id.positionBar);
        positionBar.setMax(totalTime);
        positionBar.setOnSeekBarChangeListener(
                new SeekBar.OnSeekBarChangeListener() {
                    @Override
                    public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                        if (fromUser) {
                            mp.seekTo(progress);
                            positionBar.setProgress(progress);
                        }
                    }

                    @Override
                    public void onStartTrackingTouch(SeekBar seekBar) {

                    }

                    @Override
                    public void onStopTrackingTouch(SeekBar seekBar) {

                    }
                }
        );

        //Volume Bar
        volumeBar = (SeekBar) findViewById(R.id.volumeBar);
        volumeBar.setOnSeekBarChangeListener(
                new SeekBar.OnSeekBarChangeListener() {
                    @Override
                    public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                        float volumeNum = progress/100f;
                        mp.setVolume(volumeNum, volumeNum);
                    }

                    @Override
                    public void onStartTrackingTouch(SeekBar seekBar) {

                    }

                    @Override
                    public void onStopTrackingTouch(SeekBar seekBar) {

                    }
                }
        );

        //thread (update positionBar amd timeLabel)
        new Thread(new Runnable() {
            @Override
            public void run() {
                while (mp != null) {
                    try{
                        Message msg = new Message();
                        msg.what = mp.getCurrentPosition();
                        handler.sendMessage(msg);
                        Thread.sleep(1000);
                    }catch (InterruptedException e){}
                }
            }
        }).start();



    }

    private Handler handler;

    {
        handler = new Handler(new Handler.Callback() {
            @Override
            public boolean handleMessage(Message msg) {
                int currentPosition = msg.what;
                // Update positionBar.
                positionBar.setProgress(currentPosition);

                // Update Labels.
                String elapsedTime = createTimeLabel(currentPosition);
                elapsedTimeLabel.setText(elapsedTime);

                String remainingTime = "- " + createTimeLabel(totalTime - currentPosition);
                remainingTimelabel.setText(remainingTime);

                return true;
            }
        });
    }

    public String createTimeLabel(int time) {
        String timeLabel = "";
        int min = time / 1000 / 60;
        int sec = time /1000 % 60;

        timeLabel = min + ":";
        if (sec<10) timeLabel += "0";
        timeLabel += sec;

        return timeLabel;
    }

    public void playBtnClick(View view) {
        if (!mp.isPlaying()){
            //stopping
            mp.start();
            playBtn.setBackgroundResource(R.drawable.stop);
        } else {
            //playing
            mp.pause();
            playBtn.setBackgroundResource(R.drawable.play);

        }
    }

ошибка, которую он выдает, состоит в том, что у меня неправильное начало выражения в разделе моего медиаплеера.это также указывает, чтобы дать;где-то тампри размещении в конце кода.он не выдает никаких ошибок, но приложение не запускается

ошибка здесь

the error

enter image description here

Ответы [ 3 ]

0 голосов
/ 01 октября 2018

у вас есть все 3 метода, определенные внутри onCreate ().Разместите их снаружи на Создайте и звоните им откуда угодно.если по умолчанию вы хотите setOne (), тогда просто вызовите setOne () в onCreate ().

// Media player 

    public void setOne(View view){
        mp = MediaPlayer.create(this, R.raw.music3);
    }
    public void setTwo(View view){
        mp = MediaPlayer.create(this, R.raw.music2);
    }
    public void setThree(View view){
        mp = MediaPlayer.create(this, R.raw.music1);

    }

что-то вроде 

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

    playBtn = (Button) findViewById(R.id.playBtn);
    elapsedTimeLabel = (TextView) findViewById(R.id.elapsedTimeLabel);
    remainingTimelabel = (TextView) findViewById(R.id.remainingTimeLabel);
    one = (Button) findViewById(R.id.one);
    two = (Button) findViewById(R.id.two);
    three = (Button) findViewById(R.id.three);

    //call setOne or setTwo or setThree based on what you want here.
0 голосов
/ 01 октября 2018

Здесь есть несколько логических ошибок.Во-первых, вы не можете написать метод внутри другого метода, поэтому удалите следующие строки кода из onCreate() и напишите его снаружи: 

public void setOne(View view){ mp = MediaPlayer.create(this, R.raw.music3); } 
public void setTwo(View view){ mp = MediaPlayer.create(this, R.raw.music2); } 
public void setThree(View view){ mp = MediaPlayer.create(this, R.raw.music1); }

Во-вторых, вы инициализируете свой mediaPlayer внутри этих методов, так чтоВы должны сделать это первоначально, вызывая метод setOne() из onCreate() и вызывая другие методы после окончания текущей воспроизводимой песни.

Пожалуйста, пройдите этот учебник, хотя он показывает "Построениемедиапроигрыватель в службе "но вы можете сделать это в своей деятельности и для вас" Шаг третий - создание MediaPlayer"очень важно.

0 голосов
/ 01 октября 2018

Вы не можете просто определить функции в середине функции.Они должны быть в области видимости, вне onCreate.

...