Kirjoitin Hangman-tekstipohjaisen Java-pelin, johon on sisällytettävä ominaisuudet, jotka olen sisällyttänyt koodini kommentteihin .

Lyhyesti sanottuna peli pyytää käyttäjää kirjoittamaan sanan, jonka he (tai toinen henkilö) sitten arvaa. Ohjelma sensuroi sanan. Ohjelma kertoo käyttäjälle, onko arvattava kirjain sanassa vai ei, ja näyttää sensuroidun sanan etenemisen jokaisen arvauksen jälkeen. Jos käyttäjä on jo arvannut kirjeen aiemmin, ohjelma kertoo käyttäjälle siitä ja näyttää edelliset arvauksensa toistamatta mitään kirjaimia. Ohjelma näyttää yritysten lukumäärän lopussa.

Alla kirjoittamasi koodi toimii ja siinä on kaikki luetellut ominaisuudet. Mutta se ei näytä olevan optimaalinen ja todennäköisesti erittäin huono etiketti, koska olen itseopettanut toistaiseksi. Siksi etsin mitään neuvoja, jotka parantavat tätä koodia ja varmistavat, että en pääse pahoihin tottumuksiin ( Minulla on todennäköisesti jo haha), kun jatkan Java-oppimista itse.

//Simple Hangman game where user types a word, program stores it in all CAPS for easier user readability and censors the word (i.e *****) //User then guesses one letter at a time until the entire word is guessed. Program will inform the user if the guess is in the word, and show the progress of the word after each guess. //If the guessed letter is in the word, program will print out the # of times the letter is in the word. //Program will store and print out # of guesses (attempts) needed to guess the word at the end of the program. //If user tries to duplicate a previous guess, program will inform user of that and show previous guesses by user. Attempt count will not go up for duplicate guesses. //When the program shows previous guesses by the user (using a string), it cannot contain duplicate letters. (i.e: if user guesses "s" twice, "s" will still only show up once in the string) //StackOverFlow readers: This program works as intended, but as a self-taught beginner coder, I need assistance on optimal coding style (less lines the better) and good coding principles/etiquette //I definitely think there are much better ways to code this, but I cannot think of any more (as you probably noticed, this is v3, which has more features and yet similar amount of lines as version 1 haha) //All and any help is appreciated! Thank you :D import java.util.*; public class HangmanGameV3 { public static void main(String [] args){ //Initialize all the variables used here String storedword; char[] charstring; int length; char[] censor; int attempts=0; StringBuilder pastguesses = new StringBuilder(); //String Builder to add and print out previous guesses Scanner typedword = new Scanner(System.in); System.out.println("Enter your word to guess: "); storedword = typedword.nextLine(); storedword = storedword.toUpperCase(); //stores the word and changes it to all caps length = storedword.length(); charstring = storedword.toCharArray(); //creates char array of string //creates and prints an array of chars with the same length as string censor = storedword.toCharArray(); System.out.println("Your secret word is: "); for (int index = 0; index < length; index++){ censor[index] = "*"; } //Main loop to take guesses (is this while loop the ideal loop here? while (String.valueOf(censor).equals(storedword)== false){ //Initialize all variables in loop char charguess; String tempword; String tempstring; boolean correct = false; //required for if loops below/lets the user know if the letter is in the word or not int times = 0; //number of times a letter is in the word boolean repeated = false; //check if user guessed the same letter twice //prints the censored secret word for(int a= 0; a < length; a++){ System.out.print(censor[a]); } System.out.println(); //asks user for guess, then stores guess in Char charguess and String tempstring Scanner guess = new Scanner(System.in); System.out.println("Type your guess: "); tempword = guess.next(); charguess = tempword.charAt(0); //gets char data from scanner pastguesses.append(charguess); //adds guess to previous guess string tempstring = pastguesses.toString(); //checks if user already guessed the letter previously if (tempstring.lastIndexOf(charguess, tempstring.length() -2 ) != -1){ System.out.println("You already guessed this letter! Guess again. Your previous guesses were: "); pastguesses.deleteCharAt(tempstring.length()-1); System.out.println(tempstring.substring(0, tempstring.length()-1)); repeated = true; } //if the guess is not a duplicated guess, checks if the guessed letter is in the word if (repeated == false){ for (int index = 0; index < length; index++){ if(charstring[index] == Character.toUpperCase(charguess)) { censor[index] = Character.toUpperCase(charguess); //replaces * with guessed letter in caps correct = true; times++; } } if(correct == true){ System.out.println("The letter " + charguess + " is in the secret word! There are " + times +" " + charguess + " "s in the word. Revealing the letter(s): "); } else if (correct == false){ System.out.println("Sorry, the letter is not in the word. Your secret word: "); } System.out.println(); } attempts++; } System.out.println("You guessed the entire word "+ storedword.toUpperCase() + " correctly! It took you " + attempts + " attempts!"); //typedword.close(); //StackOverFlow readers: is this necessary? Not sure how to use .close() } 

