【问题标题】:ViewPager with Fragments that contain TableLayouts loads slowly:带有包含 TableLayouts 的 Fragments 的 ViewPager 加载缓慢:
【发布时间】:2013-03-21 23:06:13
【问题描述】:

我正在开发一个带有ViewPager 的活动,其中包含片段(每次最多4 个片段),每个片段都包含TableLayout。所以基本上加载了4个表。这是将数据加载到Fragment 的代码,稍后将附加到ViewPager

 public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) 
{
    if (container == null) {
        return null;
    }

    LinearLayout.LayoutParams layoutParams=new LinearLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
    layoutParams.gravity=Gravity.CENTER_VERTICAL;
    layoutParams.setMargins(10, 0, 0, 0);

    fragmetLayout = (LinearLayout)inflater.inflate(R.layout.grid_fragment_layout, container, false);
    table = (TableLayout) fragmetLayout.findViewById(R.id.tlGridTable);
    llParameterContainer =  (LinearLayout) fragmetLayout.findViewById(R.id.llParametersContainer);
    tabName = (TextView) fragmetLayout.findViewById(R.id.tvTabName);
    tabName.setText(tab.getTabName());
    //tabName.setLayoutParams(layoutParams);

    for (Parameter parameter : SGRaportManagerAppObj.getInstance().parametersRepository.getParametersRepository())
    {
        ImageView parameterPoint = new ImageView(getActivity());
        parameterPoint.setImageResource(R.drawable.parameter_chosen_point);
        parameterPoint.setPadding(10, 10, 10, 0);
        parameterPoint.setLayoutParams(layoutParams);
        llParameterContainer.addView(parameterPoint);

        TextView parameterTextView = new TextView(getActivity());
        parameterTextView.setText(parameter.getName()+":");
        parameterTextView.setGravity(Gravity.CENTER);
        parameterTextView.setTextColor(getResources().getColor(R.color.my_black));  
        parameterTextView.setPadding(5, 10, 3, 10);
        parameterTextView.setLayoutParams(layoutParams);
        llParameterContainer.addView(parameterTextView);
        TextView parameterChosenValueTextView = new TextView(getActivity());
        parameterChosenValueTextView.setText(parameter.getChosenValue());
        parameterChosenValueTextView.setGravity(Gravity.CENTER); 
        parameterChosenValueTextView.setPadding(0, 10, 0, 10);
        parameterChosenValueTextView.setLayoutParams(layoutParams);
        parameterChosenValueTextView.setTextColor(getResources().getColor(R.color.my_black));   
        llParameterContainer.addView(parameterChosenValueTextView);

    }
    //table.setStretchAllColumns(true);  
    //table.setShrinkAllColumns(true); 
    TableRow tableRow = new TableRow(getActivity());
    TableLayout.LayoutParams tableRowParams=new TableLayout.LayoutParams(TableLayout.LayoutParams.WRAP_CONTENT,TableLayout.LayoutParams.WRAP_CONTENT);
    tableRowParams.setMargins(2, 0, 0, 0);
    tableRow.setLayoutParams(tableRowParams);
    Log.d(TAG, "TAB FROM FRAGMENT:"+tab.toString());


    TableRow.LayoutParams tlparamsFirstColumn = new TableRow.LayoutParams(TableRow.LayoutParams.WRAP_CONTENT, TableRow.LayoutParams.WRAP_CONTENT); 
    tlparamsFirstColumn.setMargins(4, 0, 2, 0);
    TableRow.LayoutParams tlparams = new TableRow.LayoutParams(TableRow.LayoutParams.WRAP_CONTENT, TableRow.LayoutParams.WRAP_CONTENT); 
    tlparams.setMargins(0, 0, 2, 0);

    //The First row for the column names.
    for (int i= 0; i < tab.getGrid().getGridColumnsList().size(); i++)
    {
        TextView tvName = new TextView(getActivity());
        String columnName = tab.getGrid().getGridColumnsList().get(i).getGridColumnAlias();
        tvName.setText(columnName);
        Log.d(TAG, "COLUMN ALIAS FROM FRAGMENT: "+columnName);
        if (i == 0)
        {
            tvName.setLayoutParams(tlparamsFirstColumn);
        }
        else
        {
            tvName.setLayoutParams(tlparams);
        }
        tvName.setGravity(Gravity.CENTER);
        tvName.setTextColor(getResources().getColor(R.color.my_white));  
        tvName.setPadding(10, 10, 10, 10);
        tvName.setBackgroundDrawable(getResources().getDrawable(R.drawable.grid_column_name_background));   
        tableRow.addView(tvName);
    }
    table.addView(tableRow);

    for (int j = 0; j < tab.getGrid().getGridData().getNumRows(); j++)
    {
        TableRow newRow = new TableRow(getActivity());
        TableLayout.LayoutParams insideRowParams=new TableLayout.LayoutParams(TableLayout.LayoutParams.WRAP_CONTENT,TableLayout.LayoutParams.WRAP_CONTENT);
        tableRowParams.setMargins(0, 2, 0, 0);
        for (int k = 0; k < tab.getGrid().getGridData().getNumCols(j); k++)
        {
            TextView tvName = new TextView(getActivity());
            String columnName = tab.getGrid().getGridData().get(j, k);
            tvName.setText(columnName);

            if ( (j % 2) == 0)
            {
                tvName.setBackgroundDrawable(getResources().getDrawable(R.drawable.grid_first_row_background)); 
            }
            else
            {
                tvName.setBackgroundDrawable(getResources().getDrawable(R.drawable.grid_second_row_background));    
            }
            if (k == 0)
            {
                tvName.setLayoutParams(tlparamsFirstColumn);
            }
            else
            {
                tvName.setLayoutParams(tlparams);
            }
            tvName.setGravity(Gravity.CENTER);
            tvName.setTextColor(getResources().getColor(R.color.my_black));  
            tvName.setPadding(10, 10, 10, 10);
            newRow.addView(tvName);  
        }
        table.addView(newRow);          
    }
    return fragmetLayout;
}

