Problem s vypisom – Java – Fórum – Programujte.com

zdravim mam problem s ktorym si neviem rady vsetko by podla mna malo fungovat ale nefunguje dam sem cast kodu ktora by mala stacit na pochopenie, robim to cez swing

final ArrayList<Hrac> hrac = new ArrayList<Hrac>();
		
		vyberHracov.addActionListener(new ActionListener()
		{
			
			public void actionPerformed(ActionEvent e) {
				
				
				
				int volba = Integer.parseInt(vyberHracov.getSelectedItem().toString());
				int volba2=volba;
				
				
				
				
				String[] menoHraca;
				menoHraca = new String[--volba2];
				
				switch(volba)
				{
					
					
					case 2: 
						
						lblMenoPrvehoHraca.setText("Meno 1.hráča");
						textFieldMenoPrvehoHraca.setVisible(true);
						menoHraca[0]=textFieldMenoPrvehoHraca.getText();
						
						lblMenoDruhehoHraca.setText("Meno 2.hráča");
						textFieldMenoDruhehoHraca.setVisible(true);
						menoHraca[1]=textFieldMenoDruhehoHraca.getText().toString();
						
						lblMenoTretiehoHraca.setText("");
						textFieldMenoTretiehoHraca.setVisible(false);
						lblMenoStvrtehoHraca.setText("");
						textFieldMenoStvrehoHracar.setVisible(false);
						
						break;
					
					case 3:
						
						lblMenoPrvehoHraca.setText("Meno 1.hráča");
						textFieldMenoPrvehoHraca.setVisible(true);
						
						
						lblMenoDruhehoHraca.setText("Meno 2.hráča");
						textFieldMenoDruhehoHraca.setVisible(true);
						menoHraca[1]=textFieldMenoDruhehoHraca.getText().toString();
						
						lblMenoTretiehoHraca.setText("Meno 3.hráča");
						textFieldMenoTretiehoHraca.setVisible(true);
						menoHraca[2]=textFieldMenoTretiehoHraca.getText().toString();
					
						lblMenoStvrtehoHraca.setText("");
						textFieldMenoStvrehoHracar.setVisible(false);
							
						break;
						
					case 4:
						lblMenoPrvehoHraca.setText("Meno 1.hráča");
						textFieldMenoPrvehoHraca.setVisible(true);
						menoHraca[0]="miso";

						lblMenoDruhehoHraca.setText("Meno 2.hráča");
						textFieldMenoDruhehoHraca.setVisible(true);
						menoHraca[1]="dano";

						lblMenoTretiehoHraca.setText("Meno 3.hráča");
						textFieldMenoTretiehoHraca.setVisible(true);
						menoHraca[2]="jano";
						
						lblMenoStvrtehoHraca.setText("Meno 4.hráča");
						textFieldMenoStvrehoHracar.setVisible(true);
						menoHraca[3]="fero";

								
						break;
					
				}
				
				
				
				for(int i=0;i<volba;i++)
				{
					hrac.add(new Hrac(menoHraca[i]));
					
					
				}

JButton btnNewButton = new JButton("New button");
		btnNewButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				
				for(int i=0;i<4;i++)
				{
					textArea.append(hrac.get(i).getMenoHraca().toString());
					
					
					
					
					
					
				}
				
				
			}
		});

v comboboxe si vyberiem moznost 2,3 alebo 4. Zobrazi sa potrebny pocet labelov a textfieldov, potom by som do nich mal zapisat mena, mena by sa mali ulozit a vypisat, mena sa ukladaju cez polia, po kliknuti na dalsie tlacitko by sa mali mena alebo meno vypisat, no nefunguje to, ako nahle spravim ale namiesto napriklad tohto menoHraca[0]=textFieldMenoPrvehoHraca.getText(); toto menoHraca[0]="jano"; uz to vypisuje bez problemov, vie niekto pomoct ?

#1 Oliver Habara
Proč dáváš při tom přiřazení do pole textFieldMenoPrvehoHrace.getText().toString() ? Vždyť samotná metoda getText() ti vrátí String toho txtFieldu :) zkus si to toString() vymazat a uvidíš jestli ti to pomůže :)

#2 sakal91
to som uz skusal predtym, teraz som to skusil znova, bohuzial nepomohlo

#3 Oliver Habara
kdybys mohl někde zabalit a nahrát celou aplikaci určitě bych ti pomohl líp :)

http://www.uloz.to/x1qZ4PJ/monopolynejde-zip

ked budes mas stiahnute tak povec, smazem to, inak je to robene cez eclipse, samozrejme treba mat stiahnuty plugin pre Swing

#4 sakal91
a problem je v uvodneOkno.java

#6 Oliver Habara
Ok du na to kouknout, muzes to smazat :)

