Java Swing练习 - 无法访问变量，不知道为什么

我正在学习Java开发和家庭作业任务我已被指示使用Swing框架编写基本应用程序。该设置非常简单：一个应用程序从卡片组（Arraylist）中选择两张卡片（字符串），让用户猜测第二张卡片是高于还是低于第一张。Java Swing练习 - 无法访问变量，不知道为什么

小菜一碟，对不对？所以我想，直到我偶然发现一个问题：我已经将一个动作链接到一个按钮，并且该动作确实被执行，但我似乎无法访问我在处理动作时在应用程序中启动的变量。看一下“Gok hoger”这个动作，它会为“kaart1”返回一个空值。然而，当我在我的程序构造函数中回显字符串“kaart1”时，它填充很好。对于这个原因，我无能为力......所以我希望这里的一些聪明人能向我解释为什么我的代码像这样行事。

if(e.getActionCommand().equals("Gok hoger")){ 
    System.out.println(kaart1); // <------- Why does this return null? 
}

是的，这是一个家庭作业，我已经标记为这样以防万一。

import javax.swing.*; 

import java.awt.*; 
import java.awt.event.ActionEvent; 
import java.util.*; 
import java.awt.Dimension; 

public class Programma extends JFrame { 

    ArrayList<String> kaartboek = new ArrayList<String>(); 
    boolean gewonnen = false; 
    int kaart1waarde; 
    int kaart2waarde; 
    String kaart1; 
    String kaart2; 

    public Programma(){ 
     super("Hoger/lager"); 

     vulKaartboek(kaartboek); 
     String kaart1 = new String(trekKaart(kaartboek)); 
     String kaart2 = new String(trekKaart(kaartboek)); 

     JButton hogerButton = new JButton("Hoger"); 
     JButton lagerButton = new JButton("Lager"); 
     hogerButton.setAction(new Actie("Gok hoger")); 
     lagerButton.setAction(new Actie("Gok lager")); 

     JLabel kaart1label = new JLabel(new ImageIcon(
      "/Users/David/Dropbox/School/eclipse/2011/VGO_Opdracht1/src/kaartboek/" 
      + kaart1 +".png")); 
     JLabel kaart2label = new JLabel(new ImageIcon(
      "/Users/David/Dropbox/School/eclipse/2011/VGO_Opdracht1/src/kaartboek/" 
      + "back" + ".png")); 

     Container c = getContentPane(); 
     c.setLayout(new GridLayout(2,2)); 
     c.add(kaart1label); 
     c.add(kaart2label); 
     c.add(hogerButton); 
     c.add(lagerButton); 

     this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     this.pack(); 
     this.setVisible(true); 
    } 

    public static void main (String args[]){ 
     JFrame frame1 = new Programma(); 
    } 

    public String getKaartSuit(String kaart){ 
     String kaart1suit = kaart.substring(0, 1); 
     if(kaart1suit.equals("c")){kaart1suit = "Klaveren";} 
     if(kaart1suit.equals("d")){kaart1suit = "Ruiten";} 
     if(kaart1suit.equals("h")){kaart1suit = "Harten";} 
     if(kaart1suit.equals("s")){kaart1suit = "Schoppen";} 
     return kaart1suit; 
    } 

    public String getKaartBeeld(String kaart){ 
     int kaartwaarde = Integer.parseInt(kaart.substring(1, kaart.length())); 
     String kaartbeeld = ""; 
     switch(kaartwaarde){ 
      case 2: kaartbeeld = "Twee"; break; 
      case 3: kaartbeeld = "Drie"; break; 
      case 4: kaartbeeld = "Vier"; break; 
      case 5: kaartbeeld = "Vijf"; break; 
      case 6: kaartbeeld = "Zes"; break; 
      case 7: kaartbeeld = "Zeven"; break; 
      case 8: kaartbeeld = "Acht"; break; 
      case 9: kaartbeeld = "Negen"; break; 
      case 10: kaartbeeld = "Tien"; break; 
      case 11: kaartbeeld = "Boer"; break; 
      case 12: kaartbeeld = "Dame"; break; 
      case 13: kaartbeeld = "Heer"; break; 
      case 14: kaartbeeld = "Aas"; break; 
     } 
     return kaartbeeld; 
    } 