更新:

我的 ViewPagerActivity 布局:

<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent" >

<TabHost
    android:id="@android:id/tabhost"
    android:layout_width="fill_parent"
    android:layout_height="fill_parent"
    android:layout_below="@+id/imageView3" >

    <LinearLayout
        android:orientation="vertical"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent" >
        <FrameLayout
            android:id="@android:id/tabcontent"
            android:layout_width="0dp"
            android:layout_height="0dp"
            android:layout_weight="0"/>

        <android.support.v4.view.ViewPager
            android:id="@+id/pager"
            android:layout_width="fill_parent"
            android:layout_height="0dp"
            android:layout_weight="1" >
        </android.support.v4.view.ViewPager>

         <TabWidget
            android:id="@android:id/tabs"
            android:orientation="horizontal"
            android:layout_gravity="center_horizontal"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content" />
    </LinearLayout>
</TabHost>

 <ImageView
    android:id="@+id/imageView3"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:layout_marginTop="-3dp"
    android:background="@drawable/login_scr_top_bar"
    android:contentDescription="@drawable/login_scr_top_bar" />

<TextView
    android:id="@+id/tvReportName"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_gravity="top|left"
    android:paddingLeft="15dp"
    android:paddingTop="13dp"
    android:text="@string/report_tabs"
    android:textColor="@color/my_black"
    android:textSize="18sp"
    android:textStyle="bold" />

<LinearLayout
    android:id="@+id/llTabsButtonsContainer"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_alignParentBottom="true"
    android:layout_alignParentLeft="true"
    android:layout_alignParentRight="true"
    android:gravity="center_horizontal"
    android:orientation="horizontal"
    android:paddingBottom="5dp"
    android:paddingTop="5dp" > 
</LinearLayout>