#6 Oliver Habara
 

        final hodnoty hodnota = new hodnoty();
        final ArrayList<Hrac> hrac = new ArrayList<Hrac>();

        vyberHracov.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {

                volba = nteger.parseInt(vyberHracov.getSelectedItem().toString());

                switch (volba) {

                    case 2:

                        lblMenoPrvehoHraca.setText("Meno 1.hr��a");
                        textFieldMenoPrvehoHraca.setVisible(true);


                        lblMenoDruhehoHraca.setText("Meno 2.hr��a");
                        textFieldMenoDruhehoHraca.setVisible(true);

                        break;

                    case 3:

                        lblMenoPrvehoHraca.setText("Meno 1.hr��a");
                        textFieldMenoPrvehoHraca.setVisible(true);


                        lblMenoDruhehoHraca.setText("Meno 2.hr��a");
                        textFieldMenoDruhehoHraca.setVisible(true);


                        lblMenoTretiehoHraca.setText("Meno 3.hr��a");
                        textFieldMenoTretiehoHraca.setVisible(true);

                        break;

                    case 4:
                        lblMenoPrvehoHraca.setText("Meno 1.hr��a");
                        textFieldMenoPrvehoHraca.setVisible(true);


                        lblMenoDruhehoHraca.setText("Meno 2.hr��a");
                        textFieldMenoDruhehoHraca.setVisible(true);


                        lblMenoTretiehoHraca.setText("Meno 3.hr��a");
                        textFieldMenoTretiehoHraca.setVisible(true);


                        lblMenoStvrtehoHraca.setText("Meno 4.hr��a");
                        textFieldMenoStvrehoHracar.setVisible(true);

                        break;

                }
                hodnota.DatHodnotu(volba);
            }

            private void hrac(String string) {
                // TODO Auto-generated method stub
            }
        });
        vyberHracov.setModel(new DefaultComboBoxModel(new String[]{"", "2", "3", "4"}));
        vyberHracov.setBounds(290, 193, 48, 20);
        contentPane.add(vyberHracov);

        JButton btnNewButton = new JButton("New button");
        btnNewButton.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                // vyprazdneni seznamu
                hrac.clear();
                textArea.setText("");
                switch (volba) {
                    case 2:

                        hrac.add(new Hrac(textFieldMenoPrvehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoDruhehoHraca.getText()));

                        break;

                    case 3:

                        hrac.add(new Hrac(textFieldMenoPrvehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoDruhehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoTretiehoHraca.getText()));
                        break;

                    case 4:
                        hrac.add(new Hrac(textFieldMenoPrvehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoDruhehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoTretiehoHraca.getText()));
                        hrac.add(new Hrac(textFieldMenoStvrehoHracar.getText()));
                        break;

                }

	// vypsani hracu
                for (Hrac hrac1 : hrac) {
                    textArea.append(hrac1.getMenoHraca() + "\n");
                }
            }
        });
        btnNewButton.setBounds(287, 406, 89, 23);
        

Tak problém byl v tom že jsi se snažil přiřadit jméno v akci comboboxu, takže ty txtFieldy byli prázdné, tudíž si nic nepřiřadil.... Jména musíš přidávat do toho seznamu až v akci toho tlačítka :) Jinak celkem prasácký kód :P Zkus dbát na čistotu kódu aby se v tom dalo dobře číst :D

#8 sakal91
diki moc, som vedel ze to bude nejaka blbost  samozrejme ze ma to nenapadlo :/ za kod sa ospravedlnujem ale riesil som tento problem a na ostante som sa v tej chvili nepozeral, inak bude to maturitna praca(monopoly) a este som len na zaciatku takze bude to vesele :D a urcite budem mat este problemy a budem sem pisat, este raz diki za pomoc

vedel by mi niekto poradit ako v tomto programe ktory sme tu uz riesili, pouzivat ziskane mena v novom jFrame ? cize tak ako som vypisoval v textArey po klinuti na tlacitko mena ktore som zadal tak po otvoreni noveho jFramu by mi ich po kliknuti na tlacitko vypisalo ties ale tie iste co som zadal uz predtym, proste tie hodnoty potrebujem preniest

nikto nevie pomoct ? 

#11 Oliver Habara
Udělej si GETTERY/SETTRY a tomu novému JFrame předej jako parametr ten první JFrame a pak k těm proměnným můžeš přistupovat přes ty metody get... např. secondFrame.getFirstTextField();

#11 Oliver Habara
pokud chceš získat jen ty jména tak tou metodou getTex...(); vrať obsah toho textFieldu nebo textArea :)

