我的Java小程序中有任何錯誤嗎？

Question

我目前正在參加高中程序設計1課，目前正在完成我的最終項目。 我們被允許創建我們想要的任何東西，我決定制作一個程序，其中有吉他琴頸的前12個品格，您可以單擊品格中的弦，它會告訴您這是什么音符以及演奏什么音符聽起來像。 我設法讓程序告訴我在設置的第一個矩形上選擇了什么音符。 然后，我將該代碼復制了一堆，並更改了值以制作其余的曲目。 但是，它不再告訴我嘗試單擊一個音符時要按的音符。 我的程序可以正確構建並執行，沒有錯誤，所以我不太確定該做什么，我真的可以用一只手。 任何和所有答復都非常感謝！

import java.awt.*;
import java.applet.Applet;

public class Final extends Applet
{
    //s6f1 stands for string 6 fret 1
    int sf;
    Rectangle s6f1, s6f2, s6f3, s6f4, s6f5, s6f6, s6f7, s6f8, s6f9, s6f10, s6f11, s6f12;

    public static void main (String[] args)
    {

    }

    public void paint (Graphics g)
    {
        drawNeck(g);
        int x = 690;
        int y = 153;
        Expo.setColor(g,Expo.red);
        Expo.setFont(g,"Arial",Font.BOLD,28);

        switch (sf)
        {
                case 1 : 
            Expo.drawString(g,"F",x,y);
            break;
                case 2 : 
            Expo.drawString(g,"F#/Gb",x,y);
            break;
                case 3 : 
            Expo.drawString(g,"G",x,y);
            break;
                case 4 : 
            Expo.drawString(g,"G#/Ab",x,y);
            break;
                case 5 : 
            Expo.drawString(g,"A",x,y);
            break;
                case 6 : 
            Expo.drawString(g,"A#,Bb",x,y);
            break;
                case 7 : 
            Expo.drawString(g,"B",x,y);
            break;
                case 8 : 
            Expo.drawString(g,"C",x,y);
            break;
                case 9 : 
            Expo.drawString(g,"C#/Db",x,y);
            break;
                case 10 : 
            Expo.drawString(g,"D",x,y);
            break;
                case 11 : 
            Expo.drawString(g,"D#/Eb",x,y);
            break;
                case 12 : 
            Expo.drawString(g,"E",x,y);
            break;
        }


    }




    public void init()
    {
        int m = 50;
        int n = 10;
        //s stands for string, f stands for fret; ex: s6f1 means String 6 Fret 1
        s6f1  = new Rectangle (50,198,m,n);
        s6f2  = new Rectangle (101,198,m,n);
        s6f3  = new Rectangle (151,198,m,n);
        s6f4  = new Rectangle (201,198,m,n);
        s6f5  = new Rectangle (251,198,m,n);
        s6f6  = new Rectangle (301,198,m,n);
        s6f7  = new Rectangle (351,198,m,n);
        s6f8  = new Rectangle (401,198,m,n);
        s6f9  = new Rectangle (451,198,m,n);
        s6f10 = new Rectangle (501,198,m,n);
        s6f11 = new Rectangle (551,198,m,n);
        s6f12 = new Rectangle (601,198,m,n);
        sf = 0;
    }


    public boolean mouseDown(Event e, int x, int y)
    {
        if(s6f1.inside(x,y))
            sf = 1;
        if(s6f2.inside(x,y))
            sf = 2;
        if(s6f3.inside(x,y))
            sf = 3;
        if(s6f4.inside(x,y))
            sf = 4;
        if(s6f5.inside(x,y))
            sf = 5;
        if(s6f6.inside(x,y))
            sf = 6;
        if(s6f7.inside(x,y))
            sf = 7;
        if(s6f8.inside(x,y))
            sf = 8;
        if(s6f9.inside(x,y))
            sf = 9;
        if(s6f10.inside(x,y))
            sf = 10;
        if(s6f11.inside(x,y))
            sf = 11;
        if(s6f12.inside(x,y))
            sf = 12;
        else
            sf = 100;
        repaint();
        return true;
    }

