TangramPageScrollLayout中的element重复添加问题 about tangram-ios HOT 9 OPEN

收到，会进行验证

from tangram-ios.

补充一下，是在一个页面中有两个及两个以上的轮播布局卡片，才会出现这种情况，如果只有一个轮播布局的卡片是没问题的。

from tangram-ios.

已重现此问题，在排查中

from tangram-ios.

和之前的研发沟通了一下，目前此 Layout 基本属于废弃状态。

最早的时候这个 Layout 是用来实现 Banner 的，所以要维持内部元素永驻内存（为了维持它滚出页面时的滚动 offset 状态），避免被重用导致各种问题。所以 pageScrollLayout 在添加元素时强制将内部元素的 reuseIdentifer 全部清空了，并且从来没有测试过 reloadLayout 之类的可能。

目前来看逻辑已经难以维护，有点接近上帝才知道的状态了……所以综合来说现在的建议：

1. 如果是做 banner 可以考虑实现一个带滚动的 element（组件），配合 container-oneColumn 使用。（我们自己目前是这么做的）

2. 对这个 Layout 整体取消重用方案，每次 reload 之前重新生成一次所有的 PageLayout。（这个方案类似临时补丁的效果）

3. 重新实现一个合适的 PageLayout，这个目前对于我们来说优先级较低，如果你乐意的话，可以参与到开源建设中来。（不排除很久以后我会重写它的可能）

from tangram-ios.

可以尝试修复方案，注释掉我所注释的对应行：

- (void)addSubview:(UIView *)view
{
    if (view && view.reuseIdentifier) {
//        view.reuseIdentifier = @"";
        [self.scrollView addSubview:view];

注：并未经过充分测试

from tangram-ios.

收到，谢谢。
目前天猫首页里面的Banner都是自己实现的element+oneColumn形式的？我感觉这种布局是非常常见有用的，还是需要提供这么个布局的。希望可以尽快完善哦～

from tangram-ios.

是的，element + oneColumn。

主要是这种 Layout 在需求上存在设计难点（而不是实现难点）：

1. banner 如果重用，可能存在多次上报曝光问题。所以 banner 控件最好由用户自行控制。
2. 如果每次滚动到 PageLayout 处时它都回归原始状态，对 pageIndex != 0 的元素曝光量很不友好。但如果 Layout 本身需要维持 pageIndex 信息，违背了只进行“布局”的初衷。

比较难取舍。

from tangram-ios.

自己实现的element+oneColumn形式 ，现在一个页面有几个滚动布局的话，会出现数据混乱的现象，不知道是什么原因？

from tangram-ios.

@yoowei oneColumn应该不会出现类似情况，你可以另开一个问题说得更清楚点

from tangram-ios.