Tangram-iOS
Tangram-iOS copied to clipboard
alibaba
TangramPageScrollLayout中的element重复添加问题
卡片内的组件数据源如果有变化，我通过 [TangramDefaultDataSourceHelper modelsWithLayoutDictionary:newLayoutDict] 生成了新的itemsModel，然后 [pageScrollLayoutView setItemModels:itemsModel] 后，执行 [TangramView reloadlayout:pageScrollLayoutView]，pageScrollLayoutView 中原来的element不会被删除，也就是说会一直添加新的element，而且如果新的itemsModel.count = 0 的话，在 TangramPageScrollLayout 的方法 calculateLayout 中直接执行了
if (nil == self.itemModels || 0 >= modelCount) {
self.height = 0.f;
return;
}
把这个layoutView的高度赋成了0，但是以前的子试图还是存在的，导致以前的子试图还是会显示的。 所以这里两个现象： 1.在没有重新生成layoutView的情况下，局部更新该layoutView的组件数据源，刷新后，老得组件没有被删除，导致重复添加组件。 2.在没有重新生成layoutView的情况下，局部更新该layoutView的组件数据源且新的组件个数为0，这个layoutView高度为0，但是老得组件还是会显示出来。 这两个现象都是同一个问题造成的，就是更新itemsModel后，应该删除老的组件视图。
麻烦你这边验证一下。
和之前的研发沟通了一下，目前此 Layout 基本属于废弃状态。
最早的时候这个 Layout 是用来实现 Banner 的，所以要维持内部元素永驻内存（为了维持它滚出页面时的滚动 offset 状态），避免被重用导致各种问题。所以 pageScrollLayout 在添加元素时强制将内部元素的 reuseIdentifer 全部清空了，并且从来没有测试过 reloadLayout 之类的可能。
目前来看逻辑已经难以维护，有点接近上帝才知道的状态了……所以综合来说现在的建议：
-
如果是做 banner 可以考虑实现一个带滚动的 element（组件），配合 container-oneColumn 使用。（我们自己目前是这么做的）
-
对这个 Layout 整体取消重用方案，每次 reload 之前重新生成一次所有的 PageLayout。（这个方案类似临时补丁的效果）
-
重新实现一个合适的 PageLayout，这个目前对于我们来说优先级较低，如果你乐意的话，可以参与到开源建设中来。（不排除很久以后我会重写它的可能）
可以尝试修复方案，注释掉我所注释的对应行：
- (void)addSubview:(UIView *)view
{
if (view && view.reuseIdentifier) {
// view.reuseIdentifier = @"";
[self.scrollView addSubview:view];
注：并未经过充分测试
收到，谢谢。 目前天猫首页里面的Banner都是自己实现的element+oneColumn形式的？我感觉这种布局是非常常见有用的，还是需要提供这么个布局的。希望可以尽快完善哦～
是的，element + oneColumn。
主要是这种 Layout 在需求上存在设计难点（而不是实现难点）：
- banner 如果重用，可能存在多次上报曝光问题。所以 banner 控件最好由用户自行控制。
- 如果每次滚动到 PageLayout 处时它都回归原始状态，对 pageIndex != 0 的元素曝光量很不友好。但如果 Layout 本身需要维持 pageIndex 信息，违背了只进行“布局”的初衷。
比较难取舍。