    public static void drawNeck (Graphics g)
    {
        int a = 50;
        int b = 225;
        int c = 650;
        //The Background
        Expo.setColor(g,Expo.black);
        Expo.fillRectangle(g,0,0,1000,650);
        Expo.setColor(g,Expo.lightPink);
        Expo.fillRectangle(g,10,10,990,640);
        //The neck
        Expo.setColor(g,Expo.brown);
        Expo.fillRectangle(g,50,50,650,225);
        //The frets
        Expo.setColor(g,Expo.black);
        Expo.drawLine(g,100,a,100,b);
        Expo.drawLine(g,150,a,150,b);
        Expo.drawLine(g,200,a,200,b);
        Expo.drawLine(g,250,a,250,b);
        Expo.drawLine(g,300,a,300,b);
        Expo.drawLine(g,350,a,350,b);
        Expo.drawLine(g,400,a,400,b);
        Expo.drawLine(g,450,a,450,b);
        Expo.drawLine(g,500,a,500,b);
        Expo.drawLine(g,550,a,550,b);
        Expo.drawLine(g,600,a,600,b);
        Expo.drawLine(g,650,a,650,b);
        //The Inlays
        Expo.setColor(g,Expo.white);
        Expo.fillCircle(g,175,137,10);
        Expo.fillCircle(g,275,137,10);
        Expo.fillCircle(g,375,137,10);
        Expo.fillCircle(g,475,137,10);
        Expo.fillCircle(g,625,112,10);
        Expo.fillCircle(g,625,163,10);  
        //The Strings
        Expo.setColor(g,Expo.grey);
        Expo.fillRectangle(g,a,73,c,77);
        Expo.fillRectangle(g,a,98,c,102);
        Expo.fillRectangle(g,a,123,c,127);
        Expo.fillRectangle(g,a,148,c,152);
        Expo.fillRectangle(g,a,173,c,177);
        Expo.fillRectangle(g,a,198,c,202);
        //String names and Fret numbers
        Expo.setColor(g,Expo.blue);
        Expo.setFont(g,"Arial",Font.BOLD,14);
        Expo.drawString(g,"E",20,210);
        Expo.drawString(g,"A",20,180);
        Expo.drawString(g,"D",20,155);
        Expo.drawString(g,"G",20,130);
        Expo.drawString(g,"B",20,105);
        Expo.drawString(g,"e",20,75);
        Expo.drawString(g,"Note:",680,120);
        Expo.setFont(g,"Arial",Font.ITALIC,24);
        Expo.drawString(g,"Created by Mitchell",640,635);
        //The User-interface
            //The Chords area
            int t = 150;
        Expo.setFont(g,"Arial",Font.BOLD,28);
        Expo.drawString(g,"Chords:",800,50);
        Expo.setColor(g,Expo.lightBlue);
        Expo.fillRectangle(g,700+t,80,740+t,120);
        Expo.fillRectangle(g,750+t,80,790+t,120);
        Expo.fillRectangle(g,700+t,130,740+t,170);
        Expo.fillRectangle(g,750+t,130,790+t,170);
        Expo.fillRectangle(g,700+t,180,740+t,220);
        Expo.fillRectangle(g,750+t,180,790+t,220);
            //The note area
        Expo.fillRectangle(g,680,123,745,163);

    }

}

附加信息：-我的高中班級使用Expo Java文件作為applet-這是我運行applet時的樣子： http : //imgur.com/N86wkkq

Answer 1

問題是您的if -cascade在mouseDown 。 正如您不使用else if ，所有if語句都經過評估，無論哪一個實際產生true 。 這不是一個大問題，但是最后一個if包含else部分， 每次您不單擊s6f12都會執行該s6f12 。

因此， sf始終為12或100。

解決方案：在以下情況下使用else if ：

if (s6f1.inside(x,y)) sf = 1;
else if (s6f2.inside(x,y)) sf = 2;
else if (s6f3.inside(x,y)) ...
...
else sf = 100;