<TextView
    android:id="@+id/tvReportTitle"
    android:layout_width="wrap_content"
    android:layout_height="20dp"
    android:layout_alignBaseline="@+id/tvReportName"
    android:layout_alignBottom="@+id/tvReportName"
    android:layout_toLeftOf="@+id/bBackToParameters"
    android:layout_toRightOf="@+id/tvReportName"
    android:text="TextView"
    android:textSize="18sp" />

<Button
    android:id="@+id/bBackToParameters"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_alignBottom="@+id/tvReportTitle"
    android:layout_marginRight="10dp"
    android:layout_alignParentRight="true"
    android:background="@drawable/button_back_to_parameters_selector"
    android:clickable="true"
    android:onClick="backToParametersButtonOnClick"
    android:padding="3dp"
    android:text="@string/back_to_parameters"
    android:textColor="@color/my_white"
    android:textStyle="bold" />

</RelativeLayout>

和我的 GridFramgnet 布局:

<?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" >

<LinearLayout
android:id="@+id/llParametersContainer"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:orientation="horizontal" >
</LinearLayout>

<TextView 
    android:id="@+id/tvTabName"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content" 
    android:textColor="@color/my_black"
    android:textStyle="bold"
    android:textSize="18sp"
    android:layout_marginLeft="10dp"
    android:text="It a test string for report Name"/>

    <ScrollView 
    android:id="@+id/layout" 
    android:layout_height="match_parent"         
    android:scrollbars="horizontal|vertical" 
    android:layout_width="match_parent"     
    android:layout_marginTop="5dip"     
    android:scrollbarStyle="outsideInset"
    android:fillViewport="true"> 

    <HorizontalScrollView 
        android:id="@+id/horizontalView" 
        android:layout_height="wrap_content"     
        android:scrollbars="horizontal|vertical" 
        android:layout_width="wrap_content"     
        android:layout_marginTop="5dip">

        <TableLayout
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:id="@+id/tlGridTable" >   
        </TableLayout>
    </HorizontalScrollView>
   </ScrollView>
 </LinearLayout>

问题 这对于 3 Fragments 大约需要 10 秒才能加载,因此用户在实际加载 Activity 之前会黑屏 10 秒。我怎样才能避免它?运行在AsyncTask 中填充表的所有过程将使我的ViewPager Activity 加载更快?还能做到吗?

更新: 大部分时间都花在填充/创建TableLayout 上。如果从服务器收到大量数据,则需要创建大量TextViews,并将它们设置在TableLayout中。

更新 #2: 好吧,我已经解决了这个问题,因为我理解我的需求(整个视图的水平和垂直滚动)我必须使用TableLayout,所以我需要等待这个时间来加载它。但是我需要找到一种方法来加载片段中的数据,至少让用户知道正在填充数据(显示适当的Dialog)。在开始初始化片段之前,我尝试在ViewPageronCreate 中放置一个对话框,但由于某种原因它没有显示,并且在加载片段之前我只看到黑屏。

更新:

请参考最后一个问题,因为我知道无法修复加载时间:

问题: 有没有办法创建一个带有几乎空片段的ViewPager(每个片段只有一个单独的标题),将它们呈现给用户,然后在活动已经加载并可见(而不是显示黑屏)之后,显示Dialog 并在显示对话框时填充片段?

任何帮助将不胜感激。

谢谢。

【问题讨论】:

  • Android TableLayouts 是出了名的慢。是否可以用列表视图和自定义适配器替换您的表格?随时发布您正在尝试完成的屏幕截图,我可以帮助您为其构建自定义适配器。
  • 不,不幸的是我不这么认为。我需要将数据加载到可以垂直和水平滚动的表中(据我所知,ListViews 无法实现)
  • 你是在连接调试器的情况下测量这个吗?
  • 什么意思?我在 Galaxy s3 上测量这个。没有检查这一刻的堆栈跟踪的输出。我在那里能看到什么?不会崩溃,只是加载活动需要很长时间
  • 您是否尝试过使用setOffscreenPageLimit?我不会将此作为长期解决方案,但现在,尝试使用零作为参数调用它。

