[JamesAngie]

Hello,

Okay...I fixed it so that it displayes all of the user names at the end now, but I only want it to display the last entry...any ideas....

I am trying to fix an IndexOutOfBoundsException, I am not sure what I need to do. Please look and advise. Thanks



import java.io.*;
import java.util.ArrayList;


public class UserList
{

public static void main(String[] args) throws IOException
{
//declare variables
String str1, str2 = "username";
int index;
int initialCapacity = 5;
BufferedReader dataIn = new BufferedReader(new InputStreamReader(System.in));
ArrayList users = new ArrayList();

//get user input
System.out.print("Enter a user name: ");
str1 = dataIn.readLine();

//compare user input
while(str1.length() > 0)
{
//check for "username"
if(str1 == str2)
System.out.println("That user name is NOT allowed!");
else
{
//check for capacity of 5
if(users.size() == initialCapacity)
{
System.out.println("List is full!");
//okay I got it to work but not able to list the last user just all of them
System.out.println("Last entry is " +users);
}
else//check if user is in list already
if(!users.contains(str1))
{
users.add(str1);
System.out.println("User \""+str1+"\" added to user list.");
}
else//
System.out.println("User \""+str1+"\" already in user list.");
}

System.out.print("\nEnter a user name: ");
str1 = dataIn.readLine();
}
//completion of program
System.out.println("Program complete.");

}
}

This post has been edited by JamesAngie: 25 March 2008 - 11:01 AM

[JamesAngie]

Does anyone have any ideas about this? Thanks

[letthecolorsrumble]

The first problem was that the while-loop never exited.

So, the program never executed the statements outside the loop

Some of the fixes might be helpful.

import java.io.*;
import java.util.ArrayList;


public class UserList {

public static void main(String[] args) throws IOException {
	//declare variables
	String str1, str2 = "username";
	int index;
	int initialCapacity = 5;
	BufferedReader dataIn = new BufferedReader(new InputStreamReader(System.in));
	ArrayList users = new ArrayList();
	
	//get user input
	System.out.print("Enter a user name: ");
	str1 = dataIn.readLine();
	
	//compare user input
	while(str1.length() > 1) { //ends when only enter is pressed
		//check for "username"
		if(str1 == str2)
			System.out.println("That user name is NOT allowed!");
		else {
			//check for capacity of 5
			if(users.size() == initialCapacity) {
				System.out.println("List is full!");
				//okay I got it to work but not able to list the last user just all of them
				System.out.println("Last entry is " +users);
			}
			else//check if user is in list already
				if(!users.contains(str1)) {
				users.add(str1);
				System.out.println("User \""+str1+"\" added to user list.");
				}
			else//
			System.out.println("User \""+str1+"\" already in user list.");
		}
		
		System.out.print("\nEnter a user name: ");
		str1 = dataIn.readLine();
	}
	//completion of program
	System.out.println("Program complete.");
	for(int i = 0 ; i < users.size(); i++) {
		System.out.println(users.get(i));
	}
	}
}

more help? keep posting.!

This post has been edited by letthecolorsrumble: 25 March 2008 - 12:47 PM

[GWatt]

This code works just fine for me. What line does it say you have a problem?

[JamesAngie]

GWatt, on 25 Mar, 2008 - 01:00 PM, said:

This code works just fine for me. What line does it say you have a problem?

It works fine...but I need it to only list the last user

in line 34

System.out.println("Last entry is " +users);

I don't know how to make it list ony the last person I entered. Currently it lists all the names I enter.

[capty99]

users.get(4));

http://java.sun.com/.../ArrayList.html

[JamesAngie]

capty99, on 25 Mar, 2008 - 01:23 PM, said:

users.get(4));

http://java.sun.com/.../ArrayList.html

Thanks a million. I was adding [5] instead of (4)...

[KoryLovre]

Yes everyone - please remember that Java indexing is zero (0) based - this means you start counting at zero instead of 1. So if the total number of elements in your array is 5 then you index the 5 array elements as such:

first element index = 0 (users.get(0);
second element index = 1 (users.get(1);
third element index = 2 (users.get(2);
fourth element index = 3 (users.get(3);
fifth (last) element index = 4 (users.get(4);

Thus, trying to refer to an array with an index of any negative number or an index equal to or greater than 5 in this case will throw an IndexOutOfBoundsException as James was seeing.