Wywoływanie metody z parametrem za pomocą jButton

0

witam,
mam problem z przypisaniem metody z parametrem do przycisku
W załączniku jest screen z błędem

Kopiuj

 import java.util.Random;


public class NewJFrame extends javax.swing.JFrame {


    public NewJFrame() {
        initComponents();

    }


    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">
    private void initComponents() {

        jComboBox1 = new javax.swing.JComboBox();
        jComboBox2 = new javax.swing.JComboBox();
        jComboBox3 = new javax.swing.JComboBox();
        jComboBox4 = new javax.swing.JComboBox();
        jButton1 = new javax.swing.JButton();
        jButton2 = new javax.swing.JButton();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jComboBox1.setModel(new javax.swing.DefaultComboBoxModel(new String[] { "Czerwony", "Zielony", "Żółty", "Niebieski", "Czarny", "Biały" }));
        jComboBox1.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                jComboBox1ActionPerformed(evt);
            }
        });

        jComboBox2.setModel(new javax.swing.DefaultComboBoxModel(new String[] { "Czerwony", "Zielony", "Żółty", "Niebieski", "Czarny", "Biały" }));

        jComboBox3.setModel(new javax.swing.DefaultComboBoxModel(new String[] { "Czerwony", "Zielony", "Żółty", "Niebieski", "Czarny", "Biały" }));

        jComboBox4.setModel(new javax.swing.DefaultComboBoxModel(new String[] { "Czerwony", "Zielony", "Żółty", "Niebieski", "Czarny", "Biały" }));

        jButton1.setText("Sprawdź");
        jButton1.addMouseListener(new java.awt.event.MouseAdapter() {
            public void mouseClicked(java.awt.event.MouseEvent evt) {
                jButton1MouseClicked(evt);
            }
        });

        jButton2.setText("Start");
        jButton2.addMouseListener(new java.awt.event.MouseAdapter() {
            public void mouseClicked(java.awt.event.MouseEvent evt) {
                jButton2MouseClicked(evt);
            }
        });

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jComboBox1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jComboBox2, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jComboBox3, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jComboBox4, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addComponent(jButton1)
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
            .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addComponent(jButton2)
                .addContainerGap())
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jButton2)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, 225, Short.MAX_VALUE)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jComboBox1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(jComboBox2, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(jComboBox3, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(jComboBox4, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(jButton1))
                .addGap(21, 21, 21))
        );

        pack();
    }// </editor-fold>

  
    
    public static void porownanie(String[] tab2){
       //String tab2[] = this.losowanie();
        if(jComboBox1.getSelectedItem()==tab2[0]){
        System.out.println("dobrze");
        }else{
            System.out.println("źle");
        }
    }
    
    
    static String[] losowanie(){
        String tab2[] = new String[4];
        tab2[0]="";
        tab2[1]="";
        tab2[2]="";
        tab2[3]="";
        
        String tab1[] = new String[6];
        tab1[0]="Czerwony";
        tab1[1]="Zielony";
        tab1[2]="Żółty";
        tab1[3]="Niebieski";
        tab1[4]="Czarny";
        tab1[5]="Biały";
        
        for(int i=0; i<=3;i++){
            Random random = new Random();
      int r = random.nextInt(5)+0;
      tab2[i]=tab1[r];
        }
System.out.println(tab2[0]);
System.out.println(tab2[1]);
System.out.println(tab2[2]);
System.out.println(tab2[3]);
    
return tab2;

 }
    
    
    

    private void jComboBox1ActionPerformed(java.awt.event.ActionEvent evt) {                                           
        System.out.println("You Selected : " + jComboBox1.getSelectedItem());
    }                                          

    private void jButton2MouseClicked(java.awt.event.MouseEvent evt) {                                      
        NewJFrame.losowanie();
        jButton2.setVisible(false);
    }                                     

 
    private void jButton1MouseClicked(java.awt.event.MouseEvent evt) {                                      
        NewJFrame.porownanie(String[] tab2)
    }                                     


    public static void main(String[] args) {

        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(NewJFrame.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(NewJFrame.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(NewJFrame.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(NewJFrame.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }
        //</editor-fold>

        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new NewJFrame().setVisible(true);
            }
        });
        

    }
    // Variables declaration - do not modify
    public static javax.swing.JButton jButton1;
    private javax.swing.JButton jButton2;
    public static javax.swing.JComboBox jComboBox1;
    private javax.swing.JComboBox jComboBox2;
    private javax.swing.JComboBox jComboBox3;
    private javax.swing.JComboBox jComboBox4;
    // End of variables declaration


}