标签: android android-fragments android-viewpager android-fragmentactivity


【解决方案1】:

这对于 3 个 Fragments 大约需要 10 秒才能加载,所以用户得到 在实际加载 Activity 之前黑屏 10 秒。

原因是您在布局中使用的小部件数量加上 Activity 的视图层次结构的深度。从你发布的内容来看,你有这样的东西:

ViewPagerActivity 中的可能包装器 -> ViewPager -> 至少一个ScrollView(正如你所说的关于滚动 垂直和水平) -> Linearlayout -> TableLayout -> TableRow -> TableRow的内容

这是 7 的深度(如果我没记错的话,支持片段框架还会在其间插入一些视图)。这对性能非常不利。 在单个 Fragment 中还有很多小部件(假设那些 for 循环最多运行 10 次迭代,否则真的很糟糕),并且考虑到第二个片段也将加载它的视图,你可以看到这将再次成为性能打击。

发布您当前的布局以查看它们是否无法优化会很有帮助。

您可以对当前代码进行一些小的改进,但这不会帮助您从 10 秒缩短到最多 1 秒:

这段代码没有意义:

if (container == null) {
    return null;
}

我不知道这是做什么的:

SGRaportManagerAppObj.getInstance().parametersRepository.getParametersRepository()

但是看看你是否不创建无用的对象(那是单例吗?)。不要把它放在for循环中,因为它每次都会运行,使用普通的for来避免使用Iterator