#13 sakal91
viem ako to myslis ale nejak mi to nejde  nemohol by si mi to prosim ta nejak ukazat v zdrojaku ? predpokladam ze ho uz nemas takze tu nahravam http://www.uloz.to/…nopolyyy-zip

#14 Oliver Habara
ty potřebeješ dostat z uvodního okna do te hrací plochy obsah té textArey?

#15 sakal91
ano, proste to co sa ulozi do objektov v uvodnomOkne cize v tomto pripade tie mena aby som mohol pouzivat a napriklad aj vypisovat v hracejPloche

e este prosim ta minule ako si mi tu daval to riesenie si pouzil takyto cyklus 

// vypsani hracu
                for (Hrac hrac1 : hrac) {
                    textArea.append(hrac1.getMenoHraca() + "\n");
                }

ja som taky cyklus nikdy nepouzival a ani som sa s nim nestretol preto som tomu velmi nechapal, pouzil som svoj vlastny ktoremu som rozumel a fungovalo to aj s nim ale rad by som vedel nieco aj o tom tvojom rieseni ak by ti to nerobilo problem zlahka vysvetlit

#16 Oliver Habara
Takže jednoduše si ve třídě uvodniOkno, kde vytváříš instanci té hracíPlochy (actionPerformed tlačítka Start) si tu instanci vytvoříš takhle:  

HraciPlocha plocha = new HraciPlocha(this);

pak ve třídě HracíPlocha si vytvoříš parametrický konstruktor (do tvojeho konstruktoru si přidáš jen parameter: public HraciPlocha(UvodniOkno okno){}) a v tom konstruktorz si prostě přiřadíš to okno do nějaké nové instance...

private UvodniOkno okno;
public HraciPlocha(UvodniOkno okno){
	this.okno = okno;	
}

no a pak jen ve třídě UvodniOkno si jen uděláš metodu get která bude vracet co potřebuješ a ve třídě HracíPlocha k nim budeš přistupovat takhle:

okno.getNeco();

teď spěchám, tak promiň že tak polopatě ti to píšu.... snad to pochopíš :) když tak ti později pošlu zdroják... :)

#17 Oliver Habara
Je to cyklus for-each... prostě ti projde celé pole/seznam .... takze v tom cyklu je hrac pole a do hrac1 se uklada postupne kazdy prvek pole... 

#19 sakal91
ten cyklus:  

for(Hrac hrac1 : hrac){
	System.out.println();
}

je stejný jako:

for(int i=0; i<hrac.length; i++){
	Hrac hrac1 = hrac[i];
	System.out.println();
}

#20 sakal91
som prisiel az teraz domov.... diki zatial za pomoc, skusim to, potom napisem ci sa podarilo alebo nie

HraciaPlocha plocha = new HraciaPlocha(this);

tuto hned to hadze chybu nieco s konstruktorom... 

najlepsie by bolo asi naozaj ukazat to nazorne v mojom zdrojaku ak by ti to nerobilo problem :)

#23 Oliver Habara
Večer se na to podívám... Teď mám momentálně napilno :)

#24 sakal91
ok nezabudni prosim, zatial to este poskusam 

#25 Oliver Habara
http://leteckaposta.cz/368323870

Tady to máš, udělal sem ti tam komentáře k tomu.... A prosím tě dříve než začneš přidávat jakoukoli funkčnost nebo prostě cokoli, vyčisti si ten kód!!! Je to totální chaos!! Nauč se psát čistý a úhledný kód!! Věř mi, jak pak budeš chtít něco změnit nebo budeš hledat chybu tak ji nenajdeš!

#26 sakal91
Dakujem velmi pekne ze si to pre mna spravil, problem ale je ze ked zapnem eclipse a vyberiem miesto kde ma vyhladat projekty, proste workspace, a spravne to nastavim tak toto co som stiahol od teba mi vobec do eclipsu nenacita

#26 sakal91
uz sa mi to podarilo, musel som to sice rucne ale ide to, inak ja tam potrebujem preniest cely objekt nie len ten text, ale myslim ze to pojde, ak by nastal problem napisem, kod upravim, este by som sa chcel opsravedlnit za to jedno neslusne slovicko tam, ani som si neuvedomil ze som to tam nechal  , diki este raz za pomoc, este sa chcem spytat ak by som este potreboval pomoc pomozes ak budes moct ? viem ze som mozno asi trocha otravny mojimy niekedy jednoduchymi otazkami ale OOP sa venujem len velmi kratky cas a tak mam niekedy nejasnosti

#28 wrany
od toho sem ti tam napsal ty komentáře, že cokoliv budeš chtít přenést tak k tomu vytvoříš ty metody GET :) klidně ti pomůžu to není problém :) jen mi když tak piš na email kdybys chtěl rychlejší odpovědi :)

#29 sakal91
ok tak na mail teda