对搭档代码的修改意见

源代码

  1 <?xml version="1.0" encoding="utf-8"?>
  2 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
  3     android:paddingBottom="@dimen/activity_vertical_margin"
  4     android:paddingLeft="@dimen/activity_horizontal_margin"
  5     android:paddingRight="@dimen/activity_horizontal_margin"
  6     android:paddingTop="@dimen/activity_vertical_margin"
  7     android:layout_width="match_parent"
  8     android:layout_height="match_parent"
  9     android:orientation="horizontal">
 10
 11     <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
 12         xmlns:tools="http://schemas.android.com/tools"
 13         android:layout_width="match_parent"
 14         android:layout_height="match_parent"
 15         android:background="#E1E0DE"
 16         tools:context=".MainActivity">
 17
 18
 19
 20             <RelativeLayout
 21                 android:id="@+id/relativeLayout2"
 22                 android:layout_width="fill_parent"
 23                 android:layout_height="80dp"
 24                 android:layout_alignParentLeft="true"
 25                 android:layout_marginTop="20dp"
 26                 android:background="@color/white"
 27                 android:clickable="true">
 28
 29                 <ImageView
 30                     android:id="@+id/imageView1"
 31                     android:layout_width="wrap_content"
 32                     android:layout_height="wrap_content"
 33                     android:layout_alignParentLeft="true"
 34                     android:layout_centerVertical="true"
 35                     android:src="@android:drawable/ic_delete" />
 36
 37                 <TextView
 38                     android:id="@+id/text_pyq"
 39                     android:layout_width="wrap_content"
 40                     android:layout_height="wrap_content"
 41                     android:layout_centerVertical="true"
 42                     android:layout_marginLeft="20dp"
 43                     android:layout_toRightOf="@+id/imageView1"
 44                     android:text="我的id"
 45                     android:textColor="#000" />
 46
 47                 <TextView
 48                     android:id="@+id/text_pyq1"
 49                     android:layout_width="wrap_content"
 50                     android:layout_height="wrap_content"
 51                     android:layout_below="@+id/text_pyq"
 52                     android:layout_marginLeft="20dp"
 53                     android:layout_toEndOf="@+id/imageView1"
 54                     android:text="微信号"
 55                     android:textColor="#000" />
 56
 57                 <ImageView
 58                     android:id="@+id/imageView2"
 59                     android:layout_width="wrap_content"
 60                     android:layout_height="wrap_content"
 61                     android:layout_alignParentRight="true"
 62                     android:layout_centerVertical="true"
 63                     android:src="@android:drawable/ic_delete" />
 64
 65             </RelativeLayout>
 66
 67
 68
 69
 70         <RelativeLayout
 71             android:id="@+id/relativeLayout3"
 72             android:layout_width="fill_parent"
 73             android:layout_height="100dp"
 74             android:layout_alignParentLeft="true"
 75             android:layout_below="@+id/relativeLayout2"
 76             android:layout_marginTop="20dp"
 77             android:background="@color/white">
 78
 79             <RelativeLayout
 80                 android:id="@+id/rel_sys"
 81                 android:layout_width="match_parent"
 82                 android:layout_height="50dp"
 83                 android:layout_alignParentLeft="true"
 84                 android:layout_alignParentTop="true"
 85
 86                 android:clickable="true">
 87
 88                 <ImageView
 89                     android:id="@+id/imageView3"
 90                     android:layout_width="wrap_content"
 91                     android:layout_height="wrap_content"
 92                     android:layout_alignParentLeft="true"
 93                     android:layout_centerVertical="true"
 94                     android:src="@android:drawable/ic_delete" />
 95
 96                 <TextView
 97                     android:id="@+id/text_sys"
 98                     android:layout_width="wrap_content"
 99                     android:layout_height="wrap_content"
