Добавление объектов файла в массив

Question

У меня, похоже, проблема с неправильным синтаксисом моего кода, но, как я только начинал с изучения, похоже, что я пропущу ошибку. Это домашнее задание, где мне нужно использовать массив объектов JxploreFile. Это часть кода у меня возникают проблемы с:

private JxploreFile[] getSubFolders() 
{ 
    File subFiles[]; 
    subFiles = file.listFiles(); 

    File subFolders[]; 
    int p = 0; 
    for(int i = 0; i < subFiles.length; i++) 
    { 
     if(subFiles[i].isDirectory() == true) 
     { 
      Array.set(subFolders, p, subFiles[i]); 
     } 
    } 

    JxploreFile foldersToReturn[] = new JxploreFile[subFolders.length]; 
    for(int i=0; i < subFolders.length; i++) 
    { 
     foldersToReturn[i] = new JxploreFile(subFolders[i]); 
    } 
    return foldersToReturn; 
} 

В частности, для цикла, где я пытаюсь добавить файлы, помеченные как .isDirectory в новый массив. Я также пробовал другие методы, помещая каждый новый файл из массива subFiles вручную в массив подпапок, объявляя indexnumbers, но это также оказалось ошибочным. На данный момент у меня нет идей, и я надеюсь, что есть кто-то, кто может указать мне на очевидное, так как я, вероятно, пропустил что-то очень важное.

Редактировать: Прошу прощения за неполный пост. Это первый раз, когда я нахожусь здесь, поскольку обычно стараюсь отфильтровывать свои собственные проблемы из сообщений других. Ошибка, которую я получил, действительно заключалась в том, что «subFolders» еще не были инициализированы, чего я не понял, потому что на шестой строке я написал File subFolders []; , который, насколько мне известно, должен объявлять переменные подпапки, чтобы стать массивом, или это где я ошибся? Кроме того, мой вопрос, возможно, не был достаточно конкретным, я искал причину ошибки (о которой я вообще не упоминал): почему «subFiles» не был инициализирован.

Answer 1

Массив subFolders не был инициализирован должным образом. Чтобы использовать массив в методе Array.set, он должен быть инициализирован и выделен с размером.

Альтернативный подход для этого заключается в использовании List. Списки хороши, когда вы работаете с более динамичными данными, например. когда вы не знаете размер массива. После этого вы можете упростить код так:

File[] subFiles = file.listFiles(); 

// Create the list 
List<JxploreFile> subFolders = new ArrayList<>(); 

// Add all the sub folders (note that "file" is a bit magic since it 
// is not specified anywhere in the original post 
for (File subFile : file.listFiles()) { 
    if (subFile.isDirectory()) { 
     subFolders.add(new JxploreFile(subFile)); 
    } 
} 

// Return an array 
return subFolders.toArray(new JxploreFile[subFolders.size()]); 

Вы также можете упростить все дело еще дальше, используя поток Java 8, как это:

return Arrays.stream(file.listFiles()) 
      .filter(File::isDirectory) 
      .toArray(JxploreFile[]::new); 

Для получения дополнительной информации:

• Creating Objects in Java
• Arrays

Answer 2

Там нет сомнений в вашем посте, но в любом случае здесь проблемы я нашел в своем коде:

File subFolders[]; 
int p = 0; 
for(int i = 0; i < subFiles.length; i++) 
{ 
    if(subFiles[i].isDirectory() == true) 
    { 
     Array.set(subFolders, p, subFiles[i]); 
    } 
} 

при вызове Array.set вы никогда не инициализируется subFolders, который бросит NullPointerException.

Кроме того, вам не нужно делать

if(subFiles[i].isDirectory() == true) 

вы можете просто сделать

if(subFiles[i].isDirectory()) 

, как subFiles[i].isDirectory() уже условие.