Prosím o radu! :) – Java – Fórum – Programujte.com

Ahoj,

napsal jsem si takový krátký prográmek, který nefunguje jak má.

I když zadám (např.) zelenou, tak mi to vypíše - Neplatná volba, zkuste to znovu.

Věřím, že se tu najde někdo, kdo mi pomůže.

package enumeratory;
import java.util.Scanner;

public class class01 {


public static enum barvy{Cerna, Cervena, Zelena, Modra, Zluta}


public static void main(String[] args) {
barva();
}



public static void barva() {



String y;
Scanner in = new Scanner(System.in);
System.out.print("Zadejte barvu: ");
y = in.nextLine();


if(y == "Cerna" || y == "Zelena" || y == "Cervena" || y == "Zluta" || y == "Modra") {

barvy barvicka = barvy.valueOf(y);
System.out.println("Vase barva je ");
System.out.println(barvicka);

} else {

            System.out.println("Neplatna volba, zkuste to znovu!");

barva();
}
}

}

tusim ze java nejak moc nemusi porovnavani retezcu pomoci ==

#2 KIIV
A jak to mám teda vyřešit? :)

je pravda ze pre string je spolahlivejsia funkcia equals() ale porovnanie by malo tiez fungovat vo vecsine pripadov ... myslim ze skor bude problem v tom co vracia scanner ... presnejsie v tom ze (ak som spravne pochopil) nextLine() je skor ako line separator ... resp funguje skor tak ze ak si predtym nieco nacitaval tak v buffery este nieco zostane a nextLine je tu na to aby si to odstranil pripadne spracoval (hlavne line delimitery - predpokladam ze tvoje volanie vracia okrem tvojho zadania aj odriadkovanie) ... pre nacitavanie len formatovaneho vstupu je skor next() kde si mozes (je tam aj verzia bez patternu) aj zadat pattern cim este viac zvysis spolahlivost

osobne by som to riesil vypisom a vyberom podla cisla ak mas len obmedzeny zoznam ... ak ziadas aby uzivatel trafil presne tvoj retazec tak to neni nic moc ... navyse aj teba to moze dost nasrat ked o pol roka budes zadavat "cervena" namiesto "Cervena"

to je pravda ale kedze JVM vecsinou nevytvara novy string ak nejaky stary ma rovnaky obsah tak sa da pouzit aj porovnanie ci odkazuju na rovnaky objekt

		String a = "that", b = "that";
		System.out.println(a==b);

#6 Flowy
no kazdopadne je to blbe pokud se zrovna opravdu nesnazis porovnavat reference... + nacteni ze scanneru ti na 100% nemuze vratit uz existujici referenci

#7 KIIV
ok sry ... preco scanner nespravi odkaz na uz existujuci string?

#8 Flowy
podle me by to bylo extremne pomale - otestovat vsechny uz definovane stringy a pak kdyztak hodit referenci, pokud se nejaky rovna...

Ikdyz je pravda, ze by to zase sedlo k "Java" pristupu :D

cize dovod preco  

		String a = "that", b = "that";
		System.out.println(a==b);

vracia true je pretoze sa to uprace pri preklade?

#10 Flowy
s velice vysokou pravdepodobnosti ano :)

a mozna i pro zmateni programatoru zacatecniku :)

Vůbec nechápu o čem tu mluvíte :D

No nic, následující 2 hodiny zřejmě strávím přemýšlením nad tím, co to tu melete :D

#12 brunoczech
jen o tom ze  ==  se neda pouzit na porovnavani obsahu retezcu v jave :)

Takze na to mám jít přes equals?

 Tak jsem to vyřešil takhle: (Kdybych věděl, že nějaký equals existuje, dřív. Tak bych se s tim nemusel tak srát :D)

package enumeratory;
import java.util.Scanner;

public class class01 {
	
	
	public static enum barvy{Cerna, Cervena, Zelena, Modra, Zluta}
	
	
	public static void main(String[] args) {
		barva();
	}
	
	
	
	public static void barva() {
		
		
		String[] x = new String[5];
		x[0]="Cervena";
		x[1]="Cerna";
		x[2]="Zelena";
		x[3]="Modra";
		x[4]="Zluta";
		
		
		String y;
		Scanner in = new Scanner(System.in);
		System.out.print("Zadejte barvu: ");
		y = in.next();
		
		
		if(y.equals(x[0]) || y.equals(x[1]) || y.equals(x[2]) || y.equals(x[3]) || y.equals(x[4]) ) {
			
			barvy barvicka = barvy.valueOf(y);
			System.out.print("Vase barva je ");
			System.out.println(barvicka);
			System.out.println("-------------------------------------\nProgram zacal odznovu\n");
			barva();
		
		} else {
			
            System.out.println("Neplatna volba, zkuste to znovu!");
			
			barva();
			}
	}

}

teoreticky muzes vyuzit i to, ze valueOf vyhodi vyjimku, kdyz ta polozka neni v seznamu.. kdyz se to dobre odchyti, tak ani nemusis pouzivat ty equals (a ani if kdyz se to tak veme)

Udělal bych to takto:

...

if("Cerna".equalsIgnoreCase(y) || 
	"Zelena".equalsIgnoreCase(y) ||
	"Cervena".equalsIgnoreCase(y) ||
	"Zluta".equalsIgnoreCase(y) ||
	"Modra".equalsIgnoreCase(y)){
  ...
}
...

bude to chodit i v případě, že y bude null + case insensitive

Pouzi radsej switch case, je to asi rovnako modularne ako toto co si napisal, kontrolujes tam veci len raz, ke to zrozumitelnejsi kod a napises toho menej: 

String farba;
switch(farba.toLowerCase()){
	case "cierna":
		....
		break;
	case "cervena":
		... // kod
		break;
	...

	default:
		// co sa ma stat ak ziadna farba nvyhovie
}

inak naco najskor overujete ci je jednou z nich, v tomto pripade je to uplne zbytocne.