final int count = SGRaportManagerAppObj.getInstance().parametersRepository.getParametersRepository().size();
for (int i = 0; i < count; i++) {
     final Parameter parameter = SGRaportManagerAppObj.getInstance().parametersRepository.getParametersRepository().get(i);
     // ... rest of code

您应该在其他 for 循环中执行相同的操作。

for 循环之外检索这些资源(颜色和可绘制对象),在循环中仅将它们分配给视图。

编辑: 您的布局非常深并且视图太多,因此性能会很差。克服这个问题的唯一方法是使用一个回收其视图的小部件(我知道你说了什么)。替代方案是:

  • 在后台线程上创建这些视图(我不会这样做,因为风险很大)
  • 您可以修改片段以简单地扩大onCreateView 中的布局并返回并显示正在加载的ProgressDialog。在Fragment 变得可见后,您将开始以块的形式创建Fragment 的真实视图,最后将其附加到片段上。这对用户体验非常不利。

当您使用ViewPager 时,将需要做额外的工作来让页面切换之间的事情顺利进行。也许你可以重新考虑一下你目前的设置。

【讨论】:

  • 我已经用 ViewPager 和 GridFragment 布局更新了这个问题。您对 TableLayout 中的视图数量是正确的(基本上很多 TextViews 通常比 10 次迭代要多得多)。但是我没有任何其他选择我必须提供双向滚动功能(垂直和水平),所以我不能在这个屏幕上使用列表视图。 TableLayout 是我发现显示此类数据的唯一方法。并且它显示得很好,只是加载很慢。
  • 你对 for 条件也是正确的,我将把它移到 for 循环之外,看看它产生的影响。但正如你已经说过的,我认为这不会有很大的不同。
  • @EmilAdz 那么您没有太多选择,请参阅我编辑的答案。您正在做的事情太多了(我已经看到您在 S3 上进行了测试,这是一款功能强大的设备,在较低规格的设备上您的代码很可能会失败)
  • 感谢@Luksprog 为您编辑的答案,现在如果您查看我问的问题:“运行在 AsyncTask 中填充表的所有过程将使我的 ViewPager Activity 加载更快?甚至可以吗?完成?”你现在说:在后台线程上创建这些视图(我不会这样做,因为它非常危险)。为什么你认为这是一个冒险的行为?我真的不明白你说的第二个介词是什么意思。对于我的问题,我现在看到的唯一解决方案是在生成片段时在 ViewPager 中显示一个 ProgressDialog 框。
  • @EmilAdz 这些视图可以使用绑定到创建它们的胎面的处理程序,这将导致问题。第二个选项(在此您指的是对吗?)您将在片段类中扩展最小布局(当前 xml 布局),并在以后在进度对话框/栏的保护下添加其余部分。基本上就是你在评论最后一部分所说的。
【解决方案2】:

由于您使用分页器,因此在启动 Activity 时加载的内容比第一个片段更多。 使带有进度条的初始片段视图可见并隐藏表格,并仅在您切换到该片段时加载数据。

在活动中使用

mPager.setOnPageChangeListener(new OnPageChangeListener() {

            @Override
            public void onPageSelected(int position) {
                Fragment f = mAdapter.getFragment(mPager.getCurrentItem());
                f.loadData(); // when you finish loading the data, hide the progress bar and show the table

            }

            @Override
            public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {

            }

            @Override
            public void onPageScrollStateChanged(int state) {
            }
        });

要使用 mAdapter.getFragment(),请从以下位置继承您的适配器:

    public abstract class SmartFragmentPagerAdapter extends FragmentPagerAdapter{

    private SparseArray<String> mFragmentTags;
    private FragmentManager mFragmentManager;

    public SmartFragmentPagerAdapter(FragmentManager fragmentManager) {
        super(fragmentManager);
        mFragmentManager = fragmentManager;
        mFragmentTags = new SparseArray<String>();
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Object obj = super.instantiateItem(container, position);
        if (obj instanceof Fragment) {
            // record the fragment tag here.
            Fragment f = (Fragment) obj;
            String tag = f.getTag();
            mFragmentTags.put(position, tag);
        }
        return obj;
    }

    public Fragment getFragment(int position) {
        String tag = mFragmentTags.get(position);
        if (tag == null)
            return null;
        return mFragmentManager.findFragmentByTag(tag);
    }

    public FragmentManager getFragmentManager(){
        return mFragmentManager;
    }

}

【讨论】:

    【解决方案3】:

    大部分时间都花在填充/创建 TableLayout 上。如果从服务器接收到大量数据,则需要创建大量 TextView 并将它们设置在 TableLayout 中

    那么不要使用TableLayout。使用ListView 或其他形式的AdapterView,这样您只需要“填充/创建”足以填充可见空间,然后可以在用户滚动时填充更多(如果用户滚动的话)。

    【讨论】:

    • 不幸的是我不能。我需要将数据加载到可以垂直和水平滚动的表中(据我所知,ListViews 或任何其他 AdapterView 都无法实现,或者我错了吗?)您可以了解我正在尝试开发的内容也在调查这个问题:stackoverflow.com/questions/15382753/…
    • @EmilAdz:“不幸的是我不能”——你表现得好像你有选择。如果创建TableLayout 需要10 秒,那么TableLayout 必须有数十万个单元格,您将耗尽堆空间。
    • 我有必须遵循的设计线...这些告诉我网格应该可以双向滚动(垂直和水平),但这不能使用适配器进行正确的?除此之外,我愿意接受任何建议。如果需要,我希望在 ViewPager 中加载所有片段时显示 10 秒的对话框。但由于某种原因,我无法让它工作。
    • @EmilAdz:“但这不能使用适配器来实现,对吧?” -- 您必须为此创建自己的AdapterView,或者查看其他人是否已经拥有。 “我希望显示一个 10 秒的对话框”——有才华的程序员希望完全避免对话框,让用户可以立即使用。
    • 如果我错了,请纠正我,但如果我创建了一个自定义布局(目前我不知道它是如何完成的,并且由于时间限制,当然更喜欢 Dialog 选项,但如果我这样做的话。 .) 这意味着每一行都将是水平滚动的,而不是整个“水平滚动的列表视图”,这是我现在使用 TableLayout 得到的期望行为,对吧?
    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2018-07-15
    • 2023-03-16
    • 1970-01-01
    相关资源
    最近更新 更多