0

誰かが私のコードがクラッシュする理由を説明してもらえますか..

Camera.Parameters p = camera.getParameters(); //crashing line

コードは

package it.elec29a.ultra.flashlight;

import java.io.IOException;
import java.util.Random;

import android.app.PendingIntent;
import android.appwidget.AppWidgetManager;
import android.appwidget.AppWidgetProvider;
import android.content.ComponentName;
import android.content.Context;
import android.content.Intent;
import android.content.pm.PackageManager;
import android.hardware.Camera;
import android.hardware.Camera.Parameters;
import android.util.Log;
import android.widget.RemoteViews;
import android.widget.Toast;

public class MyWidgetProvider extends AppWidgetProvider {

    private Camera camera=null;
  @Override
  public void onUpdate(Context context, AppWidgetManager appWidgetManager,
      int[] appWidgetIds) {

    PackageManager pm = context.getPackageManager();
        // if device support camera?
        if (!pm.hasSystemFeature(PackageManager.FEATURE_CAMERA)) {
            return;
        }
if(camera!=null) camera=Camera.open();


    // Get all ids
    ComponentName thisWidget = new ComponentName(context,
        MyWidgetProvider.class);
    int[] allWidgetIds = appWidgetManager.getAppWidgetIds(thisWidget);
    for (int widgetId : allWidgetIds) {
        Log.d("In Widget ",""+widgetId);



        RemoteViews remoteViews = new RemoteViews(context.getPackageName(),
          R.layout.widget_layout);

      // Register an onClickListener
      Intent intent = new Intent(context, MyWidgetProvider.class);

      intent.setAction(AppWidgetManager.ACTION_APPWIDGET_UPDATE);
      intent.putExtra(AppWidgetManager.EXTRA_APPWIDGET_IDS, appWidgetIds);

      PendingIntent pendingIntent = PendingIntent.getBroadcast(context,
          0, intent, PendingIntent.FLAG_UPDATE_CURRENT);
      remoteViews.setOnClickPendingIntent(R.id.buttonwidget, pendingIntent);
      appWidgetManager.updateAppWidget(widgetId, remoteViews);
    }
    Camera.Parameters p = camera.getParameters(); //crashing line
    if(true) return; //for debug exit

    String flashMode = p.getFlashMode();
    if(flashMode == null) {
     //DO something, if it has no value
    } else if(flashMode.equals(Camera.Parameters.FLASH_MODE_OFF)) {
        p.setFlashMode(Parameters.FLASH_MODE_TORCH);
        camera.setParameters(p);
    } else if(flashMode.equals(Camera.Parameters.FLASH_MODE_TORCH)) {
        p.setFlashMode(Parameters.FLASH_MODE_OFF);
        camera.setParameters(p);
    }

  }
}

誰かがこの問題を解決するのを手伝ってくれませんか... thx

4

1 に答える 1

1

コードに少なくとも 2 つの目に見える問題があります。

あなたの行：

if(camera!=null) camera=Camera.open();

...Cameraそうでない場合はオブジェクトを初期化しますnull（奇妙に聞こえます-初期化されない可能性があります）。

ただし、以下を含むすべてのコード:

Camera.Parameters p = camera.getParameters();

...という事実に依存しないため、ポインターcamera != nullを参照している可能性があります。null

したがって、あなたは非常によく投げるかもしれませんNullPointerException（スタックトレースを投稿していないので、「かもしれない」と言います）。

私はあなたのコードを次のように変更します：

// removed if(camera!=null) 
camera = Camera.open();

// checking "camera" is not null
if (camera != null) { // using curly brackets, comes very handy sometimes :D
    // ... the rest of the code
}
于 2013-08-18T14:23:05.937 に答える