Schaltfläche in ListFragment - App stürzt bei Klick ab

Question

Die App öffnet sich gut mit der ersten Fragment wird korrekt angezeigt. Beim Wischen erscheint das nächste Layout fragment auf dem Bildschirm. Dort stürzt die App beim Klicken auf die Schaltfläche ab.

Der Fehler, die ich bekommen ist:

java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String java.lang.Object.toString()' on a null object reference 

Einige der Top-Einträge aus Fehler-Stack sind wie folgt:

at android.widget.ArrayAdapter.createViewFromResource(ArrayAdapter.java:401) 

at android.widget.ArrayAdapter.getView(ArrayAdapter.java:369) 

at android.widget.AbsListView.obtainView(AbsListView.java:2346) 

at android.widget.ListView.makeAndAddView(ListView.java:1876) 

at android.widget.ListView.fillDown(ListView.java:702) 

at android.widget.ListView.fillFromTop(ListView.java:763) 

at android.widget.ListView.layoutChildren(ListView.java:1685) 

at android.widget.AbsListView.onLayout(AbsListView.java:2148) 

at android.view.View.layout(View.java:16636) 

at android.view.ViewGroup.layout(ViewGroup.java:5437) 

Die App main activity ist wie folgt:

public class MainActivity extends FragmentActivity { 

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

    ViewPager pager = (ViewPager) findViewById(R.id.viewPager); 
    pager.setAdapter(new MyPagerAdapter(getSupportFragmentManager())); 
} 

private class MyPagerAdapter extends FragmentPagerAdapter { 

    public MyPagerAdapter(FragmentManager fm) { 
     super(fm); 
    } 

    @Override 
    public Fragment getItem(int position) { 
     switch(position) { 

      case 0: return FirstFragment.newInstance("FirstFragment, Instance 1"); 
      case 1: return SecondFragment.newInstance("SecondFragment, Instance 1"); 
      default: return FirstFragment.newInstance("FirstFragment, Default"); 
     } 
    } 

} 
} 

SecondFragment.java ist wie folgt:

public class SecondFragment extends ListFragment { 

static public ArrayAdapter<String> adapter; 
static String[] values = new String[3]; 

@Override 
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { 
    View rootView = inflater.inflate(R.layout.second_frag, container, 
      false); 

    final Button btn = (Button) rootView.findViewById(R.id.readWebpage); 
    if (btn != null) { 
     btn.setOnClickListener(new View.OnClickListener() { 
      public void onClick(View v){ 
       new HttpGetReq().execute("http://xxx.xxx.xxx.xxx"); 
       adapter = new ArrayAdapter<String>(getActivity(), 
         android.R.layout.simple_list_item_1, values); 
       setListAdapter(adapter); 
      } 
     }); 
    } 
    return rootView; 
} 

public static SecondFragment newInstance(String text) { 

    SecondFragment f = new SecondFragment(); 
    Bundle b = new Bundle(); 
    b.putString("msg", text); 
    f.setArguments(b); 

    return f; 
} 


} 

Beim Klicken auf die Schaltfläche ruft die HttpGetReq.java-Worker-Klasse Daten vom Server ab und zeigt diese der Liste an.

public class HttpGetReq extends AsyncTask<String , Void ,String> { 
String server_response; 

@Override 
protected String doInBackground(String... strings) { 

    URL url; 
    HttpURLConnection urlConnection = null; 

    try { 
     url = new URL(strings[0]); 
     urlConnection = (HttpURLConnection) url.openConnection(); 

     int responseCode = urlConnection.getResponseCode(); 

     if(responseCode == HttpURLConnection.HTTP_OK){ 
      server_response = readStream(urlConnection.getInputStream()); 
     } 
    } catch (MalformedURLException e) { 
     e.printStackTrace(); 
    } catch (IOException e) { 
     e.printStackTrace(); 
    } 
    return null; 
} 

@Override 
protected void onPostExecute(String s) { 
    super.onPostExecute(s); 
    Log.i("TAG", "Assigning new value"); 
    SecondFragment.values[1] = server_response; 
} 

// Converting InputStream to String 
private String readStream(InputStream in) { 
    BufferedReader reader = null; 
    StringBuffer response = new StringBuffer(); 
    try { 
     reader = new BufferedReader(new InputStreamReader(in)); 
     String line = ""; 
     while ((line = reader.readLine()) != null) { 
      response.append(line); 
     } 
    } catch (IOException e) { 
     e.printStackTrace(); 
    } finally { 
     if (reader != null) { 
      try { 
       reader.close(); 
      } catch (IOException e) { 
       e.printStackTrace(); 
      } 
     } 
    } 
    return response.toString(); 
} 
} 

second_Frag.xml ist wie folgt:

<?xml version="1.0" encoding="utf-8"?> 

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android" 
    android:layout_width="match_parent" 
    android:layout_height="match_parent" 
    android:orientation="vertical" 
    android:id="@+id/myView2">> 
    <Button 
     android:id="@+id/readWebpage" 
     android:layout_width="match_parent" 
     android:layout_height="wrap_content" 
     android:onClick="onClick" 
     android:text="Refresh" 
     android:layout_gravity="center_horizontal"> 
    </Button> 
    <ListView xmlns:android="http://schemas.android.com/apk/res/android" 
     android:id="@id/android:list" 
     android:background="@color/background_Color" 
     android:layout_width="fill_parent" 
     android:gravity="center" 
     android:layout_height="fill_parent"/> 
</LinearLayout> 

Answer 1

new HttpGetReq().execute("http://xxx.xxx.xxx.xxx"); 
adapter = new ArrayAdapter<String>(getActivity(), 
     android.R.layout.simple_list_item_1, values); 
setListAdapter(adapter); 

Hier sind Sie Adapter einstellen, bevor die Daten von HttpGetReq bekommen. Da es sich bei HttpGetReq um eine AsyncTask handelt, wird die Zeile unmittelbar nach dem Aufruf execute ausgeführt, und das Array bleibt leer. Sie sollten die Daten in Adapter setzen, nachdem Sie die Daten von onPostExecute Ihrer HttpGetReq erhalten.

@Override 
protected void onPostExecute(String s) { 
    super.onPostExecute(s); 
    Log.i("TAG", "Assigning new value"); 
    SecondFragment.values[1] = server_response; 

    // Set your data here 

}