Android customlayout children not gettting measured height and width

Question

I have implemented and activity (called HomeScreenActivity) that consists of two fragments. One of them displays a list of items, and the other displays the details of the selected item. The details fragment (ResumeFragment in my code) consists of a series of components, which can be configured in height and width by the user by specifying it as a number of cells in a configurations file. These components are added to a custom layout called ComponentsLayout.

When opening the HomeScreenActivity i display each fragment in the onCreate method, and they are shown as intended. But when i click and item in the items list fragment, and i replace the resumeFragment with a new instance of the same class, nothing is displayed.

I have spend a lot of time debugging and i see that the resumeFragment, does fill out the height and width it is given, but the componentsLayout only has a height of 1. This i have found out is because the height and width of the components contained in the componentsLayout are not getting set, even though i set their layoutParams in the componentsLayout onMeasure method.

Below is the code of the HomeScreenActivity, the ResumeFragment and the ComponentsLayout:

public class HomeScreenActivity extends FragmentActivity implements OnItemSelectedListener {
.
.

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    setContentView(R.layout.home_screen);

    .
    .

    //Check if device is in landscape orientation 
    if(width > height && isTablet) {
        //The patient context fragment must maintain the same width as when in portrait orientation.
        findViewById(R.id.patient_resume_fragment_container).setLayoutParams(new LinearLayout.LayoutParams(height, height));
        findViewById(R.id.screen_pager).setLayoutParams(new LinearLayout.LayoutParams(width-height, height));
        fragmentManager = getSupportFragmentManager();
        patientResumeFragment = new ResumeFragment();
        testFragment = new NysomTestFragment();
        fragmentTransaction = fragmentManager.beginTransaction();
        fragmentTransaction.replace(R.id.patient_resume_fragment_container, patientResumeFragment);
        fragmentTransaction.commit();
    }   
}

.
.

@Override
public void onItemSelected(Item item) {
    itemResumeFragment.getView().findViewById(R.id.componentsContainer);
    itemResumeFragment = new ResumeFragment(item);
    fragmentTransaction = fragmentManager.beginTransaction();
    fragmentTransaction.replace(R.id.item_resume_fragment_container, itemResumeFragment);
    fragmentTransaction.commit();
}

The ResumeFragment

    @Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
        Bundle savedInstanceState) {
    View fragmentView = inflater.inflate(R.layout.resume_fragment,container, false);
    ComponentsLayout componentsContainer = (ComponentsLayout) fragmentView.findViewById(R.id.componentsContainer);
    placeComponents(componentsContainer);
    return fragmentView;
}

private void placeComponents(ComponentsLayout layout) {
    List<ComponentConfig> configs = new ArrayList<ComponentConfig>(testMobile.getModel().getComponentConfiguration());

    int viewId = 1;

    for (ComponentConfig currConfig : configs) {
        ComponentsLayout.LayoutParams params = new ComponentsLayout.LayoutParams(0, 0);
        params.setWidthInCells(currConfig.getWidth());
        params.setHeightInCells(currConfig.getHeight());

        if (viewId == 1) {
            params.addRule(RelativeLayout.ALIGN_PARENT_TOP);
        } else {
            if (currConfig.getPositionX() > 0) {
                // looking for rightOf viewId
                params.addRule(RelativeLayout.RIGHT_OF, getRightOfComponentId(configs, currConfig, viewId - 1));
            }
            if (currConfig.getPositionY() > 0) {
                // looking for below viewId
                params.addRule(RelativeLayout.BELOW, getBelowComponentId(configs, currConfig, viewId - 1));
            }
        }

        try {
            // this code will be changed in story 9
            View view = currConfig.getClassType().newInstance().getView(getActivity());
            view.setId(viewId);
            view.setBackgroundColor(colors[(viewId - 1) % 6]);
            layout.addView(view, params);
        } catch (Exception e) {
            // we should never get to this point as configuration is validated
            Log.e(TAG, "getView: unable to create component for position " + viewId, e);
            throw new RuntimeException(e);
        }

        viewId++;
    }
}

And the ComponentLayout

public class ComponentsLayout extends RelativeLayout {

public ComponentsLayout(Context context, AttributeSet attrs) {
    super(context, attrs);
}

@Override
protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
    super.onMeasure(widthMeasureSpec, heightMeasureSpec);

    int parentWidth = MeasureSpec.getSize(widthMeasureSpec);
    int horizontalCellsNum = getResources().getInteger(R.integer.resume_horizontal_cells_num);
    int cellHeight = (int) (getResources().getDimension(R.dimen.resume_cell_height));

    int count = getChildCount();
    for (int i = 0; i < count; i++) {
        final View child = getChildAt(i);
        LayoutParams params = (LayoutParams) child.getLayoutParams();
        params.width = parentWidth * params.getWidthInCells() / horizontalCellsNum;
        params.height = cellHeight * params.getHeightInCells();
        child.setLayoutParams(params);
    }
}

And once again, this works when called from the HomeScreenActivity onCreate method, but not when done from the onItemSelected method.

Can anyone help??

Answer 1

I have created a work around by adding all layout params to the components before onMeasure is called on the componentLayout. It was not the solution i was looking for, but it was quite simple, which is possibly why i overlooked it.