android 中 imageview 设置可见性的问题

执行了 adapter.notifyOnDataChange 了吗？

正常的做法是在 adapter 里面设置一个辅助数组保存 imageview 的状态，在 getview 方法中根据这个数组设定 imageview 的可见性。当当按下按钮后，改变状态数组的内容然后 notifydatachange 。你遍历每个 view 没用的，因为当 view 看不见时会被回收。

首先你这么写代码的习惯很不好 耦合性太高 万一布局一变 imageview 不是在第 0 个了 代码就失效了 应该像楼上说的这么写

其次 千万不要手动调用 getView 方法
如果你 getView 方法里根据 xml 渲染了一个 listView 里的一行的话 你再次调用还会再渲染一次 而且还会触发 listView 的 view 循环利用机制 非常浪费

难道是因为 textview android:layout_width="match_parent"覆盖了？
不过不管是什么原因，你这写法都很不常规

cityListView.getAdapter ().getView (i, null, null ) 改成 cityListView.getChildAt (i )

一看就是新手，问问题的问法以及代码一看就像是来自百度知道的那种，拉低了 v 站的逼格～

没仔细看题。。 都有 view 重用，正确做法是 adapter 里面一个 list 记录每个 item 的状态是否 visible ， getView 里面根据对应 index 的状态设置 visibility ，然后点击事件里面修改这个状态记录再 notifyDataSetChange

你这样写 ListView 还有什么意义，建议把 ListView 原理好好看几遍。另外，请尽量转用 RecyclerView 。

@JayFang1993 按您的说法，新手应该去哪呢？

@ssynhtn 我是新手，请原谅，这样写为什么不常规呢

@allan1st 抱歉，我是新手，我还不太懂代码的优化之类的，能不能详细说点

@ljbha007 多谢指导，我会好好看的，谢谢您

@xhuuanniqege 回答的很好啊也很耐心

@creatorYC 我不知道你是哪里学的，反正随便搜索 listview 的用法都不是这样的。
因为 listview 里面的子 view 对象是会被回收重复利用的，所以就算这个写法一开始能正常工作，只要滑动几下，一个 view 滑出屏幕就会被回收，用到别的 position 去。
入门推荐 udacity 的免费课程

@xhuuanniqege 我刚学没多久，请问您有没有什么好的学习建议，或者能不能推荐一些书以及博客之类的，不胜感激

看了下代码，说下个人看法。

从 layout 上看，应该是一个 RelativeLayout ，在 adapter 里面应该是左边有一个 TextView, 右边有一个 ImageView ，左右两边分别有 20sp 的 padding 。我的看法其实和 @ssynhtn 一样，看起来是 TextView 覆盖了 ImageView ，因为 TextView 的 android:layout_width 属性是 match_parent ，建议你：
1 改成"wrap_content"，或者；
2 在 TextView 里面加上 android:layout_toLeftOf="@+id/ImageView", 或者；
3 把 TextView 和 ImageView 在 XML 文件里面换个位置，先定义 TextView ，再定义 ImageView 。

我没有在代码里尝试，但是凭经验来说，上述任何一个方法应该都能起码把 ImageView 给显示出来。

下面说说在 setOnClickListener 里面的 code 。一般来说不建议直接调用 getView 这个方法，因为这里面的逻辑是得到一个重绘的 View 。如果放在 for 循环从 0 到 list.getCount 的话，则相当于重绘这个 ListView 里面所有的 Item ，这是相当耗费资源的。建议你按照 @xhuuanniqege 提供的办法，在 Object 里面增加一个辅助的 flag 来定义这个 ImageView 的 visibility 。之后的 code 则类似于：

// 假设你的 model 是 TextObject.class
List<TextObject> mTextObjects = new ArrayList<TextObject> ();
ListView mListView = (ListView ) findViewById (...);

CustomAdapter<TextObject> mAdapter = new CustomAdapter (this, R.layout.xxx, mTextObjects );

set.setOnClickListener (new OnClickListener () {
@Override
public void onClick (View v ) {
//全选遍历 ListView 的选项，每个选项就相当于布局配置文件中的 RelativeLayout
for (int i = 0; i < cityListView.getCount (); i++){
mTextObjects.get (i ).setImageViewVisibility (true );
}

mAdapter.notifyDatasetChanged ();
}
});

希望对你有帮助。

好吧，反正不是 textview 遮挡的问题，因为 textview 的背景色默认是透明的

@ssynhtn 有道理，这点确实忽略了

@gengrui 我刚接触 android 不久，我应该去详细了解一下 ListView 的机制和原理，您的意思是说我应该设置标记去更新相应的 item 的可见性，而不应该每次都是去重绘 item ，的确我没有考虑到这些，没有考虑到代码的优化，多谢您的指点，我会尝试着改变这个写法，谢谢您详细的解答

@creatorYC StackOverflow 或者 SegmentFault