求帮忙看一下我这一题，哪儿逻辑有问题。。。输出的结构总是不对。。。
我是希望被找出我写的代码中的逻辑问题。。不需要帮我写出别的解决方法,谢谢。。。
package cn.itcast;
/*
* 匹配5-15位数字，首位非0
*/
public class Demo4 {

        public static void main(String[] args) {
                String s = "1032aa20";
                System.out.println(checkQQ(s));

        }
       
        public static boolean checkQQ(String QQString)
        {
                boolean flag = true;

                if(QQString.charAt(0)!=0)
                {
                        if (QQString.length()>15 || QQString.length()<5 )
                        {
                                flag = false;
                        }else
                                {
                                for (int i = 0; i < QQString.length(); i++) {
                                       
                                        if(QQString.charAt(i)<'0' && QQString.charAt(i)>'9')
                                        {       
                                                flag = false;
                                                break;
                                        }
                                }
                        }
                }
                else
                {
                        flag =false;
                }
                return flag;               
        }

}

发代码格式的，总是出问题。。。所以编辑了重新发的。。。。。。
要求：判断String s 是否为5-15位的纯数字。。。而且要求首位为非0的数字。。。

1. 
   
2. if(QQString.charAt(i)<'0' && QQString.charAt(i)>'9')//这里是或的关系，不是与
   

复制代码

hejinzhong 发表于 2014-8-4 23:06

感谢楼上回答。。。。这确实是个问题。。。不过除了这个问题。。还有别的问题。。。

正则表达式应该是最简单的方法吧。

if(QQString.charAt(0)!=0)改成 if(QQString.charAt(0)!='0')  因为charAt()方法返回值是char

      for (int i = 0; i < QQString.length(); i++) {
                                       
                                        if(QQString.charAt(i)<'0' && QQString.charAt(i)>'9')
                                        {        
                                                flag = false;
                                                break;
                                        }
                                }
首先如果这个字符串中有字母的话if判断中的flag=false不会执行的，这个你知道的吧，不满足条件不执行，所以这个时候就直接返回flag了，也就是true，为了避免这个问题，在这个if判断后面还要继续判断才行
你可以这样判断

1. for (int i = 0; i < s.length(); i++) {
   
2. 
   
3. if(s.charAt(i)>='0' && s.charAt(i)<='9'){
   
4. flag = true;
   
5. }
   
6. else{
   
7. System.out.println("ok");
   
8. flag=false;
   
9. }
   
10. 
    
11. }

复制代码

这样的话就不要加，break了，不然会出现之判断一次的就退出了，结果还是错的

当然按照你的方法也可以这样判断

1. if(s.charAt(i)<'0' && s.charAt(i)>'9')
   
2. {
   
3. flag = false;
   
4. break;
   
5. }
   
6. else if(s.charAt(i)>='a'&&s.charAt(i)<='z'){
   
7. flag=false;
   
8. }

复制代码

再判断一下，不然直接返回flag=true了，我简单的写了，希望能帮上忙

1.   if(QQString.charAt(0)!=0) //charAt(index)方法返回的是字符类型，改成
   
2. if(QQString.charAt(0)!='0')

复制代码

唕 发表于 2014-8-4 23:50
if(QQString.charAt(0)!=0)改成 if(QQString.charAt(0)!='0')  因为charAt()方法返回值是char

这个地方我开始也没注意。。。。
不错。。迅速就能看出这个错误，，很牛叉。。。

燿陚√揚葳 发表于 2014-8-5 00:07
for (int i = 0; i < QQString.length(); i++) {
                                       
        ...

感谢回复。。
不过这地方。。if(s.charAt(i)<'0' && s.charAt(i)>'9')
这个判断的意思，是在非数字范围内（非0-9之间），就返回false ，
而这一题，出现一个字符，就直接返回false了，所以这儿只要在非数字范围内，就直接得到false ，然后break跳出所有循环，直接return false了。。

所以不需要用if else进行判断了。。。。
不过还是感谢回复。。。

QQString.charAt(0)返回的是Unicode编码，不能直接和0比较，0的Unicode编码是48

烟海 发表于 2014-8-5 00:35
感谢回复。。
不过这地方。。if(s.charAt(i)'9')
这个判断的意思，是在非数字范围内（非0-9之间），就返 ...

if(s.charAt(i)<'0' && s.charAt(i)>'9')
这个并不是在非数字范围内（非0-9之间），就返回false，而是 s.charAt(i) 大于57 且 小于 48 相当于
if(false)
好像有人说过了

来晚了。...........

你这一大串代码正则一句话就搞定了flag = s.matches("[1-9]\\d{4,14}")?true:false;