Android startCamera gives me null Intent and ... does it destroy my global variable?

Question

I have the next problem:

when I try to start my camera, I can take the picture, even save it on my sdcard, but when I'm going to get the path for showing it on my device I get errors.

My global variables are 2 (I used 1 but 2 are better for making sure it's a strange error):

    private File photofile;
private Uri mMakePhotoUri;

and this is my start-camera function:

@SuppressLint("SimpleDateFormat")
public void farefoto(int num){
// For naming the picture
    SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMdd_HHmmss");
    String n = sdf.format(new Date());
    String fotoname = "Immagine-"+ n +".jpg";

//Going through files and  folders
    File photostorage = Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_PICTURES);
    File photostorage2 = new File(photostorage, "Immagini");
    System.out.println(photostorage+"\n"+photostorage2);
    photostorage2.mkdirs();
// My file (global)
    photofile = new File(photostorage2, fotoname);
    Intent i = new Intent(MediaStore.ACTION_IMAGE_CAPTURE); //intent to start camera
// My URI (global)
    mMakePhotoUri = Uri.fromFile(photofile);
    new Bundle(); // I took this code from internet, but if I remove this line, it's the same
    i.putExtra(MediaStore.EXTRA_OUTPUT, mMakePhotoUri);
    startActivityForResult(i, num); //num would be 1 on calling function
}

and my activityresult:

   @Override
    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
        // TODO Auto-generated method stub
        if (requestCode == 1){

            try{ // tring my global URI
                photo = f.decodeAndResizeFile(new File(mMakePhotoUri.getPath()));
            }
            catch(NullPointerException ex){
                System.out.println("fail");
                ex.printStackTrace();
                try{ // Trying my global FILE
                photo = BitmapFactory.decodeFile(photofile.getAbsolutePath());
                } catch (Exception e){
                    e.printStackTrace();
                    Toast.makeText(this, "C'è stato un errore. Riprova a scattare la foto.", Toast.LENGTH_LONG).show();
                }

......
......
.......
}

Always getting NullPointerException

But... if I take the picture again, IT WORKS!!.

I've read everything I could here... but it doesn't have logic when I modify a global variable and I cannot take it again...

Thanks in advance. Cheers.

SOLUTION

As Alex Cohn said, my problem was that I was calling onCreate before onActivityResult because of a probably push out of memory (because sometimes doesn't do it), so I wanted to have my app "healthy" and I tried some try / catch and so I get the data even if it's calling onCreate or onActivityResult for the first call, and I wrote that data in a Bundle like explained in the link of restoring our state.

Thanks!.

have you added permission too you manifest READ EXTERNAL STORAGE — Nitin Misra, Dec 06 '13 at 13:40

score 2 · Accepted Answer · answered Dec 06 '13 at 21:19

2

It's possible that launching of ACTION_IMAGE_CAPTURE will push your activity out of memory. You should check (I'd simply a log, debugger may have its own side effects) that onCreate() of your activity is called before onActivityResult(). If this is the case, you should prepare your activity to reinitialize itself, probably using onSaveInstanceState(Bundle).

Note that the decision whether to shut down the activity, or keep it in background, depends on the overall system state that is beyond your control. It won't surprise me if the decision when you take the first picture, is "shut him down!", but when you take picture again, it is "keep him in background".

answered Dec 06 '13 at 21:19

Alex Cohn

56,089
9
113
307

1

Yes, that's the point Alex. I'm calling `onCreate` when I get back on my activity... – Rafael Ruiz Muñoz Dec 07 '13 at 16:17
Sorry I had to comment some more: yes, it was because onCreate()..... but now I realise that it happens sometimes, not always. I don't know what to do, maybe getting the URI from the bundle on `onCreate` and checking if first it calls oncreate or not??? – Rafael Ruiz Muñoz Dec 07 '13 at 17:43
Please read http://stackoverflow.com/questions/4096169/onsaveinstancestate-and-onrestoreinstancestate for explanation how the activity should be restored. – Alex Cohn Dec 07 '13 at 17:47

score 2 · Answer 2 · answered Jun 02 '15 at 07:39

It doesnot destroy the variables. But after So many days of research LOL I have got a solution of my mistake. When I put the debugger the method calls it like after taking a picture.

onCreate()
onActivityResult()
onCeate()
onResume()

Its fixed by just put these following lines in t the menifest. It happens due to camera config changes & window soft input mode.

             <activity
                android:name="packageName.Activity"
                android:configChanges="orientation|keyboardHidden|screenSize"
                android:label="@string/app_name"
                android:screenOrientation="portrait"
                android:windowSoftInputMode="adjustResize" >

Android startCamera gives me null Intent and ... does it destroy my global variable?

SOLUTION

2 Answers2

Linked

Related