Näyte koodistani tarvittaessa tarvittaessa viitteeksi:

kirjoita kuvan kuvaus tähän

Vastaa

Joitakin yksinkertaisia muutoksia:

Luod kaksi skanneria , yhden silmukan sisään ja toisen huonosti nimitetyn alussa. Nimetän typedwordinput ja korvaa guess -kohdan input.

if(repeated == false) 

kirjoitettaisiin paremmin

if(!repeated) 

Vastaavasti muutan muita, jos lauseet

Käytän Set<String> -tallennusta aikaisempien arvausten tallentamiseen

Olen siirtänyt d times -ilmoitus !repeated loop -alueelle siten, että sen ilmoitus on lähempänä käyttöä ja rajoitettu soveltamisalaan.

Muut ilmoitukset on yhdistetty arvon asetukseen, ja jotkut määritykset ketjutettu, kuten uusi

String wordToGuess = input.nextLine().toUpperCase(); 

tempstring on poistettu, se rakennetaan nyt vain tarvittaessa.

Useat muuttujat on nimetty uudelleen selittävämmiksi nimiksi.

Lopullinen koodi:

import java.util.HashSet; import java.util.Scanner; import java.util.Set; public class HangmanGameV3 { public static void main(String[] args) { int attempts = 0; Set<String> previousGuesses = new HashSet<>(); Scanner input = new Scanner(System.in); System.out.println("Enter your word to guess: "); String wordToGuess = input.nextLine().toUpperCase(); int length = wordToGuess.length(); char[] wordToGuessChars = wordToGuess.toCharArray(); //creates char array of string //creates and prints an array of chars with the same length as string char[] censor = wordToGuess.toCharArray(); System.out.println("Your secret word is: "); for (int index = 0; index < length; index++) { censor[index] = "*"; } //Main loop to take guesses (is this while loop the ideal loop here? while (!String.valueOf(censor).equals(wordToGuess)) { //Initialize all variables in loop boolean correct = false; //required for if loops below/lets the user know if the letter is in the word or not boolean repeated = false; //check if user guessed the same letter twice //prints the censored secret word for (int a = 0; a < length; a++) { System.out.print(censor[a]); } System.out.println(); //asks user for guess, then stores guess in Char charguess and String tempstring System.out.println("Type your guess: "); String currentGuess = input.next().toUpperCase().substring(0, 1); char currentGuessChar = currentGuess.charAt(0); //gets char data from scanner //checks if user already guessed the letter previously if (previousGuesses.contains(currentGuess)) { System.out.println("You already guessed this letter! Guess again. Your previous guesses were: "); System.out.println(previousGuesses.stream().reduce("", String::concat)); repeated = true; } previousGuesses.add(currentGuess); //if the guess is not a duplicated guess, checks if the guessed letter is in the word if (!repeated) { int times = 0; //number of times a letter is in the word for (int index = 0; index < length; index++) { if (wordToGuessChars[index] == currentGuessChar) { censor[index] = currentGuessChar; //replaces * with guessed letter in caps correct = true; times++; } } if (correct) { System.out.println("The letter " + currentGuessChar + " is in the secret word! There are " + times + " " + currentGuessChar + " "s in the word. Revealing the letter(s): "); } else { System.out.println("Sorry, the letter is not in the word. Your secret word: "); } System.out.println(); } attempts++; } System.out.println("You guessed the entire word " + wordToGuess.toUpperCase() + " correctly! It took you " + attempts + " attempts!"); } } 

Kommentit

  • toinen ehdotus: Voi olla mielenkiintoista tallentaa arvaus char merkkijonon sijaan, mutta se muuttaa joitain pieniä asioita logiikassa …

Vastaa

Kiitos, että jaoit koodisi! Se näyttää melko hyvältä, mutta (kuten aina) on joitain korjattavia ja kiinnitettäviä asioita:

  1. Useiden Scanner -ilmentymien luominen: Älä ”t. Tarvitset vain yhden. Useamman kuin yhden luominen vie vain enemmän tilaa. Sen sijaan, että luot typed word ja guess, luo vain yksi nimeltään input tai jotain sellaista.

  2. Sulje Scanners: Tee aina Joten kun olet valmis käyttämään niitä, muuten saat varoituksen ”Resurssivuoto”. Scanner -kohdan sulkeminen tekee siitä niin, että Scanner ei voida käyttää uudelleen. Se on kuin sammuttaa valot, kun poistut huoneesta, ei ole mitään syytä jättää niitä päälle. Se on vain tuhlausta, jos et.

  3. == -toiminnon käyttö boolean kanssa. Käytä == -kohdan sijaan !, näin:

    if(condition) { //if "condition" is true. 

    tai

    if(!condition) { //if "condition" is false 

Vastaa

Sähköpostiosoitettasi ei julkaista. Pakolliset kentät on merkitty *