仅返回ArrayList中的最后一个元素

时间:2018-01-16 17:43:27

标签: java arraylist try-catch java.util.scanner getter-setter

我一直在教自己java,我一直坚持一个问题,无论我做什么似乎都无法解决。我做过一些研究,但所提供的所有选项似乎都不起作用。希望你们可以教我一些东西。

我有一个包含以下内容的.txt文件:

AccountName1:Password1  
AccountName2:Password2  
AccountName3:Password3  
AccountName4:Password4  
AccountName5:Password5   

然后读取文件的元素并将其插入List:

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public abstract class AccountFileReader {

  private static Scanner sc;

  public static void main(String[] args) {  

      try {         
        // Enables ability to find file in any OS.
           String file = File.separator + "some folder name"
                         + File.seperator + "AccNamePw.txt";

           File f = new File(file);
           sc = new Scanner(f);

           List<AccountInfo> accounts = new ArrayList<AccountInfo>();
           String name = "";
           String password = "";

           while (sc.hasNext()){
            // Reads and checks if there is a new line
               String line = sc.nextLine();
            // Creates delimiter to make the different elements on file f
               String[] details = line.split(":");
            // Initializes 1st element
               name = details[0];
            // Initializes 2nd element
               password = details[1];           
            // Creates new object "a" that has the 2 elements from each line
               AccountInfo a = new AccountInfo(name, password);
            // Adds the "a" object to the "accounts" List
               accounts.add(a);
           }
        // Iterates list and prints out the list
           for(AccountInfo a: accounts){
        // The hiccup is in here somewhere. This for loop isn't working in 
        // a way I think it's supposed to.

        // Create new object of the getter, setter class to use in this loop
           AccountInfo namPw = new AccountInfo(name, password);
              name = namPw.getName();
              password = namPw.getPassword();               
              System.out.println(a.toString() + "         " + name 
                                 + " " + password);
           }
        } catch (FileNotFoundException e) {e.printStackTrace();}
    }
}

getter / setter类如下:

public class AccountInfo{
private String name;
private String password;

public AccountInfo(String name, String password) {
    this.setName(name);
    this.setPassword(password);
}

public void setName(String name) { this.name = name; }

public String getName() { return name; }

public void setPassword(String password) { this.password = password; }    

public String getPassword() { return password; }

public String toString(){ return name + " "+ password; }
}

我的输出是:

AccountName1:Password1      AccountName5:Password5  
AccountName2:Password2      AccountName5:Password5  
AccountName3:Password3      AccountName5:Password5  
AccountName4:Password4      AccountName5:Password5  
AccountName5:Password5      AccountName5:Password5  

但我希望它能够回归:

AccountName1:Password1      AccountName1:Password1  
AccountName2:Password2      AccountName2:Password2  
AccountName3:Password3      AccountName3:Password3  
AccountName4:Password4      AccountName4:Password4  
AccountName5:Password5      AccountName5:Password5  

我知道a.toString()正在正确返回,但我的namPw.getName()namPw.getPassword()只是给了我列表的最后一个元素。

我不理解或缺少什么?如何让namPw.getName()namPw.getPassword()正确返回列表?

3 个答案:

答案 0 :(得分:5)

问题是在while循环之前声明namepassword。这些变量存储上次遇到的用户名和密码。当while循环结束时,这些变量分别存储值AccountName5Password5

当您输入第二个for循环时,首先使用UserAccountname创建一个新的password,其中存储AccountName5Password5。< / p>

如果您只想打印此列表,则无需创建列表内容的副本。只是做:

 for(AccountInfo a : accounts) {
     System.out.println(a.toString() + " " + a.getName() + " " + a.getPassword());
 }

答案 1 :(得分:1)

正因为如此:

for(AccountInfo a: accounts){
    **AccountInfo namPw = new AccountInfo(name, password);**
          name = namPw.getName();
          password = namPw.getPassword();               
          System.out.println(a.toString() + "         " + name 
                             + " " + password);

您正在遍历已创建的AccountInfo对象,然后创建一个新的AccountInfo对象并传入名称和密码(每次读入新行时都会设置它,因此它们的值将是他们最后的事情将在读取文件时设置)

不确定为什么要创建新的AccountInfo对象。但是为了得到你想要的东西,你需要这样做:

AccountInfo namPw = new AccountInfo(a.getName(), a.getPassword());

答案 2 :(得分:1)

无需在循环中创建新对象。你已经得到了一个对象。 删除对象创建行。它正在创建具有名称和密码的对象,它永远不会改变,因为它在循环之外。 检查以下解决方案:

for(AccountInfo a: accounts){
        name = a.getName();
        password = a.getPassword();               
        System.out.println(a.toString() + "         " + name + " " + password);
}