এমভিসিতে পুনর্লিখনের পরে জিইউআই কাজ করছে না


123

আমি এমভিসি স্টাইল প্রোগ্রামিং অনুশীলন করছি। আমার একটি একক ফাইলে একটি মাস্টারমাইন্ড গেম রয়েছে, ভাল কাজ করছে (সম্ভবত "চেক" বোতামটি শুরুতে অদৃশ্য।

http://paste.pocoo.org/show/226726/

তবে যখন আমি এটিকে মডেল, ভিউ, কন্ট্রোলার ফাইলগুলিতে পুনরায় লিখেছি - এবং যখন আমি খালি পিনটিতে ক্লিক করি (এটি আপডেট করা উচিত এবং নতুন রঙ দিয়ে পুনরায় রঙ করা উচিত) - লক্ষণ ঘটে। এখানে কেউ কি কোনও সমস্যা দেখতে পাবে? আমি বিভিন্ন জায়গায় পুনর্নির্মাণ () রাখার চেষ্টা করেছি, তবে এটি মোটেই কার্যকর হয় না: /

প্রধান:

public class Main { 
    public static void main(String[] args){
        Model model = new Model();
        View view = new View("Mastermind", 400, 590, model);
        Controller controller = new Controller(model, view); 
        view.setVisible(true);
    }
}

মডেল :

import java.util.Random;

public class Model{
    static final int
    LINE = 5,
    SCORE = 10, OPTIONS = 20;
    Pin pins[][] = new Pin[21][LINE];
    int combination[] = new int[LINE];
    int curPin = 0;
    int turn = 1;
    Random generator = new Random();
    int repaintPin;
    boolean pinsRepaint=false;
    int pinsToRepaint;
    boolean isUpdate = true, isPlaying = true, isRowFull = false;
    static final int HIT_X[] = {270,290,310,290,310}, HIT_Y[] = {506,496,496,516,516};

    public Model(){

        for ( int i=0; i < SCORE; i++ ){
            for ( int j = 0; j < LINE; j++ ){
                pins[i][j] = new Pin(20,0);
                pins[i][j].setPosition(j*50+30,510-i*50);
                pins[i+SCORE][j] = new Pin(8,0);
                pins[i+SCORE][j].setPosition(HIT_X[j],HIT_Y[j]-i*50);
            }
        }
        for ( int i=0; i < LINE; i++ ){
            pins[OPTIONS][i] = new Pin( 20, i+2 );
            pins[OPTIONS][i].setPosition( 370,i * 50 + 56);
        }

    }

    void fillHole(int color) {
        pins[turn-1][curPin].setColor(color+1);
        pinsRepaint = true;
        pinsToRepaint = turn;
        curPin = (curPin+1) % LINE;
        if (curPin == 0){
            isRowFull = true;
        }
        pinsRepaint = false;
        pinsToRepaint = 0;
    }

    void check() {
        int junkPins[] = new int[LINE], junkCode[] = new int[LINE];
        int pinCount = 0, pico = 0;

        for ( int i = 0; i < LINE; i++ ) {
            junkPins[i] = pins[turn-1][i].getColor();
            junkCode[i] = combination[i];
        }
        for ( int i = 0; i < LINE; i++ ){
            if (junkPins[i]==junkCode[i]) {
                pins[turn+SCORE][pinCount].setColor(1);
                pinCount++;
                pico++;
                junkPins[i] = 98;
                junkCode[i] = 99;
            }
        }
        for ( int i = 0; i < LINE; i++ ){
            for ( int j = 0; j < LINE; j++ )
                if (junkPins[i]==junkCode[j]) {
                    pins[turn+SCORE][pinCount].setColor(2);
                    pinCount++;
                    junkPins[i] = 98;
                    junkCode[j] = 99;
                    j = LINE;
            }
        }
        pinsRepaint = true;
        pinsToRepaint = turn + SCORE;
        pinsRepaint = false;
        pinsToRepaint=0;

        if ( pico == LINE ){
            isPlaying = false;
        }
        else if ( turn >= 10 ){
                isPlaying = false;
        }
        else{
            curPin = 0;
            isRowFull = false;
            turn++;
        }
    }

    void combination() {
        for ( int i = 0; i < LINE; i++ ){
          combination[i] = generator.nextInt(6) + 1;
        }
    }
}

class Pin{
    private int color, X, Y, radius;

    public Pin(){
        X = 0; Y = 0; radius = 0; color = 0;
    }

    public Pin( int r,int c ){
        X = 0; Y = 0; radius = r; color = c;
    }

    public int getX(){
        return X;
    }

    public int getY(){
        return Y;
    }

    public int getRadius(){
        return radius;
    }

    public void setRadius(int r){
        radius = r;
    }

    public void setPosition( int x,int y ){
        this.X = x ;
        this.Y = y ;
    }
    public void setColor( int c ){
        color = c;
    }
    public int getColor() {
        return color;
    }
}

দেখুন:

import java.awt.*;
import javax.swing.*;

public class View extends Frame{  
    Model model;
    JButton checkAnswer;
    private JPanel button;
    private static final Color COLORS[] = {Color.black, Color.white, Color.red, Color.yellow, Color.green, Color.blue, new Color(7, 254, 250)};

    public View(String name, int w, int h, Model m){
        model = m;
        setTitle( name );
        setSize( w,h );
        setResizable( false );
        this.setLayout(new BorderLayout());

        button = new JPanel();
        button.setSize( new Dimension(400, 100));
        button.setVisible(true);
        checkAnswer = new JButton("Check");
        checkAnswer.setSize( new Dimension(200, 30));
        button.add( checkAnswer );
        this.add( button, BorderLayout.SOUTH);
        button.setVisible(true);
    }

    @Override
    public void paint( Graphics g ) {
        g.setColor( new Color(238, 238, 238));
        g.fillRect( 0,0,400,590);

        for ( int i=0; i < model.pins.length; i++ ) {
            paintPins(model.pins[i][0],g);
            paintPins(model.pins[i][1],g);
            paintPins(model.pins[i][2],g);
            paintPins(model.pins[i][3],g);
            paintPins(model.pins[i][4],g);
        }
    }

    @Override
    public void update( Graphics g ) {
        if ( model.isUpdate ) {
            paint(g);
        }
        else {
            model.isUpdate = true;
            paintPins(model.pins[model.repaintPin-1][0],g);
            paintPins(model.pins[model.repaintPin-1][1],g);
            paintPins(model.pins[model.repaintPin-1][2],g);
            paintPins(model.pins[model.repaintPin-1][3],g);
            paintPins(model.pins[model.repaintPin-1][4],g);
        }
    }

    void repaintPins( int pin ) {
        model.repaintPin = pin;
        model.isUpdate = false;
        repaint();
    }

    public void paintPins(Pin p, Graphics g ){
        int X = p.getX();
        int Y = p.getY();
        int color = p.getColor();
        int radius = p.getRadius();
        int x = X-radius;
        int y = Y-radius;

        if (color > 0){
            g.setColor( COLORS[color]);
            g.fillOval( x,y,2*radius,2*radius );
        }
        else{
            g.setColor( new Color(238, 238, 238) );
            g.drawOval( x,y,2*radius-1,2*radius-1 );
        }
        g.setColor( Color.black );
        g.drawOval( x,y,2*radius,2*radius );
    }
}

নিয়ন্ত্রক:

import java.awt.*;
import java.awt.event.*;

public class Controller implements MouseListener, ActionListener { 
    private Model model;
    private View view;

    public Controller(Model m, View v){ 
        model = m;
        view = v;

        view.addWindowListener( new WindowAdapter(){
            public void windowClosing(WindowEvent e){
            System.exit(0);
        } });
        view.addMouseListener(this);
        view.checkAnswer.addActionListener(this);
        model.combination();
    }

    public void actionPerformed( ActionEvent e ) {
        if(e.getSource() == view.checkAnswer){
            if(model.isRowFull){
                model.check();
            }
        }
    }

    public void mousePressed(MouseEvent e) {
        Point mouse = new Point();

        mouse = e.getPoint();
        if (model.isPlaying){
            if (mouse.x > 350) {
                int button = 1 + (int)((mouse.y - 32) / 50);
                if ((button >= 1) && (button <= 5)){
                    model.fillHole(button);
                    if(model.pinsRepaint){
                        view.repaintPins( model.pinsToRepaint );
                    }
                }
            }
        }
    }

    public void mouseClicked(MouseEvent e) {}
    public void mouseReleased(MouseEvent e){}
    public void mouseEntered(MouseEvent e) {}
    public void mouseExited(MouseEvent e)  {}
}

5
পুরানো এবং নতুন কোড উভয়ই এডাব্লুটি এবং সুইং উপাদানগুলি মিশ্রণ সম্পর্কিত সমস্যা রয়েছে। আরও দেখুন stackoverflow.com/questions/2687871
trashgod

সুতরাং কোনও আপডেটের সাথে সমস্যাটির কারণ হতে পারে?
ট্রেভর_নিজ


আমি যুক্ত করেছি এবং উদাহরণ দিয়েছি যা আপনার পুনরায় নকশাকে গাইড করতে পারে।
ট্র্যাশগোড

উত্তর:


147

যেমন আপনি আবিষ্কার করেছেন, মডেল – ভিউ – কন্ট্রোলার প্যাটার্নটি কোনও প্যানিসিয়া নয়, তবে এটি কিছু সুবিধা দেয়। এমভিসি-র মূলী , সুইং বিভাজক মডেল আর্কিটেকচারটি এ সুইং আর্কিটেকচার ওভারভিউতে আলোচিত হয়েছে । এই রূপরেখার উপর ভিত্তি করে , নিম্নলিখিত উদাহরণটি একটি এমভিসি দেখায় বাস্তবায়নকে অনেক সহজ গেমের বাস্তবায়ন যা অনুরূপ নীতিগুলিকে চিত্রিত করে। নোট করুন যে এলোমেলোভাবে বেছে নেওয়া Modelএকটি একক পরিচালনা করে Piece। কোনও ব্যবহারকারীর নির্বাচনের প্রতিক্রিয়া হিসাবে, Viewউপায়টির মাধ্যমে check()প্রতিক্রিয়া শোনার সময়, পদ্ধতিটি আহ্বান করে । তারপর থেকে প্রাপ্ত তথ্য ব্যবহার করে নিজেই আপডেট । একইভাবে, mayModelupdate()ViewModelControllerreset()Model। বিশেষত, এর কোনও অঙ্কন Modelনেই এবং এর মধ্যে কোনও গেম যুক্তি নেই View। এই কিছুটা আরও জটিল গেমটি একই ধারণাগুলি চিত্রিত করার জন্য তৈরি করা হয়েছিল।

সংযোজন: এমভিসি কিভাবে Viewপ্রকৃতির পরিবর্তন না করে একজনকে বাড়ানোর অনুমতি দেয় তা দেখানোর জন্য আমি আসল উদাহরণটি সংশোধন করেছি Model

সংযোজন: যেমন @ অ্যাকফ পর্যবেক্ষণ করেছেন, এমভিসি পর্যবেক্ষক প্যাটার্নে জড়িত । আপনার পরিবর্তনগুলির Modelঅবহিত করার একটি উপায় প্রয়োজন View। বেশ কয়েকটি পদ্ধতি ব্যাপকভাবে ব্যবহৃত হয়:

  • নীচের উদাহরণে, সরলতার জন্য Modelপ্রসারিত Observable

  • আরও সাধারণ পদ্ধতির একটি ব্যবহার করে EventListenerListConverterঅ্যাপ্লিকেশনটিতে প্রদর্শিত এবং প্রচুর সংখ্যক EventListenerউপমঞ্চল এবং প্রয়োগকারী ক্লাস দ্বারা প্রস্তাবিত হিসাবে ।

  • তৃতীয় বিকল্পটি হ'ল একটি PropertyChangeListenerহিসাবে ব্যবহার করা shown এখানে এবং এখানে

সংযোজন: সুইং নিয়ন্ত্রকদের সম্পর্কে কিছু সাধারণ প্রশ্ন এখানে এবং এখানে সম্বোধন করা হয় ।

স্ক্রিন ক্যাপচার

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.EventQueue;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Observable;
import java.util.Observer;
import java.util.Random;
import javax.swing.Icon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

/**
 * @see https://stackoverflow.com/q/3066590/230513
 * 15-Mar-2011 r8 /programming/5274962
 * 26-Mar-2013 r17 per comment
 */
public class MVCGame implements Runnable {

    public static void main(String[] args) {
        EventQueue.invokeLater(new MVCGame());
    }

    @Override
    public void run() {
        JFrame f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.add(new MainPanel());
        f.pack();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }
}

class MainPanel extends JPanel {

    public MainPanel() {
        super(new BorderLayout());
        Model model = new Model();
        View view = new View(model);
        Control control = new Control(model, view);
        JLabel label = new JLabel("Guess what color!", JLabel.CENTER);
        this.add(label, BorderLayout.NORTH);
        this.add(view, BorderLayout.CENTER);
        this.add(control, BorderLayout.SOUTH);
    }
}

/**
 * Control panel
 */
class Control extends JPanel {

    private Model model;
    private View view;
    private JButton reset = new JButton("Reset");

    public Control(Model model, View view) {
        this.model = model;
        this.view = view;
        this.add(reset);
        reset.addActionListener(new ButtonHandler());
    }

    private class ButtonHandler implements ActionListener {

        @Override
        public void actionPerformed(ActionEvent e) {
            String cmd = e.getActionCommand();
            if ("Reset".equals(cmd)) {
                model.reset();
            }
        }
    }
}

/**
 * View
 */
class View extends JPanel {

    private static final String s = "Click a button.";
    private Model model;
    private ColorIcon icon = new ColorIcon(80, Color.gray);
    private JLabel label = new JLabel(s, icon, JLabel.CENTER);

    public View(Model model) {
        super(new BorderLayout());
        this.model = model;
        label.setVerticalTextPosition(JLabel.BOTTOM);
        label.setHorizontalTextPosition(JLabel.CENTER);
        this.add(label, BorderLayout.CENTER);
        this.add(genButtonPanel(), BorderLayout.SOUTH);
        model.addObserver(new ModelObserver());
    }

    private JPanel genButtonPanel() {
        JPanel panel = new JPanel();
        for (Piece p : Piece.values()) {
            PieceButton pb = new PieceButton(p);
            pb.addActionListener(new ButtonHandler());
            panel.add(pb);
        }
        return panel;
    }

    private class ModelObserver implements Observer {

        @Override
        public void update(Observable o, Object arg) {
            if (arg == null) {
                label.setText(s);
                icon.color = Color.gray;
            } else {
                if ((Boolean) arg) {
                    label.setText("Win!");
                } else {
                    label.setText("Keep trying.");
                }
            }
        }
    }

    private class ButtonHandler implements ActionListener {

        @Override
        public void actionPerformed(ActionEvent e) {
            PieceButton pb = (PieceButton) e.getSource();
            icon.color = pb.piece.color;
            label.repaint();
            model.check(pb.piece);
        }
    }

    private static class PieceButton extends JButton {

        Piece piece;

        public PieceButton(Piece piece) {
            this.piece = piece;
            this.setIcon(new ColorIcon(16, piece.color));
        }
    }

    private static class ColorIcon implements Icon {

        private int size;
        private Color color;

        public ColorIcon(int size, Color color) {
            this.size = size;
            this.color = color;
        }

        @Override
        public void paintIcon(Component c, Graphics g, int x, int y) {
            Graphics2D g2d = (Graphics2D) g;
            g2d.setRenderingHint(
                RenderingHints.KEY_ANTIALIASING,
                RenderingHints.VALUE_ANTIALIAS_ON);
            g2d.setColor(color);
            g2d.fillOval(x, y, size, size);
        }

        @Override
        public int getIconWidth() {
            return size;
        }

        @Override
        public int getIconHeight() {
            return size;
        }
    }
}

/**
 * Model
 */
class Model extends Observable {

    private static final Random rnd = new Random();
    private static final Piece[] pieces = Piece.values();
    private Piece hidden = init();

    private Piece init() {
        return pieces[rnd.nextInt(pieces.length)];
    }

    public void reset() {
        hidden = init();
        setChanged();
        notifyObservers();
    }

    public void check(Piece guess) {
        setChanged();
        notifyObservers(guess.equals(hidden));
    }
}

enum Piece {

    Red(Color.red), Green(Color.green), Blue(Color.blue);
    public Color color;

    private Piece(Color color) {
        this.color = color;
    }
}

1
@ ট্র্যাভার_নিজ: আমি উপরের উদাহরণটি আপডেট করেছি। আপনি সংশোধনগুলি তুলনা করতে দরকারী হতে পারে।
ট্র্যাশগোড

2
কৌতূহলী ফাউলারের যে কেউ 2006 এর জন্য নিম্নলিখিত নিবন্ধটি লিখেছেন
জেমস পি।


20
দুর্দান্ত উত্তর, তবে এটি আমার কাছে কিছুটা অদ্ভুত বলে মনে হচ্ছে একজন নিয়ামক জেপানেল উত্তরাধিকার সূত্রে পেয়েছেন এবং মূল প্যানেলে যুক্ত হয়েছেন। নিয়ামকটি কি যৌক্তিক কিছু বলে মনে হয় না এবং এভাবে দৃশ্যমান হয় না? আমি কী মিস করছি?
মাইগুয়েলকোবাইন

1
@ মিগুয়েলকোবাইন: ভাল পর্যবেক্ষণ; আমি উদাহরণস্বরূপ বলতে চেয়েছিলাম যে কীভাবে নিয়ামকটি বোতামের দৃষ্টিভঙ্গি এবং মডেলকে একত্রিত করে সেই প্যাটার্নটির পৃথক বাস্তবায়নের মাধ্যমে দৃষ্টিভঙ্গি এবং মডেল উভয়কেই পরিবর্তন করতে পারে। Controlকোনও পদ্ধতিকে ওভাররাইড করে না JPanel, সুতরাং একটি স্ট্যাটিক কারখানাটি আরও ভাল।
ট্র্যাশগোড

20

সুইংয়ের মাধ্যমে দেখার সময়, ডিজাইনাররা তার এমভিসি বাস্তবায়নে নিয়মিতভাবে ভিউ উপাদানগুলি আপডেট করার কাজে নিযুক্ত একটি উপায় পর্যবেক্ষক / পর্যবেক্ষণযোগ্য কলব্যাকের মাধ্যমে। উদাহরণটিতে দেখা যায় AbstractTableModel, এর বিভিন্ন fireTable*Changed/Updated/etcপদ্ধতি রয়েছে যা TableModelListenerমডেলের প্রতি তার সমস্ত পর্যবেক্ষককে সতর্ক করবে ।

আপনার কাছে একটি বিকল্প হ'ল শ্রোতার ধরণটি আপনার Modelশ্রেণিতে যুক্ত করা এবং তারপরে আপনার মডেলের অবস্থাতে কোনও মোডের নিবন্ধিত পর্যবেক্ষককে অবহিত করা। আপনার Viewশ্রোতা হওয়া উচিত এবং আপডেট পাওয়ার পরে এটি পুনরায় রঙ করা উচিত।

সম্পাদনা: ট্র্যাশগোডে +1 করুন। এটি তার ব্যাখ্যাতে একটি বিকল্প শব্দ বিবেচনা করুন।

আমাদের সাইট ব্যবহার করে, আপনি স্বীকার করেছেন যে আপনি আমাদের কুকি নীতি এবং গোপনীয়তা নীতিটি পড়েছেন এবং বুঝতে পেরেছেন ।
Licensed under cc by-sa 3.0 with attribution required.