<image>Bez tytułu.png</image>

0

Wywołaj metodę poprawnie:

NewFrame.porownanie(tab2);

0

shusty napisał(a):

Wywołaj metodę poprawnie:

NewFrame.porownanie(tab2);

W ten sposób też już próbowałem

1

Komunikat jasno mówi cannot find symbol tab2</code>dlatego, że <code class="java">String[] tab2zdefiniowałeś wewnątrz funkcji i zmienna nie jest widoczna poza zakresem funkcji. Zdefiniuj tablicę w klasie i przekaż ją do funkcji a będzie działać.

0

pk_82 napisał(a):

Komunikat jasno mówi cannot find symbol tab2</code>dlatego, że <code class="java">String[] tab2zdefiniowałeś wewnątrz funkcji i zmienna nie jest widoczna poza zakresem funkcji. Zdefiniuj tablicę w klasie i przekaż ją do funkcji a będzie działać.

Byłbym wdzięczny gdybyś mógł jeszcze powiedzieć jak prawidłowo to zrobić, bo nie bardzo wiem.
Sorry, że zawracam głowę o takie pierdoły, ale dopiero zaczynam uczyć się programowania i jestem całkiem zielony.
Wcześniej nie programowałem w żadnym języku.

Edit:

Już dowiedziałem się jak to zrobić i teraz NetBeans nie pokazuje błędu w kodzie programu, ale po odpaleniu go i użyciu przycisku jButton1 w konsoli pojawiają się błędy.

1

Bo nie zdefiniowałeś tej tablicy (nie podałeś jaki ma mieć rozmiar), np:

Kopiuj

public static String[] tab2=new String[4];

0

pk_82 napisał(a):

Bo nie zdefiniowałeś tej tablicy (nie podałeś jaki ma mieć rozmiar), np:

Kopiuj

public static String[] tab2=new String[4];

Wielkie dzięki!!!
Mam nadzieję, że będę mógł liczyć na dalszą pomoc jeśli znowu będę miał problemy przy tym programie :)

0

No i pojawił się kolejny problem.
W moim programie jComboBox1 pozwala wybrać jedną z kilku opcji(napisów) i chciałbym aby program przypisywał wybrany przez użytkownika napis do utworzonego stringa: tekst1.
Niestety nie mogę zapisać tego tak:
tekst1 = jComboBox1.getSelectedItem();
ponieważ NetBeans wyświetla błąd:

incompatible types
required: String
found: Object

problem próbowałem rozwiązać zapisując to w ten sposób:

Kopiuj

tekst1 = "";
tekst1 += jComboBox1.getSelectedItem();

ale okazuje się że gdy :

Kopiuj

tekst1 = "";
tekst1 += "costam"
tekstA = "costam"

to wtedy string o nazwie tekst1 nie jest równy stringowi o nazwie tekstA,
a jeśli tak jest to w moim programie nie działa pewien kluczowy mechanizm.
Jak w takim razie powinienem to zapisać aby string tekst1 był równy stringowi tekstA
Bardzo proszę o pomoc.

1

Czemu chcesz przypisać jakiś obiekt nie będący Stringiem do Stringa? WTF? Do inta też byś przypisywał napis?
Przecież błąd Ci nawet o tym mówi:
incompatible types
required: String
found: Object

Zrób tak: tekst1 = jComboBox1.getSelectedItem().getText();

Jeśli porównujesz Stringi operatorem == to źle robisz bo porównujesz tylko referencje, porównuj za pomocą equals(String).

Zawieszasz się na takich podstawowych rzeczach, że wg mnie szkoda byś dalej to pisał, bo co chwilę będziesz marnował kupę czasu na takie błahostki, które są ultra podstawowe. Bierz w rękę książkę lub tutorial z neta i się ucz, bo inaczej się więcej namęczysz niż faktycznie napiszesz. To tak jakbyś chciał całki rozwiązywać a miał spore braki w dodawaniu, mnożeniu, ułamkach itd. i co chwilę się zawieszał, bo coś źle robisz.

0

Zrobiłem trochę inaczej:
tekst1 = (String)jComboBox1.getSelectedItem();

Stringi porównuję teraz tak jak kazałeś. Dzięki!

shusty napisał(a):

Zawieszasz się na takich podstawowych rzeczach, że wg mnie szkoda byś dalej to pisał, bo co chwilę będziesz marnował kupę czasu na takie błahostki, które są ultra podstawowe. Bierz w rękę książkę lub tutorial z neta i się ucz, bo inaczej się więcej namęczysz niż faktycznie napiszesz. To tak jakbyś chciał całki rozwiązywać a miał spore braki w dodawaniu, mnożeniu, ułamkach itd. i co chwilę się zawieszał, bo coś źle robisz.

