0

3 つのビューのレイアウトを膨らませるフラグメントがありEditTextます。ユーザーの入力を収集した後、そのデータを SQLite データベースに保存したいと考えています。[保存] ボタンはオプション メニューにあり、フラグメントを呼び出す同じアクティビティによって拡張されます。

データをデータベースに保存しようとすると、次の NPE が表示されます。

NullPointerException: Attempt to invoke virtual method 'android.text.Editable android.widget.EditText.getText()' on a null object reference

[保存] をクリックすると、3 つのフィールドすべてに有効なデータが入力されています。

また、フラグメントとアクティビティにどのロジックを配置する必要があるかについても混乱しています。現在、すべてのロジックはアクティビティにあり、フラグメントは実際にはレイアウトを膨らませるだけです。

編集活動

public class EditActivity extends ActionBarActivity implements DatePickerDialog.OnDateSetListener{

    EditText mTitleText;
    EditText mDueDate;
    EditText mTaskNotes;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_edit);
    }


    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.menu_edit, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        // Handle action bar item clicks here. The action bar will
        // automatically handle clicks on the Home/Up button, so long
        // as you specify a parent activity in AndroidManifest.xml.
       switch (item.getItemId()) {
           case R.id.action_settings:
               return true;
           case R.id.action_save:
               insertNewTask(findViewById(R.id.task_title));
               return true;
           case R.id.action_delete:
               startActivity(new Intent(this, MainActivity.class));
               return true;
           default:
               return super.onOptionsItemSelected(item);
       }
    }

    @Override
    public void onDateSet(DatePicker view, int year, int month, int day) {
        //do some stuff for example write on log and update TextField on activity
        String myFormat = "MM/dd/yy"; //In which you need put here
        SimpleDateFormat sdf = new SimpleDateFormat(myFormat, Locale.US);
        Calendar calendar = Calendar.getInstance();
        calendar.set(year, month, day);
        ((EditText) findViewById(R.id.task_due_date)).setText(sdf.format(calendar.getTime()));
    }

    public void showDatePickerDialog(View v) {
        DialogFragment newFragment = new DatePickerFragment();
        newFragment.show(getFragmentManager(), "datePicker");
    }

    private void insertNewTask(View view) {
        Task task = new Task();
        //TODO this is horribles and only for test
        task.setId("TaskId" + Math.random());
        mTitleText = ((EditText) view.findViewById(R.id.task_title));
        mDueDate = ((EditText) view.findViewById(R.id.task_due_date));
        mTaskNotes = ((EditText) view.findViewById(R.id.task_notes));

        task.setTitle(mTitleText.getText().toString());

        //Holy Shit do the date dance.
        try {
            SimpleDateFormat sdf = new SimpleDateFormat();
            Date taskDueDate = sdf.parse(mDueDate.getText().toString());
            DateTime googleDate = new DateTime(taskDueDate);
            task.setDue(googleDate);
        } catch (ParseException ex) {
            Logger.getLogger("EDITFRAGMENT").log(Level.SEVERE, null, ex);
        }

        task.setNotes(mTaskNotes.getText().toString());

        boolean insertSuccess = new TaskTableController(this).insertRow(task);
        if(insertSuccess){
            Toast.makeText(this, "Task information was saved.", Toast.LENGTH_SHORT).show();
        }else {
            Toast.makeText(this, "Unable to save task information.", Toast.LENGTH_SHORT).show();
        }
    }

編集フラグメント

public class EditFragment extends Fragment {

    public EditFragment() {
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        return inflater.inflate(R.layout.fragment_edit, container, false);
    }

TaskTableController

public class TaskTableController extends TaskDbHelper {

    public TaskTableController(Context context) {
        super(context);
    }

    /***
     *
     * @param task
     * @return
     */
    public boolean insertRow(Task task) {
        ContentValues values = new ContentValues();
        values.put("_id",task.getId());
        values.put("title",task.getTitle());
        values.put("due", task.getDue().toString());
        values.put("notes", task.getNotes());

        SQLiteDatabase db = this.getWritableDatabase();
        boolean createSuccessful = db.insert(TaskContract.TaskEntry.TABLE_NAME, null, values) > 0;
        db.close();
        return createSuccessful;
    }
}

NPE がスローされ、どちらの Toasts も表示されない時点で、Activity のfrominsertRowによってラップされている呼び出しを行っています。3 つのフィールドすべてにデータが入力された後でも、null オブジェクト参照を取得するのはなぜですか? そして、従うべきより良いパターンはありますか?現在、私のフラグメントにはロジックが含まれていません。insertNewTaskOnOptionsItemSelected

4

1 に答える 1