我的setter不初始化我的属性，我的属性总是等于零[关闭]

Question

我不知道为什么“选中”总是发送0值。我想发送onItemSelected在MainMenuSuperviser中选择的属性中收到的值，但它不起作用，我不知道为什么。

我试图创建另一个对象，但没有帮助。

package kingdomino.supervisers;

import java.util.*;

public final class MainMenuSuperviser extends Superviser {

    private int selected;
    public static final String QUIT = "Quitter";
    public static final String NEW_4_PLAYERS = "Jouer à 4";
    public static final String NEW_3_PLAYERS = "Jouer à 3";
    public static final String NEW_2_PLAYERS = "Jouer à 2";

    private MainMenuView view;

    public MainMenuSuperviser() {
    }

    public void setSelected(int selected) {
        this.selected = selected;
    }

    public void setView(MainMenuView view) {
        // TODO : définir les items à afficher par la vue
        this.view = view;
        view.setItems(List.of(NEW_2_PLAYERS, NEW_3_PLAYERS, NEW_4_PLAYERS, QUIT));
    }

    public void onItemSelected(int selected) {
        // TODO : réagir à l'événement utilisateur item sélectionné
        this.setSelected(selected);
        if ((selected == 0) || (selected == 1) || (selected == 2)) {
            this.selected = selected;
            view.goTo("PlayGame");
        }
        if (selected == 3)
            System.exit(0);
    }

    public int getSelected() {
        return selected;
    }

}

然后是程序部分，启动代码的部分

public class Program {
    private final static List<Domino> dominoes = CsvDataReader.readDominoesFromFile("resources/data/dominoes.csv").stream()
            .map(dto -> new Domino(dto.getId(), 
                    new Tile(Terrain.valueOf(dto.getTerrain1()), dto.getCrownsCount1()),
                    new Tile(Terrain.valueOf(dto.getTerrain2()), dto.getCrownsCount2())))
            .collect(Collectors.toList());
    
    private final static List<Player> players = CsvDataReader.readPlayersFromFile("resources/data/players.csv").stream()
            .map(dto -> new Player(dto.getName(), dto.getHexColor()))
            .collect(Collectors.toList());
    
    /**
     * Point d'entrée d'une exécution.
     * L'application ne tient pas compte des arguments.
     * 
     * @param args une liste d'arguments. 
     */
    public static void main(String[] args) {
        
        MainMenuSuperviser mSuperviser = new MainMenuSuperviser();
        GameFactory gF = new GameFactory(mSuperviser.getSelected(), players, dominoes);
        NewGame nG = new NewGame(gF.getSelected(), gF.getListDominoes(), gF.getListPlayers());
        Game game = new Game(players, nG.getPile(), nG.getSelected());

        PlayGameSuperviser superviser = new PlayGameSuperviser(game);
        
        MainWindow window = new MainWindow("Ai 2022 - KingDomino - Lawson Jefferson",
            new SwingMainMenuView("MainMenu", mSuperviser),
            new SwingPlayGameView("PlayGame", superviser, mSuperviser.getSelected())
        );
        window.start("MainMenu");
    }

}

这就是调用onItemSelected的地方

public final class SwingMainMenuView extends SwingView implements MainMenuView {
    private static final long serialVersionUID = -2211970715714357966L;
    private final ImageIcon background = new ImageIcon("resources/images/main_menu_background.jpg");
    private final MainMenuSuperviser superviser;
    
    private List<String> items; 
    private int selected = 0;
    
