Kirjautuminen

Haku

Tehtävät

Keskustelu: Ohjelmointikysymykset: Kommentteja Java-koodistani?

E1ss [14.04.2016 16:25:46]

#

Voisko joku kattoo tän ohjelman että onko tää oikeaoppisesti ohjelmoitu.

public class Luokka {

	public static void main(String[] args) {
		Luku luku = new Luku();
		Tallennus tallennus = new Tallennus();

		System.out.println("Syötä kymmenen numeroa 1-3");
		for(int counter = 0; counter < 10; counter++){
			System.out.print(counter + 1 + ". ");
			luku.otaJaTarkistaLuku();
			int x = luku.getLuku();
			tallennus.tallennaLuku(x);
		}

		System.out.println("Syötit eniten numeroa: ");
		Vertaa vertaa = new Vertaa(tallennus.getMuistiYksi(),tallennus.getMuistiKaksi(),tallennus.getMuistiKolme());
		System.out.println(vertaa.vartailu());
	}

}
import java.util.Scanner;

public class Luku {
	private Scanner scan;
	private int luku;


	public Luku(){

	}


	public void otaJaTarkistaLuku(){
		while(true){
			this.scan = new Scanner(System.in);
			try{
				this.luku = this.scan.nextInt();
			}catch(Exception e){
				System.out.println("Et syöttänyt kunnollista numeroa!");
				continue;
			}
			if(this.luku < 1 || this.luku > 3){
				continue;
			}
			break;
		}
	}


	public int getLuku(){
		return this.luku;
	}
}
public class Tallennus {
	private int muistiYksi = 0;
	private int muistiKaksi = 0;
	private int muistiKolme = 0;


	public void tallennaLuku(int luku){
		int tallennettava = luku;

		switch(tallennettava){
		case 1:
			this.muistiYksi++;
			break;
		case 2:
			this.muistiKaksi++;
			break;
		case 3:
			this.muistiKolme++;
			break;
		}
	}


	public int getMuistiYksi(){
		return this.muistiYksi;
	}


	public int getMuistiKaksi(){
		return this.muistiKaksi;
	}


	public int getMuistiKolme(){
		return this.muistiKolme;
	}
}
public class Vertaa {
	int eka;
	int toka;
	int kolmas;


	public Vertaa(int eka, int toka, int kolmas){
		this.eka = eka;
		this.toka = toka;
		this.kolmas = kolmas;
	}


	public int vartailu(){
		if(toka > eka || kolmas > eka){
			if(toka > kolmas){
				return 2;
			}else{
				return 3;
			}
		}
		return 1;
	}
}

Grez [14.04.2016 16:51:17]

#

Ei ole. Keskeisin aivojani syövyttävä ominaisuus on noi samanarvoiset erilliset muuttujat jotka olisi järkevämpää laittaa taulukoksi. Koko tallennusluokka on periaatteessa turha, koska int[3] ajaa saman asian.

Jos kuitenkin haluat tehdä oman taulukkoluokan niin käytä sen sisällä taulukkoa.

Vastaus

Aihe on jo aika vanha, joten et voi enää vastata siihen.

Tietoa sivustosta