Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Android Android Data Persistence File Storage Writing to Internal Storage

The following code is similar to what you just wrote. Modify this code to only write the file if it does not already exi

i keep getting errors but i dont see were im going wrong

FileUtilities.java
import java.io.InputStream;
import java.io.FileNotFoundException;
import java.io.IOException;

public class FileUtilities {

  public static boolean copyResult;

  public static void saveAssetImage(Context context, String assetName) {
    File fileToWrite = new File(context.getFilesDir(), assetName);
    AssetManager assetManager = context.getAssets();
if(fileToWrite.exists()){
        copyResult = copyFile(in, out);
      }

    try {
      InputStream in = assetManager.open(assetName);
      FileOutputStream out = new FileOutputStream(fileToWrite);
      copyResult = copyFile(in, out);
    } catch(FileNotFoundException e) {
      e.printStackTrace();
    } catch (IOException e) {
      e.printStackTrace();
    }
  }

  private static boolean copyFile(InputStream in, FileOutputStream out) {
    // Copy magic intentionally omitted
    return true;
  }
}

1 Answer

Ryan Ruscett
Ryan Ruscett
23,309 Points

hey, you are close. See, the error you get is "what is in and what is out?" Because they are not defined yet. You are copying a file if the file does not exist.

file.exists() returns true if it does exist, we only want to copy when it does NOT exist.

so (!file.exists()) copy the files.

This has to go underneath the input and output streams or else I have not yet defined what to copy. So check out the code I have below.

import java.io.InputStream;
import java.io.FileNotFoundException;
import java.io.IOException;

public class FileUtilities {

  public static boolean copyResult;

  public static void saveAssetImage(Context context, String assetName) {
    File fileToWrite = new File(context.getFilesDir(), assetName);
    AssetManager assetManager = context.getAssets();

  try {
      InputStream in = assetManager.open(assetName);
      FileOutputStream out = new FileOutputStream(fileToWrite);

  ////////////////////////////////////////////////////////////////    
      if(!fileToWrite.exists()) {
        copyResult = copyFile(in, out);
      }
   ///////////////////////////////////////////////////////////////  

 } catch(FileNotFoundException e) {
      e.printStackTrace();
    } catch (IOException e) {
      e.printStackTrace();
    }
  }

 private static boolean copyFile(InputStream in, FileOutputStream out) {
    // Copy magic intentionally omitted
    return true;
  }
}

aww thanx alot it worked!!