Question

I’m trying to build and app that shows organized data so I’m thinking that basically a TableLayout would be a good idea, just to keep them in rows, so I’m wondering whats the correct way to do it? I have this in my 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" >

    <TableLayout
        android:id="@+id/tableLayout1"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent">
        <TableRow>
            <TextView
                android:layout_width="fill_parent"
                android:layout_height="fill_parent"
                android:text="@string/hello" >
             </TextView>

            <ListView
                android:id="@+id/listView1"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:layout_marginLeft="233dp"
                android:layout_marginTop="44dp" >
            </ListView>
        </TableRow>
    </TableLayout>

</LinearLayout>

then it’s just that the TableLayout tag and TableRow tag displays a warning that says:

"This TableLayout layout or its LinearLayout parent is possibly useless"

so what I understand is that is not picking up the TableLayout. how can I fix this? is there another way to do this easly?

Answer 1

so what i understand is that is not picking up the tablelayout. how can i fix this? is there another way to do this easly?

Not quite, the renderer is taking account of the TableLayout but because the lint tool has detected that you have two views when you only need one (A LinearLayout and TableLayout) it is warning you that you should remove one of these layouts as it will boost performance while not effecting the functionality of the page.

I will add, you have a TableLayout with just a single row. TableLayout’s are designed to allow their contents to format their columns based upon all of their rows. With a single Row the TableLayout & Row together are acting as a LinearLayout:

<LinearLayout>
    <LinearLayout>
        <ListView>
        <TextView>

Since your TableLayout isn’t adding any addition layout in that case it becomes obsolete and you will gain a resource boost from removing it and changing the orientation of LinearLayout to horizontal.

If you’re planning to add more TableRow‘s later, then you should instead remove the LinearLayout for the same reason – it will then become the view which is not adding extra information: your table layout will be entirely responsible for the layout so you might as well make it the parent of the layout file (remember to add xmlns:android="http://schemas.android.com/apk/res/android" attribute to it if you do.)