質問

私はインターネット上のこのコードを見つけました(タイマーを作成しようとしているから)。誰かがこのコードがIndexOutOfBoundsExceptionを投げていない理由を教えてください。

これはコードです:

hour = new int[30];
min = new int[30];
sec = new int[30];
msec = new int[30];
start = false;
stop = true;
for(int j = 0  ; j <= 30 ; j++)
{
    hour[j] = 0;
    min[j] = 0;
    sec[j] = 0;
    msec[j] = 0;
}
.

それはタイマータスクでレンダリングされます

    public void run()
    {
        msec[count]++;
        if(msec[count] == 100)
        {
            msec[count] = 0 ;
            sec[count]++;
        }
        else if(sec[count] ==60)
        {
            sec[count] = 0;
            min[count]++;
        }
        else if(min[count] == 60)
        {
            min[count] = 0;
            hour[count]++;
        }
        else if(hour[count] == 24)
        {
            hour[count] = 0;
        } 
        repaint();
    } 
};
timer = new Timer();
timer.scheduleAtFixedRate(task,10,67);    
.

IndexOutOfBoundsExceptionを投げていないのはなぜですか。それが30の値でインスタンス化されていて、そしてこれを実行しようとしているとき、そして私がこれを実行しようとしているときには、MSECで100のような30歳までの30歳までの30歳までの30歳までの30歳までの30枚のEXCEEEDS。

そしてこれがフルコードです:

 public class TimerCan extends Canvas
{
private Timer timer;
private Midlet myMid;
private Player z;
private int habaNgString,hour[],sec[],min[],msec[],maxX,maxY,count,length,x,y;
private String runningTime;
private boolean start,stop;
public Image img;
public TimerCan(Midlet midlet)
{
    this.myMid= midlet;
    try
    {
        maxX = getWidth();
        maxY = getHeight();
        count = 0;
        hour = new int[30];
        min = new int[30];
        sec = new int[30];
        msec = new int[30];
        start = false;
        stop = true;
        for(int j = 0  ; j <= 30 ; j++)
        {
            hour[j] = 0;
            min[j] = 0;
            sec[j] = 0;
            msec[j] = 0;
        }
    }catch(Exception e)
    {}
}
public void paint(Graphics g)
{
         if(hour[count] < 10)
        {
            runningTime = "0"+String.valueOf(hour[count])+":";
        }
        else
        {
            runningTime = String.valueOf(hour[count]) + ":";  
        }
        if(min[count] < 10)
        {
            runningTime = runningTime+"0"+String.valueOf(min[count]) + ":";
        }
        else
        {
            runningTime = runningTime+String.valueOf(min[count]) + ":";
        }
        if(sec[count] < 10)
        {
            runningTime = runningTime+"0"+String.valueOf(sec[count]) + ":";
        }
        else
        {
            runningTime = runningTime + String.valueOf(sec[count]) + ":";
        }
        if(msec[count] < 10)
        {
            runningTime = runningTime+"0"+String.valueOf(msec[count]);
        }
        else
        {
            runningTime = runningTime+String.valueOf(msec[count]);
        }

    try{
         img = Image.createImage("/picture/aa.png");
    }
    catch(Exception error){
    }
    x = getWidth()/2;
    y = getHeight()/2;
    g.setColor(63,155,191);
    g.fillRect(0,0,maxX, maxY);
    g.drawImage(img, x, y, Graphics.VCENTER|Graphics.HCENTER);
    g.setColor(0,0,0) ;                                               
    g.drawString(runningTime,maxX,maxY,Graphics.TOP|Graphics.LEFT);
}
private void startTimer()
{
    TimerTask task = new TimerTask()
    {
        public void run()
        {
           msec[count]++;
            if(msec[count] == 100)
            {
                msec[count] = 0 ;
                sec[count]++;
            }
            else if(sec[count] ==60)
            {

                sec[count] = 0;
                min[count]++;
            }
            else if(min[count] == 60)
            {
                min[count] = 0;

               hour[count]++;
            }
            else if(hour[count] == 24)
            {
                hour[count] = 0;
            } 
                   repaint();
        } 
    };
    timer = new Timer();
    timer.scheduleAtFixedRate(task,10,67);           
}
protected  void keyPressed(int keyCode)
{
    if(keyCode == Canvas.KEY_NUM1)
    {
        if(start == false)
        {
            start=true;
            stop=false;
        }
        else if(stop == false)
        {
            start = false ;
            stop = true ;
            timer.cancel();
        }
        if(start==true)
        {
            startTimer();
        }
    }
    if(keyCode == Canvas.KEY_NUM2)
    {                 
        min[count]=0;
        sec[count]=0;
        msec[count]=0;       
        start = false;
        stop = true;
        timer.cancel();
        try{
        z.deallocate();
        }
        catch(Exception e){}
        repaint();
    }
    if(keyCode == Canvas.KEY_NUM3)
        {
            if(stop == false)
            {
            start = false;
            stop = true;
            timer.cancel();
            try{
                InputStream inss = getClass().getResourceAsStream("alarm.wav");
                InputStreamReader iis= new InputStreamReader(inss);  
                z = Manager.createPlayer(inss,"audio/x-wav");
                z.prefetch();
                z.setLoopCount(2);
                z.start();
                }
    catch(Exception e){
    }
            }
        }
    if(keyCode==Canvas.KEY_NUM0)
    {
        try{
        z.deallocate();
        }
        catch(Exception e){}
        myMid.exit();
    }
 }
}
.

役に立ちましたか?

解決

例外がスローされていないようながスローされていない理由は、コードがひどいものであり、例外をキャッチするためです。

try
{
    maxX = getWidth();
    maxY = getHeight();
    count = 0;
    hour = new int[30];
    min = new int[30];
    sec = new int[30];
    msec = new int[30];
    start = false;
    stop = true;
    for(int j = 0  ; j <= 30 ; j++)
    {
        hour[j] = 0;
        min[j] = 0;
        sec[j] = 0;
        msec[j] = 0;
    }
}catch(Exception e)
{} // <- catches the exception and does nothing
.

キャッチブロックをこれに変更します。

}catch(Exception e) {
    e.printStackTrace();
}
.

とあなたは例外がスローされることがわかります。それからあなたは次のいずれかを実行できます:

  • a)try / catchを削除し、<= 30< 30に変更します。
  • b)それが貧弱な例だからコードを破棄します。

runのブロックは、わかりやすい限り、Countは決して変更されませんので、常に0です。

他のヒント

    hour = new int[30];
    min = new int[30];
    sec = new int[30];
    msec = new int[30];
    start = false;
    stop = true;
    for(int j = 0  ; j <= 30 ; j++)
    {
        hour[j] = 0;
        min[j] = 0;
        sec[j] = 0;
        msec[j] = 0;
    }
.

このコードは、indexOutofbounds例外を介してになります。ループjを通る最後のパスは30であり、任意の配列の最高インデックスは29です。

例外を投げていない場合、このコードは実行されていないか、例外が捕捉されています。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top