    public String getKaartNaam(String kaart){ 
     String kaartnaam = getKaartSuit(kaart) + " " + getKaartBeeld(kaart); 
     return kaartnaam; 
    } 

    public String trekKaart(ArrayList<String> kaartboek){ 
     Random random = new Random(); 
     int willekeurig = random.nextInt(kaartboek.size()); 

     String kaart = kaartboek.get(willekeurig); 
     kaartboek.remove(willekeurig); 

     return kaart; 
    } 

    public void vulKaartboek(ArrayList kaartboek){ 
     for(int i = 2; i < 14; i++){ 
      kaartboek.add("c" + i); // Clubs 
      kaartboek.add("d" + i); // Diamonds 
      kaartboek.add("h" + i); // Hearts 
      kaartboek.add("s" + i); // Spades 
     } 
    } 

    public int getKaartWaarde(String kaart){ 
     int kaartwaarde = Integer.parseInt(kaart.substring(1, kaart.length())); 
     return kaartwaarde; 
    } 

    class Actie extends AbstractAction { // Inner klasse 
     public Actie(String s){ 
      super(s); 
     } 
     public void actionPerformed(ActionEvent e) { 
      if(e.getActionCommand().equals("Gok hoger")){ 
       System.out.println(kaart1); // <------- Why does this return null? 

       } 
      if(e.getActionCommand().equals("Gok lager")){ 
       System.out.println("Test"); 
      } 
     } 
    } 
}

来源

2011-02-22 David M

+1为sscce http://sscce.org/，虽然图像是不相关的。 – trashgod

重新格式化的代码;如果不正确请回复。 – trashgod

您有两个名为kaart1的变量。一个是Programma构造函数的局部。在“Actie”中访问的是实例变量。

您可能不想在构造函数中声明第二个变量，而是初始化实例变量。替换“String kaart1 = new String（trekKaart（kaartboek））;” “kaart1 = new String（trekKaart（kaartboek））;”

来源

2011-02-22 16:07:11 DJClayworth

令人惊讶的是，有些事情会变得如此清晰。我正在寻找一个更复杂的原因。非常感谢！ :) –

-2

变量kaart1未在类Actie中声明。那就是为什么它是空的。你的代码很难为我读（部分你的语言部分设计），所以我不完全知道你在做什么。

来源

2011-02-22 16:05:36 Voooza

-1您应该将其置于评论之下。 –

你得到空，因为您使用的是这里的局部变量：

String kaart1 = new String(trekKaart(kaartboek));

与

this.kaart1 = new String(trekKaart(kaartboek));

来源

2011-02-22 16:09:18 Spajus

在你宣布两个新的对象该地图的构造代替。内部类尝试访问未分配，因此“空”老该地图变量

String kaart1; 
String kaart2; 

public Programma(){ 
    super("Hoger/lager"); 

    vulKaartboek(kaartboek); 
    String kaart1 = new String(trekKaart(kaartboek)); // You shoud initialize it as this.kaart1 
    String kaart2 = new String(trekKaart(kaartboek)); // You shoud initialize it as this.kaart2 

}

来源

2011-02-22 16:26:23 Bartzilla

正如其他人指出，问题围绕Hiding Fields。此外，考虑到编程接口：

List<String> kaartboek = new ArrayList<String>();

在trekKaart()，不要实例Random()反复。相反洗牌一次：

Collections.shuffle(kaartboek);

然后你就可以很容易地用简单的trekKaart()检索随机卡：

kaart1 = trekKaart(kaartboek, 1); 
kaart2 = trekKaart(kaartboek, 2); 

private String trekKaart(List<String> kaartboek, int index) { 
    return kaartboek.get(index); 
}

最后，trekKaart()和vulKaartboek()从Programma构造函数调用，所以他们应该是私有的。

来源

2011-02-22 17:06:42 trashgod

Java Swing练习 - 无法访问变量，不知道为什么

回答

相关问题