100                     android:layout_centerVertical="true"
101                     android:layout_marginLeft="20dp"
102                     android:layout_toRightOf="@+id/imageView3"
103                     android:text="相册"
104                     android:textColor="#000" />
105
106
107             </RelativeLayout>
108
109             <View
110                 android:layout_width="match_parent"
111                 android:layout_height="0.5dp"
112                 android:layout_alignBottom="@+id/rel_sys"
113                 android:background="#E1E0DE">
114
115             </View>
116
117             <RelativeLayout
118                 android:layout_width="match_parent"
119                 android:layout_height="50dp"
120                 android:layout_alignParentBottom="true"
121                 android:layout_marginRight="-10.5dp"
122
123                 android:clickable="true">
124
125                 <ImageView
126                     android:id="@+id/imageView5"
127                     android:layout_width="wrap_content"
128                     android:layout_height="wrap_content"
129                     android:layout_alignParentLeft="true"
130                     android:layout_centerVertical="true"
131                     android:src="@android:drawable/ic_delete" />
132
133                 <TextView
134                     android:id="@+id/text_yyy"
135                     android:layout_width="wrap_content"
136                     android:layout_height="wrap_content"
137                     android:layout_centerVertical="true"
138                     android:layout_marginLeft="20dp"
139                     android:layout_toRightOf="@+id/imageView5"
140                     android:text="收藏"
141                     android:textColor="#000" />
142
143
144             </RelativeLayout>
145         </RelativeLayout>
146
147         <RelativeLayout
148             android:id="@+id/relativeLayout4"
149             android:layout_width="match_parent"
150             android:layout_height="100dp"
151             android:layout_alignParentLeft="true"
152             android:layout_below="@+id/relativeLayout3"
153             android:layout_marginTop="20dp"
154             android:background="@color/white">
155
156             <RelativeLayout
157                 android:id="@+id/rel_fjdr"
158                 android:layout_width="match_parent"
159                 android:layout_height="50dp"
160                 android:layout_alignParentLeft="true"
161                 android:layout_alignParentTop="true"
162
163                 android:clickable="true">
164
165                 <ImageView
166                     android:id="@+id/imageView7"
167                     android:layout_width="wrap_content"
168                     android:layout_height="wrap_content"
169                     android:layout_alignParentLeft="true"
170                     android:layout_centerVertical="true"
171                     android:src="@android:drawable/ic_delete" />
172
173                 <TextView
174                     android:id="@+id/text_fjdr"
175                     android:layout_width="wrap_content"
176                     android:layout_height="wrap_content"
177                     android:layout_centerVertical="true"
178                     android:layout_marginLeft="20dp"
179                     android:layout_toRightOf="@+id/imageView7"
180                     android:text="钱包"
181
182                     android:textColor="#000" />
183
184
185             </RelativeLayout>
186
187             <View
188                 android:layout_width="match_parent"
189                 android:layout_height="0.5dp"
190                 android:layout_alignBottom="@+id/rel_fjdr"
191                 android:background="#E1E0DE">
192
193             </View>
194
195             <RelativeLayout
196                 android:layout_width="match_parent"
197                 android:layout_height="50dp"
198                 android:layout_alignParentBottom="true"
199                 android:layout_marginRight="-10.5dp"
200
201                 android:clickable="true">
202
203                 <ImageView
204                     android:id="@+id/imageView9"
205                     android:layout_width="wrap_content"
206                     android:layout_height="wrap_content"
207                     android:layout_alignParentLeft="true"
208                     android:layout_centerVertical="true"
209                     android:src="@android:drawable/ic_delete" />
210
211                 <TextView
212                     android:id="@+id/text_plp"
213                     android:layout_width="wrap_content"
214                     android:layout_height="wrap_content"
215                     android:layout_centerVertical="true"
216                     android:layout_marginLeft="20dp"
217                     android:layout_toRightOf="@+id/imageView9"
218                     android:text="卡包"
219
220                     android:textColor="#000" />
221
222
223             </RelativeLayout>
224         </RelativeLayout>
225
226         <RelativeLayout
227             android:id="@+id/relativeLayout5"
228             android:layout_width="match_parent"
229             android:layout_height="50dp"
230             android:layout_alignParentLeft="true"
231             android:layout_below="@+id/relativeLayout4"
232             android:layout_marginTop="20dp"
233             android:background="@color/white"
234             android:clickable="true">
235
236             <ImageView
237                 android:id="@+id/imageView11"
238                 android:layout_width="wrap_content"
239                 android:layout_height="wrap_content"
240                 android:layout_alignParentLeft="true"
241                 android:layout_centerVertical="true"
242                 android:src="@android:drawable/ic_delete" />
243
244             <TextView
245                 android:id="@+id/text_yxzx"
246                 android:layout_width="wrap_content"
247                 android:layout_height="wrap_content"
248                 android:layout_centerVertical="true"
249                 android:layout_marginLeft="20dp"
250                 android:layout_toRightOf="@+id/imageView11"
251                 android:text="表情"
252                 android:textColor="#000" />
253
254
255         </RelativeLayout>
256
257         <RelativeLayout
258             android:id="@+id/relativeLayout6"
259             android:layout_width="match_parent"
260             android:layout_height="50dp"
261             android:layout_alignParentLeft="true"
262             android:layout_below="@+id/relativeLayout5"
263             android:layout_marginTop="20dp"
264             android:background="@color/white"
265             android:clickable="true">
266
267             <ImageView
268                 android:id="@+id/imageView12"
269                 android:layout_width="wrap_content"
270                 android:layout_height="wrap_content"
271                 android:layout_alignParentLeft="true"
272                 android:layout_centerVertical="true"
273                 android:src="@android:drawable/ic_delete" />
274
275             <TextView
276                 android:id="@+id/text_yxzx1"
277                 android:layout_width="wrap_content"
278                 android:layout_height="wrap_content"
279                 android:layout_centerVertical="true"
280                 android:layout_marginLeft="20dp"
281                 android:layout_toRightOf="@+id/imageView12"
282                 android:text="设置"
283                 android:textColor="#000" />
284
285
286         </RelativeLayout>
287
288
289     </RelativeLayout>
290
291
292 </LinearLayout>

