黑马程序员技术交流社区

标题: 写着写着问题就出来了，想着想着就不明白了。我困惑了 [打印本页]

作者: 常万 时间: 2012-4-25 10:08
标题: 写着写着问题就出来了，想着想着就不明白了。我困惑了
同样的实现不一样的写法.....我困惑了

import java.util.*;
class TreeMapTest
{
public static void main(String[] args)
{
String s = "aafsdl,gdslkj/ssjf";
charCount(s);

}
public static String charCount(String str)
{
char[] chs = str.toCharArray();

int count = 0;
TreeMap<Character,Integer> tm = new TreeMap<Character,Integer>();
for (int x=0;x<chs.length; x++)
{ //字符串中可能出现其他字符，所以这里我们要判断一下
if(chs[x]>='a'&&chs[x]<='z'||chs[x]>='A'&&chs[x]<='Z')
{ /*
Integer Value = tm.get(chs[x]);
if(!(Value==null))//if(Value!==null)这两句我觉得是相同的意思，但是却有着不一样的效果？ count = Value;
count++;
tm.put(chs[x],count);
*/
Integer Value = tm.get(chs[x]);
if(Value == null)
tm.put(chs[x],1);
else
{
Value+=1;
tm.put(chs[x],Value);
}
}
//else不加这个运行也正常，加else有什么意义？ //continue;

}
//System.out.println(tm);
//StringBuilder sb = new StringBuilder();下面那样也可以实现结果，为什么要要定义容器在去开辟空间？ Set<Map.Entry<Character,Integer>> entrySet = tm.entrySet();
Iterator<Map.Entry<Character,Integer>> it = entrySet.iterator();
while(it.hasNext())
{
Map.Entry<Character,Integer> me = it.next();
Character ch = me.getKey();
Integer in = me.getValue();
System.out.print(ch.toString()+"("+in.toString()+")");
}

return null;

}
}
高手帮我分析下我这样写的弊端:(

作者: 常万 时间: 2012-4-25 10:09
汗....加个红色字体把代码挤歪了...

作者: 刘基军 时间: 2012-4-25 10:41
1.
if(!(Value==null))与if(Value!==null) //不等于应该是"!=",而不是"!=="
2.
else
continue; //这边加不加确实是一样的,因为已经是for循环最后了,后面没有其他代码了
3.
没明白意思,...-_-

作者: 常万 时间: 2012-4-25 17:50

刘基军发表于 2012-4-25 10:41
1.
if(!(Value==null))与if(Value!==null) //不等于应该是"!=",而不是"!=="
2.

            StringBuilder sb = new StringBuilder();
            Set<Map.Entry<Character,Integer>> entrySet = tm.entrySet();
            Iterator<Map.Entry<Character,Integer>> it = entrySet.iterator();
            while(it.hasNext())
            {
                     Map.Entry<Character,Integer> me = it.next();
                     Character ch = me.getKey();
                     Integer in = me.getValue();
                     sb.append(ch+"("+in+")");
                     //System.out.print(ch.toString()+"("+in.toString()+")");
            }
            return sb.toString();

我的意思是定义一个容器然后添加数据return sb.toString();

和直接System.out.print(ch.toString()+"("+in.toString()+")");
然后在return null哪个好点？

作者: 刘基军 时间: 2012-4-25 17:58

常万发表于 2012-4-25 17:50
StringBuilder sb = new StringBuilder();
Set entrySet = tm.entrySet ...

我认为方法返回字符串给调用者,让其自己对返回值操作比较合适一点.

作者: 丁佼 时间: 2012-4-25 18:21
1、if(!(Value==null))是判断Value是否为空，之后取反，最终条件判断的是Value不等于空则执行if后的语句
if(Value!=null)，直接判断Value是否为非空，最终条件执行要求和上面是一样的。
感觉上好像是下面一个少运算一步，电脑应该是不在乎这么点性能了，不过对于人类，我是觉得下面的一种更简洁点，上面的那个就是跟自己过不去，画蛇添足的，给自己增加逻辑难度。
2、esle确实没什么用，如果是你自己的代码的话那就可以删了，如果是别人代码的话，也许是别人预留用来以后扩展代码的位置；也可能是人家的书写习惯，或者是人家写代码的软件自动添加的。
3、可以返回StringBuilder对象共调用者对数据进行处理，也可以直接在函数里将数据处理掉，不提供返回值。这完全取决于你对函数的需求，以及你的调用函数是否对返回值有处理需求，如果没有处理需求的话尽量不要返回，这好像在软件工程里有个词叫减少“耦合度”什么的。

总之
1、代码的原则就是尽量简单易读，有个很厉害的人说过“如果对一个问题有两种解决方式，那简单的那个就是正确的方式”。
2、扩展性很重要，不过哪里要留。。看个人能力，像我的话就一般不留了，因为能力不足，完全不会预判，都是现用现加。。。。
3、是否要提供返回值，这完全取决于你的数据是否需要调用者去处理，尽量把事情在函数里做掉，做不完的再把值返回调用者处理。

作者: 常万 时间: 2012-4-25 19:19

丁佼发表于 2012-4-25 18:21
1、if(!(Value==null))是判断Value是否为空，之后取反，最终条件判断的是Value不等于空则执行if后的语句
...

分析的很有道理..感谢~~其实那个esle是写在前面判断字符串中可能出现其它不需要的字符，我觉得把if(!(chs[x]>='a'&&chs[x]<='z'||chs[x]>='A'&&chs[x]<='Z'))
然后在return continue没有必要。不如直接判断字符是否在字符串中存在。就不知道加一个continue和不加有什么区别

作者: 邵中国 时间: 2012-4-26 17:31
1.两个不要的，“！=”
2.如果单个字符过多的话，每次都要执行下面的代码，浪费时间
3.一般来说，加sb就是用空间换时间

欢迎光临黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2