D'abord, je ne suis pas sûr si j'ai transmis le bon problème dans le titre de cette question, et si oui, mes excuses.java: boucle à travers deux ArrayLists sans ListIterators utilisant des types de données abstraits
Mon problème est le suivant: j'utilise une liste ADT pour simuler un jeu de sélection. L'idée est un groupe d'enfants assis en cercle et chanter une rime. Chaque enfant dit un mot de la rime jusqu'à ce que la rime soit terminée. La dernière personne à dire la rime est hors du jeu. La rime est alors recommencée depuis l'enfant suivant. Le dernier enfant restant est le gagnant.
L'application lira le nombre de joueurs dans le jeu et la rime du clavier. Deux listes seront créées, une pour les joueurs et une pour la rime. Les arraylists utilise une interface pour les méthodes. Je suis en train d'écrire le code pour boucler les deux listes, au moins une fois (je pense), afin de parcourir les joueurs le long de la rime, pour trouver qui sera éliminé. Une fois que cette personne est éliminée, la liste des joueurs va diminuer de un (la liste des rimes reste la même), et la boucle recommencera.
Ceci est la méthode doRhyme().
Mon problème est à ce jour quand je lance ce que j'ai dans le code, mon retour est le suivant:
Please enter the number of players
(It should be an integer value greater than or equal to 2):5
The players list is: 1 2 3 4 5
Please enter a rhyme:
One Two Three Four Five
Player 1: One
Player 2: Two
Player 5: Three
Player null: Four
Player null: Five
Rhyme performance error!
The players list is: 5
The winner is: 5
I utilisé Un Deux Trois Quatre Cinq comme la rime juste pour tester le code. J'espère que cela ne déroutera personne.
Le code de sortie devrait ressembler à ceci:
Please enter the number of players
(It should be an integer value greater than or equal to 2):5
The players list is: 1 2 3 4 5
Please enter a rhyme: One Two Three Four Five
Player 1: One
Player 2: Two
Player 3: Three
Player 4: Four
Player 5: Five
Removing player 5
The players list is: 1 2 3 4
Player 1: One
Player 2: Two
Player 3: Three
Player 4: Four
Player 1: Five
Removing player 1
The players list is: 2 3 4
and so on until only one player is left....
Inclus ici est le code jusqu'à présent. Toute aide serait grandement appréciée. Je continuerai à trouver le code correct entre-temps.
public class RhymeGame
{
public static void main(String args[])
{
ListInterface<Integer> players = null;
ListInterface<String> rhyme = null;
int max;
int position = 1;
max = getInt();
players = new AList<Integer>();
for(int i = 1; i <= max; i++)
{
players.add(i);
}
System.out.println("The players list is: " + players);
rhyme = getRhyme();
while(players.getLength() > 1)
{
position = doRhyme(players, rhyme, position);
System.out.println("The players list is: " + players);
System.out.println();
}
System.out.println("The winner is: " + players.getEntry(1));
}//end of main
//Requires user to input the number of players.
private static int getInt()
{
Scanner input;
int result = 10; //default value is 10
try
{
System.out.print("Please enter the number of players\n(It should be an integer value greater than or equal to 2):");
input = new Scanner(System.in);
result = input.nextInt();
}
catch(NumberFormatException e)
{
System.out.println("Could not convert input to an integer");
System.out.println(e.getMessage());
System.out.println("Will use 10 as the default value");
}
catch(Exception e)
{
System.out.println("There was an error with System.in");
System.out.println(e.getMessage());
System.out.println("Will use 10 as the default value");
}
return result;
}//end of getInt
//Requires user to input a rhyme
//(does not necessarily have to rhyme)
private static ListInterface<String> getRhyme()
{
Scanner input;
Scanner rhymeWords;
String inString;
ListInterface<String> rhyme = new AList<String>();
try
{
input = new Scanner(System.in);
System.out.println("Please enter a rhyme:");
inString = input.nextLine().trim();
rhymeWords = new Scanner(inString);
while(rhymeWords.hasNext())
{
rhyme.add(rhymeWords.next());
}
}
catch(Exception e)
{
System.out.println("Input error!");
System.out.println(e.getMessage());
getRhyme();
}
//At least one word in the rhyme
if(rhyme.getLength() < 1)
{
System.out.println("You must enter at least one word in the rhyme");
getRhyme();
}
return (ListInterface<String>)rhyme;
}//end of getRhyme
//Loops through the rhyme with the players in the list,
//removing the selected player at the end of the rhyme.
//
//players = the list holding the players
//rhyme = the list holding the words of the rhyme
//startAt = a position to start the rhyme at
//
//the position of the player eliminated will be returned.
public static int doRhyme(ListInterface<Integer> players, ListInterface<String> rhyme, int startAt)
{
int numPlayers;
try
{
numPlayers = players.getLength();
while(numPlayers > 2)
{
for(startAt = 1; startAt < players.getLength(); startAt++)
{
for(startAt = 1; startAt < rhyme.getLength() + 1; startAt++)
{
System.out.println("Player " + players.getEntry(startAt) + ": " + rhyme.getEntry(startAt));
numPlayers = players.remove(--numPlayers);
}
}
}
}
catch(Exception e)
{
System.out.println("Rhyme performance error!");
}
return startAt;
}//end of doRhyme
}//end of class
L'interface pour la liste est la suivante (au cas où quelqu'un voudrait voir les méthodes avec lesquelles je dois travailler).
/**
An interface for the ADT list.
Entries in the list have positions that begin with 1.
*/
public interface ListInterface<T>
{
/** Adds a new entry to the end of this list.
Entries currently in the list are unaffected.
The list's size is increased by 1.
@param newEntry the object to be added as a new entry */
public void add(T newEntry);
/** Adds a new entry at a specified position within this list.
Entries originally at and above the specified position
are at the next higher position within the list.
The list's size is increased by 1.
@param newPosition an integer that specifies the desired
position of the new entry
@param newEntry the object to be added as a new entry
@return true if the addition is successful, or
false if newPosition < 1, or newPosition > getLength() + 1 */
public boolean add(int newPosition, T newEntry);
/** Removes the entry at a given position from this list.
Entries originally at positions higher than the given
position are at the next lower position within the list,
and the list's size is decreased by 1.
@param givenPosition an integer that indicates the position of
the entry to be removed
@return a reference to the removed entry or null, if either
the list was empty, givenPosition < 1, or
givenPosition > getLength() */
public T remove(int givenPosition);
/** Removes all entries from this list. */
public void clear();
/** Replaces the entry at a given position in this list.
@param givenPosition an integer that indicates the position of
the entry to be replaced
@param newEntry the object that will replace the entry at the
position givenPosition
@return true if the replacement occurs, or false if either the
list is empty, givenPosition < 1, or
givenPosition > getLength() */
public boolean replace(int givenPosition, T newEntry);
/** Retrieves the entry at a given position in this list.
@param givenPosition an integer that indicates the position of
the desired entry
@return a reference to the indicated entry or null, if either
the list is empty, givenPosition < 1, or
givenPosition > getLength() */
public T getEntry(int givenPosition);
/** Sees whether this list contains a given entry.
@param anEntry the object that is the desired entry
@return true if the list contains anEntry, or false if not */
public boolean contains(T anEntry);
/** Gets the length of this list.
@return the integer number of entries currently in the list */
public int getLength();
/** Sees whether this list is empty.
@return true if the list is empty, or false if not */
public boolean isEmpty();
/** Retrieves all entries that are in this list in the order in which
they occur in the list. */
public T[] toArray();
}
Les définitions de AList sont les suivantes:
/**
A class that implements the ADT list by using an array.
The list is unbounded.
*/
public class AList<T> implements ListInterface<T>
{
private T[] list; // array of list entries
private int numberOfEntries;
private static final int DEFAULT_INITIAL_CAPACITY = 25;
public AList()
{
this(DEFAULT_INITIAL_CAPACITY);
}//end of constructor
public AList(int initialCapacity)
{
numberOfEntries = 0;
T[] tempList = (T[])new Object[initialCapacity];
list = tempList;
}//end of constructor
public void add(T newEntry)
{
boolean isSuccessful = true;
if(!isFull())
{
//ensureCapacity();
list[numberOfEntries] = newEntry;
numberOfEntries++;
}
else
{
isSuccessful = false;
}
}//end of add
public boolean add(int newPosition, T newEntry)
{
boolean isSuccessful = true;
if(!isFull() && (newPosition >= 1) && (newPosition <= numberOfEntries + 1))
{
ensureCapacity();
makeRoom(newPosition);
list[newPosition - 1] = newEntry;
numberOfEntries++;
}
else
{
isSuccessful = false;
}
return isSuccessful;
}//end of add overload
public T remove(int givenPosition)
{
T result = null; // return value
if((givenPosition >= 1) && (givenPosition <= numberOfEntries))
{
assert !isEmpty();
result = list[givenPosition - 1]; // get entry to be removed
// move subsequent entries towards entry to be removed,
// unless it is last in list
if(givenPosition < numberOfEntries)
{
removeGap(givenPosition);
}
numberOfEntries--;
} // end if
return result; // return reference to removed entry, or
// null if either list is empty or givenPosition
// is invalid
}//end of remove
public void clear()
{
for(int index = 0; index < numberOfEntries; index++)
{
list[index] = null;
}
numberOfEntries = 0;
}//end of clear
public boolean replace(int givenPosition, T newEntry)
{
boolean isSuccessful = true;
if(!isFull() && (givenPosition >= 1) && (givenPosition <= numberOfEntries)) // test catches empty list
{
assert !isEmpty();
list[givenPosition - 1] = newEntry;
}
else
{
isSuccessful = false;
}
return isSuccessful;
}//end of replace
public T getEntry(int givenPosition)
{
T result = null; // result to return
if((givenPosition >= 1) && (givenPosition <= numberOfEntries))
{
assert !isEmpty();
result = list[givenPosition - 1];
}
return result;
}//end of getEntry
public boolean contains(T anEntry)
{
boolean found = false;
for(int index = 0; !found && (index < numberOfEntries); index++)
{
if(anEntry.equals(list[index]))
{
found = true;
}
}
return found;
}//end of contains
public int getLength()
{
return numberOfEntries;
}//end of getLength
public boolean isEmpty()
{
return numberOfEntries == 0;
}//end of isEmpty
public boolean isFull()
{
return numberOfEntries == list.length;
}
public T[] toArray()
{
T[] result = (T[])new Object[numberOfEntries];
for(int index = 0; index < numberOfEntries; index++)
{
result[index] = list[index];
}
return result;
}//end of toArray
public String toString()
// Returns a nicely formatted string that represents this list.
{
String listString = "";
for(int i = 0; i < numberOfEntries; i++)
{
listString = listString + " " + list[i];
}
return listString;
}//end of toString
// Doubles the size of the array list if it is full.
private void ensureCapacity()
{
if(numberOfEntries == list.length)
{
list = Arrays.copyOf(list, 2 * list.length);
}
}//end of ensureCapacity
// Makes room for a new entry at newPosition.
// Precondition: 1 <= newPosition <= numberOfEntries + 1;
// numberOfEntries is list's length before addition.
private void makeRoom(int newPosition)
{
assert(newPosition >= 1) && (newPosition <= numberOfEntries + 1);
int newIndex = newPosition - 1;
int lastIndex = numberOfEntries - 1;
// move each entry to next higher index, starting at end of
// list and continuing until the entry at newIndex is moved
for(int index = lastIndex; index >= newIndex; index--)
{
list[index + 1] = list[index];
}
}//end of makeRoom
// Shifts entries that are beyond the entry to be removed to the
// next lower position.
// Precondition: 1 <= givenPosition < numberOfEntries;
// numberOfEntries is list's length before removal.
private void removeGap(int givenPosition)
{
assert(givenPosition >= 1) && (givenPosition < numberOfEntries);
int removedIndex = givenPosition - 1;
int lastIndex = numberOfEntries - 1;
for(int index = removedIndex; index < lastIndex; index++)
{
list[index] = list[index + 1];
}
}//end of removeGap
}//end of class
Vous devez utiliser cette ligne dans vos prises: 'System.out.println ("! Erreur de performance Rhyme \ n" + e.getMessage());' 'ou e.printStackTrace() '. Et puis ajoutez la sortie ici. – ChiefTwoPencils
Je l'ai eu de cette façon à l'origine, mais il ajoute seulement "null" au-dessous de la "erreur de performance rime!" – Elekidcore
AList n'est pas défini dans le code, votre code a besoin d'être refactorisé. – firephil