Android Fragmentについて

Question

Fragmentで複数Tabを用意して一つ目のTabにListviewを実装したいのですが何故か表示されません
いろいろサイトを回ったのですがわかりません、初心者なので簡単な見落としミスかもしれないのですが宜しくお願いします

listview.xml

<?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" >
    
    <ListView 
    android:layout_height="wrap_content" 
    android:id="@+id/listView" 
    android:layout_width="match_parent" 
    ></ListView>
    
</LinearLayout>


Fragment1.java

package com.example.fragmenttest;

import android.os.Bundle;
import android.app.Fragment;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;

public class Fragment1 extends Fragment {
  
  @Override
  public View onCreateView(
    LayoutInflater inflater,
    ViewGroup container,
    Bundle savedInstanceState) {
	  return inflater.inflate(R.layout.listview, container, false);
  }
  
}

ListActivity.java


package com.example.fragmenttest;

import android.app.Activity;
import android.os.Bundle;
import android.view.View;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.ListView;
import android.widget.Toast;

import com.example.fragmenttest.R.id;

public class ListViewActivity extends Activity {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.listview);

        ArrayAdapter<String> adapter = new ArrayAdapter<String>(this, android.R.layout.simple_list_item_1);
        // アイテムを追加します
        adapter.add("red");
        adapter.add("green");
        adapter.add("blue");
        ListView listView = (ListView) findViewById(id.listview);
        // アダプターを設定します
        listView.setAdapter(adapter);
        // リストビューのアイテムがクリックされた時に呼び出されるコールバックリスナーを登録します
        listView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view,
                    int position, long id) {
                ListView listView = (ListView) parent;
                // クリックされたアイテムを取得します
                String item = (String) listView.getItemAtPosition(position);
                Toast.makeText(ListViewActivity.this, item, Toast.LENGTH_LONG).show();
            }
        });
        // リストビューのアイテムが選択された時に呼び出されるコールバックリスナーを登録します
        listView.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
            @Override
            public void onItemSelected(AdapterView<?> parent, View view,
                    int position, long id) {
                ListView listView = (ListView) parent;
                // 選択されたアイテムを取得します
                String item = (String) listView.getSelectedItem();
                Toast.makeText(ListViewActivity.this, item, Toast.LENGTH_LONG).show();
            }
            @Override
            public void onNothingSelected(AdapterView<?> parent) {
            }
        });
    }
}

MainActivity.java


package com.example.fragmenttest;


import android.os.Bundle;
import android.app.ActionBar;
import android.app.Activity;
import android.app.FragmentTransaction;
import android.app.ActionBar.Tab;
import android.app.ActionBar.TabListener;
import android.app.Fragment;

public class MainActivity extends Activity {

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    
    final ActionBar actionBar = getActionBar();
    actionBar.setNavigationMode(ActionBar.NAVIGATION_MODE_TABS);

    actionBar.addTab(actionBar
      .newTab()
      .setText("TAB 1")
      .setTabListener(
        new MainTabListener<Fragment1>(
          this,
          "f1",
          Fragment1.class
        )
      )
    );
    actionBar.addTab(actionBar
      .newTab()
      .setText("TAB 2")
      .setTabListener(
        new MainTabListener<Fragment2>(
          this,
          "f2",
          Fragment2.class
        )
      )
    );
    }
    
  public static class MainTabListener<T extends Fragment> 
    implements TabListener {
    
    private Fragment fragment;
    private final Activity activity;
    private final String tag;
    private final Class<T> cls;
    
    public MainTabListener(
      Activity activity, String tag, Class<T> cls){
      this.activity = activity;
      this.tag = tag;
      this.cls = cls;
    }
      
    @Override
    public void onTabReselected(Tab tab, FragmentTransaction ft) {
    }

    @Override
    public void onTabSelected(Tab tab, FragmentTransaction ft) {
      if(fragment == null){
        fragment = Fragment.instantiate(activity, cls.getName());
        ft.add(android.R.id.content, fragment, tag);
      }
      else{
        ft.attach(fragment);
      }      
    }

    @Override
    public void onTabUnselected(Tab tab, FragmentTransaction ft) {
      if(fragment != null){
        ft.detach(fragment);
      }      
    }
    }
}

です。宜しくお願いします

Answer 1

とりあえず動いたので報告。
自分も勉強中の身なので間違えたことを言っているかもしれませんが、ご了承ください。

まず、Fragmentの中にListViewを表示したいならActiviteではなくてFragmentを継承しないといけないと思います。

ListViewActivity を ListViewFragment として作り直します。修正が必要なところは適時修正。
ディレクトリ構成がk135002様とは違うのでもしエラーが出たら適当な形に直してください。

ListViewFragment.java

import android.app.Fragment;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.os.Bundle;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.ListView;
import android.widget.Toast;

public class ListViewFragment extends Fragment {

	View view;

	@Override
	public View onCreateView(LayoutInflater inflater, ViewGroup container,
			Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);

		view = inflater.inflate(R.layout.listview, container, false);

		return view;
	}

	@Override
	public void onActivityCreated(Bundle savedInstanceState) {
		super.onActivityCreated(savedInstanceState);

		ArrayAdapter<String> adapter = new ArrayAdapter<String>(
				view.getContext(), android.R.layout.simple_list_item_1);
		// アイテムを追加します
		adapter.add("red");
		adapter.add("green");
		adapter.add("blue");
		ListView listView = (ListView) view.findViewById(R.id.listView);
		// アダプターを設定します
		listView.setAdapter(adapter);
		// リストビューのアイテムがクリックされた時に呼び出されるコールバックリスナーを登録します
		listView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
			@Override
			public void onItemClick(AdapterView<?> parent, View view,
					int position, long id) {
				ListView listView = (ListView) parent;
				// クリックされたアイテムを取得します
				String item = (String) listView.getItemAtPosition(position);
				Toast.makeText(view.getContext(), item, Toast.LENGTH_LONG)
						.show();
			}
		});
		// リストビューのアイテムが選択された時に呼び出されるコールバックリスナーを登録します
		listView.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
			@Override
			public void onItemSelected(AdapterView<?> parent, View view,
					int position, long id) {
				ListView listView = (ListView) parent;
				// 選択されたアイテムを取得します
				String item = (String) listView.getSelectedItem();
				Toast.makeText(view.getContext(), item, Toast.LENGTH_LONG)
						.show();
			}

			@Override
			public void onNothingSelected(AdapterView<?> parent) {
			}
		});
	}
}

Fragment1、Fragment2がないので、MainActivityの中のaddTabは上記で作成したListViewFragmentだけにします。

actionBar.addTab(actionBar
    .newTab()
    .setIcon(getResources().getDrawable(R.drawable.ic_launcher))
    .setTabListener(
        new MainTabListener<ListViewFragment>(this, "f3",ListViewFragment.class)));

いかがでしょうか？
これでも動かなかったら修正個所の記述が漏れているかもしれないので教えてくださいませ。

Comments

[退会済みユーザー]

言われた通りにソースを変更したら無事動きました！
ありがとうございます
変更前と見比べて自分でも理解していけるよう頑張ります

Answer 2

Fragment1.java

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

  ListView listView = (ListView) view.findViewById(R.id.listview);

  // TODO ここにListViewActivity#onCreate() 内の処理を移動

  return view;
}