Słuszna uwaga. Dzięki za radę, na pewno tak zrobię, ale jak już napiszę ten program, bo za kilka dni muszę go mieć gotowego.

I jeszcze raz dzięki za pomoc!

0

Wiem, że nie powinienem już nikomu zawracać głowy, ale (obiecuję, że to już po raz ostatni) znów potrzebuję pomocy. Mój program jest już prawie gotowy, brakuje mi tylko jednego mechanizmu, który już zrobiłem, ale okazał się zły. Jako, że mój projekt ma być grą Mastermind potrzebuję mechanizmu, który będzie zliczał ile użytkownik podał dobrych kolorów na złych miejscach. Mechanizm zliczający kolory na właściwych miejscach już mam.
Mam stworzoną czteroelementową tablicę stringów tab1, która jest uzupełniona losowo nazwami kolorów(kolory mogą się powtarzać jak to w Mastermind bywa).
Druga czteroelementowa tablica stringów - tab2 zawiera nazwy kolorów podanych przez użytkownika.
Jak więc powinienem zapisać taki mechanizm

0

Nie wiem czy oto chodzi.

Kopiuj

 for(int i=0; i<tab2.lenght;i++){
	 for(int j=0; j<tab1.lenght;j++){
		 if(i==j){
			 if(tab2[i]==tab1[j])
			 System.out.println("kolor prawidłowy na prawidłowym miejscu");
	 }
		 else{
			 if(tab2[i] == tab1[j]){
				 System.out.println("kolor "+tab2[i] +" jest prawidłowy, ale ustawiony w złym miejscu");
			 }
		 }
 }
 }

Pisane na szybko z palca. Trzeba użyć pewnie equals, dostosować obsługę tego po swojemu itd., ale ogólnie powinno być widać po tym kodzie o co chodzi.

0

Niestety nie działa dobrze we wszystkich przypadkach.
Chodzi o to żeby na przykład, gdy:

Kopiuj

tab1[0] = "a";
tab1[1] = "b";
tab1[2] = "a";
tab1[3] = "a";

Kopiuj

tab2[0] = "b";
tab2[1] = "b";
tab2[2] = "b";
tab2[3] = "a";

wtedy na konsoli powinno wypisać:

Kopiuj

(tutaj nic nie wypisuje, ponieważ kolor b występuje tylko raz w tab1 a użytkownik podał go na właściwym miejscu)
kolor b jest prawidłowy na prawidłowym miejscu
(tutaj nic nie wypisuje, ponieważ kolor b występuje tylko raz w tab1 a użytkownik podał go na właściwym miejscu))
kolor a jest prawidłowy na prawidłowym miejscu

Jeśli tak byłoby prościej to może to być w formie inkrementacji jakichś zmiennych, tzn. że wynik byłby wypisywany przez np.:

Kopiuj

System.out.println("Ilość prawidłowych kolorów na prawidłowych miejscach: "+k);
System.out.println("Ilość prawidłowych kolorów na złych miejscach: "+m);

Wtedy przy tak wypełnionych tablicach wynik wyglądałby tak:
Ilość prawidłowych kolorów na prawidłowych miejscach: 2
Ilość prawidłowych kolorów na złych miejscach: 0

Próbuję właśnie przekształcić Twój mechanizm, ale nie bardzo mi to wychodzi.

Edit:
Przekształcając Twój mechanizm doszedłem do czegoś takiego:

Kopiuj

for(int i=0; i<tab1.length;i++){
         for( int j=0; j<tab2.length;j++){
                 if(i==j){
                         if(tab1[i].equals(tab2[j])) {
                         System.out.println("kolor prawidłowy na prawidłowym miejscu");
                         k++;
                         if(m!=0) {
                                 m=m-2;
                             }
                     }
                         
         }else{
                         if(tab1[i].equals(tab2[j])){
                                 System.out.println("kolor "+tab2[j] +" jest prawidłowy, ale ustawiony w złym miejscu");
                                 m++;
                         }
                 }
 }
 }
         System.out.println();
         System.out.println("Ilość prawidłowych kolorów na prawidłowych miejscach: "+k);
         System.out.println("Ilość prawidłowych kolorów na złych miejscach: "+m);

Sprawdziłem na szybko i chyba działa, ale jeszcze muszę to sprawdzić dokładnie dla wszystkich przypadków wypełnienia tablic.

Edit2:

Nie działa dla wszystkich przypadków.