    public SwingMainMenuView(String title, MainMenuSuperviser superviser) {
        super(title);
        this.superviser = superviser;
        this.superviser.setView(this);
    }
    
    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);    
        drawBackground(g);
        drawCommands(g);
        drawMenuItems(g);
    }

    private void drawCommands(Graphics g) {
        int left = Theme.PANEL_WIDTH/100*2;
        int top = Theme.PANEL_HEIGHT/100*3;
        
        g.setColor(new Color(255,255,255,128));
        g.fillRect(left-5, top-11, 200, 62);
        g.setColor(Color.BLACK);
        
        g.drawString("\u2191 : déplacer vers le haut",left, top);
        g.drawString("\u2193 : déplacer vers le bas",left, top+20);
        g.drawString("\u23ce : choisir",left, top+40);
        
    }

    private void drawBackground(Graphics g) {
        g.drawImage(background.getImage(), 0, 0, null);
    }
    
    private void drawMenuItems(Graphics g) {
        int left = Theme.PANEL_WIDTH/3;
        int top = Theme.PANEL_HEIGHT/3;
        int width = Theme.PANEL_WIDTH/3;
        int height = Theme.PANEL_HEIGHT/12;
        int gap = Theme.PANEL_HEIGHT/15;
        
        
        for(int i=0; i < items.size(); ++i) {
            var item = items.get(i);
            
            if(i == selected) {
                g.setColor(Theme.SECONDARY_BACKGROUND_COLOR_ALPHA);
                g.fillRoundRect(left - gap/2, top - gap/2, width + gap, height + gap, gap, gap);
                
                g.setColor(Theme.SECONDARY_BACKGROUND_COLOR);
                g.drawRoundRect(left - gap/2, top - gap/2, width + gap, height + gap, gap, gap);
            }
            
            g.setColor(Theme.SECONDARY_BACKGROUND_COLOR);
            g.fillRoundRect(left, top, width, height, 5, 5);
            
            g.setFont(Theme.ITEM_FONT);
            g.setColor(Theme.PRIMARY_COLOR);
            g.drawString(item, left+gap/2, top+height/2);       
            
            top += gap + height;
        }
        
    }

    @Override
    public void onKeyTyped(int keyCode) {
        if(keyCode == KeyEvent.VK_DOWN) {
            this.selected = (selected + 1) % items.size();
        } else if(keyCode == KeyEvent.VK_UP) {
            this.selected = (selected == 0) ? items.size() - 1 : selected - 1;
        } else if(KeyEvent.VK_ENTER == keyCode) {
            this.superviser.onItemSelected(this.selected);
        }
        
        this.repaint();
    }

    @Override
    public void setItems(List<String> items) {
        this.items = List.copyOf(items);
        this.selected = 0;
        this.repaint();
    }

}

Answer 1

我不会像你问的那样回答这个问题，所以请原谅我，但是你的问题并不是一个简单的人可以回答的形式。请参阅有关最小可复制示例的评论。

但是，一般来说，编程的很大一部分是测试和调试。学习成为一名Java程序员的一个基本部分是学习该行业的工具。测试和调试通常比最初编写程序所需的时间更长。没关系。

解决这类问题的最佳方法是单独测试每一位，然后构建测试交互，然后测试或观察整个执行过程。

1. 为似乎属于问题的每个组件的每个元素编写单元测试。使用JUnit或其他选择

作为最简单的单元测试，实例化一个MainMenuSuperviser并将其选择的属性设置为许多不同的值，获取值并比较您设置的值是否是您获得的值。对于简单的getter和setter，您根本不需要编写单元测试-但这是第一步。

然后调用已知值为selected的onItemSelected方法，查看您得到的值是否与之匹配。请注意，如果使用值3调用它，测试将无法完成！

使用JUnit或类似的方法意味着您为功能的各个方面或至少为您需要调查的每个错误构建了一套可复制的测试。在每个构建周期中运行这些测试。修复任何损坏的问题。

实例化一个MainMenuSuperviser，然后实例化一个SwingMainMenuView，并将视图提供给主管。模拟调用onKeyTyped（…） keyCode值编码>并检查所选的值是否符合预期。您可能需要添加一个get方法来允许测试。

然后，做同样的事情，并在主管中测试选择的值。

使用模拟框架来测试更复杂的交互。

随着时间的推移，先学会设计测试，然后再开发功能代码。

使用调试器附加到正在运行的代码。使用断点、监视变量并在调用代码时逐步遍历代码。密切关注选定变量的值。

在代码运行时，使用日志框架（例如log4j或替代方案）记录方法调用和变量值<代码>系统。出来println可以作为一个非常快速的