-1

Here's my code:

    import android.content.Context;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.TextView;

import org.joda.time.YearMonth;

import java.util.ArrayList;

public class DayAdapter extends BaseAdapter {
    private int startDay;
    private int endDay;
    Context mContext;
    LayoutInflater mInflater;
    private ArrayList<Integer> theList = new ArrayList<Integer>();

    public DayAdapter(Context c, int sD, int eD){
        startDay = sD;
        endDay = eD;
        mContext = c;
        mInflater = LayoutInflater.from(c);
        if (sD > eD){
            startDay = eD;
            endDay = sD;

        }
        for (int i = startDay; i<= endDay; i++){
            theList.add(new Integer(i));
        }
    }

    @Override
    public int getCount() {
        return theList.size();
    }

    @Override
    public Object getItem(int position) {
        return theList.get(position);
    }

    @Override
    public long getItemId(int position) {
        return 0;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        View newView = convertView;
        DayHolder holder;
        Integer curr = (Integer) theList.get(position);
        if (null == convertView){
            holder = new DayHolder();
            newView = mInflater.inflate(R.layout.activity_select_day,null);
            holder.mText = (TextView) newView.findViewById(R.id.selectDays);
        }else{
            holder = (DayHolder) newView.getTag();
        }
        holder.mText.setText(curr.toString());
        return newView;
    }

    static class DayHolder{
        TextView mText;
    }
}

And here's the error message:

11-30 12:49:58.030  15317-15317/apps.me.jj.recordtimeeveryhour E/AndroidRuntime﹕ FATAL EXCEPTION: main
    java.lang.NullPointerException
            at apps.me.jj.recordtimeeveryhour.DayAdapter.getView(DayAdapter.java:63)
            at android.widget.HeaderViewListAdapter.getView(HeaderViewListAdapter.java:220)
            at android.widget.AbsListView.obtainView(AbsListView.java:2031)
            at android.widget.ListView.makeAndAddView(ListView.java:1820)
            at android.widget.ListView.fillUp(ListView.java:705)
            at android.widget.ListView.fillGap(ListView.java:645)
            at android.widget.AbsListView.trackMotionScroll(AbsListView.java:4607)
            at android.widget.AbsListView.scrollIfNeeded(AbsListView.java:2871)
            at android.widget.AbsListView.onTouchEvent(AbsListView.java:3136)
            at android.view.View.dispatchTouchEvent(View.java:5579)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:1959)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1720)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:1965)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1734)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:1965)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1734)
            at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:1965)
            at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1734)
            at com.android.internal.policy.impl.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:1982)
            at com.android.internal.policy.impl.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1441)
            at android.app.Activity.dispatchTouchEvent(Activity.java:2421)
            at com.android.internal.policy.impl.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:1930)
            at android.view.View.dispatchPointerEvent(View.java:5759)
            at android.view.ViewRootImpl.deliverPointerEvent(ViewRootImpl.java:2970)
            at android.view.ViewRootImpl.handleMessage(ViewRootImpl.java:2529)
            at android.view.ViewRootImpl.processInputEvents(ViewRootImpl.java:869)
            at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:883)
            at android.view.ViewRootImpl.handleMessage(ViewRootImpl.java:2505)
            at android.os.Handler.dispatchMessage(Handler.java:99)
            at android.os.Looper.loop(Looper.java:154)
            at android.app.ActivityThread.main(ActivityThread.java:4945)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:511)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
            at dalvik.system.NativeStart.main(Native Method)

The ListView loads alright.. but I keep getting this error whenever I start scrolling the display. Any ideas how to fix this without radically changing my code (i.e. adding new libraries that someone who only learned Java and Android programming a month ago might not be familiar with.)

Line 63: holder.mText.setText(curr.toString());

xjdeng
  • 49
  • 7
  • What is line 63 of `DayAdapter`? – stkent Nov 30 '14 at 21:01
  • 1
    possible duplicate of [What is a Null Pointer Exception, and how do I fix it?](http://stackoverflow.com/questions/218384/what-is-a-null-pointer-exception-and-how-do-i-fix-it) – Selvin Nov 30 '14 at 21:02
  • change holder.mText.setText(curr.toString()); to holder.mText.setText(""+curr.toString()); – Rami Nov 30 '14 at 21:04

1 Answers1

1

First, do not use LayoutInflater.from(). Use a LayoutInflater you get from your Activity. Otherwise, your layouts will not work properly.

Second, do not use inflate(R.layout.activity_select_day,null);. Use inflate(R.layout.activity_select_day, parent, false);. Otherwise, your layouts will not work properly when the root container of the layout file is a RelativeLayout.

Third, if you call getTag(), you should be calling setTag(). You are not; hence, your getTag() call will always return null. Call newView.setTag(holder) in your if block, after having created your holder instance.

CommonsWare
  • 986,068
  • 189
  • 2,389
  • 2,491
  • Thanks.. I implemented #2 and #3 (still figuring #1) and it seems to work now.. sorry about having to ask such a basic question.. this variation of code i wrote here (which I copied from elsewhere) seemed to work on several menus until i made a long list that needed scrolling. – xjdeng Nov 30 '14 at 21:16
  • @ragnarkar: "which I copied from elsewhere" -- you may wish to try to find a better "elsewhere". :-) Glad to hear that it is working for you. – CommonsWare Nov 30 '14 at 21:20