csv 파일에서 데이터를 추출하고 2D 배열에 넣기 - 리팩토링

Question

CSV 파일에서 데이터를 읽어야하고 2D 배열에 배치하는 편이 훨씬 좋습니다 (이 "일정"데이터로 작업하는 가장 쉬운 방법입니다) . 각 파일 라인은 다음과 같은 형식으로 정보를 포함 : 예를 들면 다음과

Instructor, Course, Group, Student, Result

같이

Paul Schwartz,Introduction to Computer Architecture,I1,Ben Dunkin,88 
Muhamed Olji,Object Oriented Programming,I4,Mike Brown,73 

을하지만 내 코드가 약간 단순화를 필요로한다. 그러나 나는 그것을 더 쉽게 만드는 방법을 모른다. 너.

코드 :

private String[][] fileContent(String pathToCSVFile) { 
    final int ROWS = 100; 
    final int COLUMNS = 5; 
    String fileData[][] = new String[ROWS][COLUMNS]; 
    Scanner scanner = new Scanner(pathToCSVFile); 
    boolean done = false; 
    int i, j; 

    while (!done) { 
     for (i = 0; i >= 0; i++) { 
      for (j = 0; j >= 0; j++) { 
       String str[] = scanner.nextLine().split(","); 
       for (int element = 0; element < str.length; element++) { 
        fileData[i][element] = str[element]; 
        if (i >= ROWS) { 
         Arrays.copyOf(fileData, fileData.length * 2); 
        } 
       }     
      } 
     } 

     if (!scanner.hasNextLine()) done = true; 
    } 

    return fileData; 
} 

• 어떻게 더 단순화를 위해이 코드 조각을 리팩토링하는?
• 부분적으로 채워진 배열 (Arrays.copyOf(fileData, fileData.length * 2))에 대한 더 좋은 방법이 있습니까?

Answer 1

같은

무엇인가, 당신은 모든 라인을 포함하는 목록을 얻을 수 및 배열로 변환 (또는 목록을 유지) 할 수 있습니다

try (CSVReader reader = new CSVReader(new BufferedReader(
      new FileReader(pathToCSVFile)));) { 

    List<String[]> lines = reader.readAll(); 
    return lines.toArray(new String[lines.size()][]); 
} 

(사용하여 Java 7 시도 -with-resources 구문)

Answer 2

먼저 루프 용으로주의하십시오. 그들은 "거의"정의되지 않은 루프입니다. 왜냐하면 i, j = 0으로 시작하고> = 0 인 동안 루프합니다 (항상 음수로 오버플로 할 때까지). 어쨌든 왜 필요합니까? 나는 너와 함께 생각하고 (요소) 너는 끝났다고 생각해? 그런 뭔가 이유를 ArrayList에 같은 객체를 사용하지 않는, 그런데

private String[][] fileContent(String pathToCSVFile) { 
    final int ROWS = 100; 
    final int COLUMNS = 5; 
    String fileData[][] = new String[ROWS][COLUMNS]; 
    Scanner scanner = new Scanner(pathToCSVFile); 
    boolean done = false; 
    int i=0; 

    while (!done) { 
     String str[] = scanner.nextLine().split(","); 
     for (int element = 0; element < str.length; element++) { 
      fileData[i][element] = str[element]; 
      if (i >= ROWS) { 
       Arrays.copyOf(fileData, fileData.length * 2); 
      } 
     } 
     if (!scanner.hasNextLine()) 
      done = true; 
     else 
      i++; 
    } 
    return fileData; 
} 

(나는 시도하지 않은, 단지 개념을 설명하는 것입니다)? 그것은 당신의 삶을 편하게 해줄 것이므로 메모리 처리에 대해 걱정할 필요가 없습니다. 새로운 객체를 추가하기 만하면됩니다. openCSV를 사용하여 ArrayList <ArrayList <String>>