通过分析代码,发现了搭档的几点不足:

  1. 在LinearLayout中使用了如下代码:

    1     android:paddingBottom="@dimen/activity_vertical_margin"
    2     android:paddingLeft="@dimen/activity_horizontal_margin"
    3     android:paddingRight="@dimen/activity_horizontal_margin"
    4     android:paddingTop="@dimen/activity_vertical_margin"

    这些代码是用于RelativeLayout相对布局中的,写在这,会增加无用代码。

  2. XML文件并未添加注释,后期阅读会造成困难。
  3. XML文件格式不美观,没有完整对齐。
  4. 直接在XML文件中使用文字,应该将文字单独放在string目录下。
  5. 使用了诸如以下的已经淘汰的用法: 1 android:layout_width="fill_parent"
  6. 主体框架用的是线性布局,对于安卓设备的兼容性不高。建议更改其他布局方案。
时间: 2024-11-03 01:30:10

对搭档代码的修改意见的相关文章

对搭档代码的一些意见

和搭档一起学,这几天他写了一个demo有关登录界面.我看了下,提出了一些个人的意见. ·代码 1 protected void onCreate(Bundle savedInstanceState) { 2 super.onCreate(savedInstanceState); 3 setContentView(R.layout.activity_main); 4 userName = (EditText) findViewById(R.id.userName); 5 userPassword

git小技巧:git blame && git show 查看某一行代码的修改历史

先查看某行代码由谁写的,在哪个commit中提交的: git blame file_name git blame -L 58,100 KeyboardActivity.java 其显示格式为: commit ID | 代码提交作者 | 提交时间 | 代码位于文件中的行数 | 实际代码 类似于下面这样: 这样,我们就可以知道commit ID了,然后使用命令:Git show commitID来看~ git小技巧:git blame && git show 查看某一行代码的修改历史

代码手动修改约束(AutoLayout)

当使用xib或storyBoard构建项目,并使用了AutoLayout之后,当需要为视图添加动画,或者手动更改视图的frame的时候,就需要修改约束啦.别以为代码中修改约束很麻烦,其实还蛮简单的啦. 例如: 跟随键盘弹出的ToolBar,原来在视图底部,当键盘弹出时,ToolBar跟随键盘弹出 首先将ToolBar到底部的约束添加一个IBOutlet @property (weak, nonatomic) IBOutlet NSLayoutConstraint *toolViewBottomC

历次PCB板修改意见汇总

历次PCB板修改意见汇总: 1 对于主控芯片,建议参考官方的PCB布局,官方的PCB布局肯定是为了最大程度的发挥主控的性能. 2 LDO要选择低功耗的,静态电流越小越好,估算一下板子的最大电流,选择LDO最大电流大于工作电流一倍即可.可以参考低功耗稳压芯片HT7333,HT7333-A系列的LDO 3 天线和高速晶振(16MHz)要放到一边,高速晶振附近尽量少布线.(这里也可以参考官方MCU的layout) 4 芯片的一定要考虑是否容易买,量是否大,是否容易替换.不要出现板子打出来了,但是芯片不

iOS开发小技巧-修改SliderBar指针的样式(牢记这个方法,只能通过代码来修改)

代码: // 修改进度条的指针图片 [self.progressSlider setThumbImage:[UIImage imageNamed:@"player_slider_playback_thumb"] forState:UIControlStateNormal]; 原来的样子: 现在的样子:

git fork代码并修改胡提交到自己的git仓库

最近在参加阿里天池大数据中间件比赛(毫无头绪,打酱油中).看参赛要求,需要将官网的git工程clone下来,在此基础上做修改后提交到自己的仓库中. 由于以前并没有使用过git,所以差了比较多的资料,做了不少尝试,现在终于搞成了. 主要参考了这个链接:http://blog.csdn.net/killzero/article/details/10441169 1.首先需要在git仓库中建立自己的工程,我这直接fork了官方demo,地址为:[email protected]:middlewarer

使用JS代码打印 修改博客园(或网站)的title

title在html中属于特殊的节点元素. 因为它可以使用doucment.getElementsByTagName("title")[0]来获取它的标题,但却无法用doucment.getElementsByTagName("title")[0]用更改它的值. 但是,总有解决的方法. 在javascript中,修改title的方法是: <script type="text/javascript"> document.title =

Android button, xml文件定义形状,代码中修改背景颜色

1. 首先在drawable文件夹定义一个shape.xml文件,内容如下: <?xml version="1.0" encoding="UTF-8"?><shape    xmlns:android="http://schemas.android.com/apk/res/android"    android:shape="rectangle">    <solid android:color=

Eclipse快捷键自动生成注释代码,修改预格式

用Eclipse编写Java代码,可以快捷键自动生成注释代码,但是自动生成的注释信息都是预先设置好的格式. 其实我们可以在Eclipse中进行设置自己希望显示的信息格式. 方法:点击Windows->Preferences->Java->Code Style->Code Templates, 选择右边窗口中的Comments,对具体的注释信息进行编辑设置. 例如:设置作者作者信息以及日期信息. 选择Types,点击Edit,可以对原来的模板进行修改,原格式如下: /**  * @a