Assigning constructor argument to member variable, ".this cannot be referenced from a static context"

Question

I have a class, DFragment. I'm passing a List<Map<String, String>> to its constructor, and I need to store the passed-in value in the member variable subVitalList.

I tried doing this:

this.subVitalList = subVitalList;

But I receive the following error on that line:

com.example.DFragment.this cannot be referenced from a static context

I'm not sure how to do this. Here is the code for DFragment:

public class DFragment extends DialogFragment {

    Context context;
    List<Map<String, String>> subVitalList;
    ListView vitalEntryListView;
    LayoutInflater mInflater;

    public static DFragment newInstance(List<Map<String, String>> subVitalList,int i) {
        DFragment f = new DFragment();
        // Supply num input as an argument.
        Bundle args = new Bundle();
        //args.putInt("num", num);
        args.putInt("num",i);

        //List<List<String>> svl = getArguments().getStringArrayList(subVitalList);
        return f;
    }

}

What am I doing wrong and how do I correctly store the passed-in value in my member variable?

it says com.example.DFragment.this cannot be referenced from a static context — Drunken Daddy, Apr 07 '15 at 06:21
you must pass `subVitalList` to argument and get that in `onCreateView` or other fragment's method. you can't set like `this.subVitalList = subVitalList;` because `newInstance` is `static` method, you can't set non static value to static value. — Shayan Pourvatan, Apr 07 '15 at 06:25
you must create class and implement `Parcelable` or `serializable` and use getArgument().get.... http://stackoverflow.com/questions/10953121/android-arraylistmyobject-pass-as-parcelable — Shayan Pourvatan, Apr 07 '15 at 06:30
`static DFragment newInstance(...)` is a _factory method_, not a _constructor_! — Seelenvirtuose, Apr 07 '15 at 06:31
Sorry, I do not have a class to implement Parcelable. I'm only using list.add(map) to add elements to the list. — Drunken Daddy, Apr 07 '15 at 07:11
Cant you just do `f.subVitalList = subVitalList` ? (you also may need to make a deep copy of the list instead of just holding the reference) — chancea, Apr 07 '15 at 13:43

score 4 · Accepted Answer · edited Apr 10 '15 at 04:10

4

You can try this ,

Fragment class ,

public class DFragment extends DialogFragment {

    List<Map<String, String>> mylist;

    public DFragment () {   
        // Empty constructor required for DialogFragment
    }

    public DFragment (List<Map<String, String>> mylist) {
        this.mylist= mylist;
    }

}

You can pass the list as follows:

DFragment dialog = new DFragment (mylist);

edited Apr 10 '15 at 04:10

Jason C

38,729
14
126
182

answered Apr 07 '15 at 11:16

Heshan Sandeepa

3,388
2
35
45

Assigning constructor argument to member variable, ".this cannot be referenced from a static context"

1 Answers1