黑马程序员技术交流社区

标题: 代码题！帮我找一下错误 [打印本页]

---

作者: 樊占江    时间: 2012-8-14 00:21
标题: 代码题！帮我找一下错误

import java.io.*;
public class wordtext
{
public static void main(String[] args)
{
String s=" ";
String w=" ";
    System.out.print("input:\n");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
s=in.readLine();}
catch(IOException e){}
System.out.print("input text word!");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
w=in.readLine();}
catch(IOException e){}
int first=0,second=0,end=0;
for(int i=1;i<=s.length();i+=second+1){
second=s.indexOf(" ");
String t=s.substring(first,second);
s=s.substring(second+1);
if(w.equalsIgnoreCase(t))end++;
}
if(end==0)System.out.println("there no "+w);
else System.out.println("there"+end+w);
}
}

输入文章，再输入要查询的单词，统计查询单词出现几次！！

---

作者: 黎健东    时间: 2012-8-14 01:00

1. package com.heima.test;
   
2. 
   
3. import java.io.*;
   
4. 
   
5. public class wordtext {
   
6.         public static void main(String[] args) {
   
7.                 String s = " ";
   
8.                 String w = " ";
   
9.                 System.out.print("input:\n");
   
10.                 try {
    
11.                         BufferedReader in = new BufferedReader(new InputStreamReader(
    
12.                                         System.in));
    
13.                         s = in.readLine();
    
14.                         System.out.println(s);
    
15.                 } catch (IOException e) {
    
16.                 }
    
17.                 System.out.print("input text word!");
    
18.                 try {
    
19.                         BufferedReader in = new BufferedReader(new InputStreamReader(
    
20.                                         System.in));
    
21.                         w = in.readLine();
    
22.                         System.out.println(w);
    
23.                 } catch (IOException e) {
    
24.                 }
    
25.                
    
26. 
    
27.                 int wLen = w.length();
    
28.                 int sLen = s.length();       
    
29.                 int beginIndex = 0;
    
30.                 int endIndex = wLen;
    
31.                 int count = 0;
    
32.                 //sLen - wLen就是走过的距离
    
33.                 for(int i = 0; i < sLen - wLen; i++){
    
34.                         String word = s.substring(beginIndex, endIndex);
    
35.                         if(word.equalsIgnoreCase(w)){
    
36.                                 count++;
    
37.                         }
    
38.                         beginIndex++;
    
39.                         endIndex++;
    
40.                 }       
    
41.                 if (count == 0)
    
42.                         System.out.println("there no " + w);
    
43.                 else
    
44.                         System.out.println("there" + count + w);
    
45.                
    
46.                
    
47. //                int first = 0, second = 0, end = 0;
    
48. //                for (int i = 1; i <= s.length(); i += second + 1) {
    
49. //                        second = s.indexOf(" ");
    
50. //                        String t = s.substring(first, second);
    
51. //                        s = s.substring(second + 1);
    
52. //                        if (w.equalsIgnoreCase(t))
    
53. //                                end++;
    
54. //                }
    
55. //                if (end == 0)
    
56. //                        System.out.println("there no " + w);
    
57. //                else
    
58. //                        System.out.println("there" + end + w);
    
59.         }
    
60. }
    
61. 
    

复制代码

我把你的代码中的fo修改了，你的问题是出现在indexOf这里你想表达什么不清楚，这个值取到的是一个负数，你在数组中使用这个的话，当然是数组越界了
而且你的思路感觉不清醒或者有些麻烦

比如：s = aaabcddddbceeeebc
查询w = bc出现的次数

用s的长度- w的长度，就得到你需要比较的次数了
然后用substring(beginIndex,statrIndex)依次比较就好了
比完依次++就OK了

---

作者: 周世阳    时间: 2012-8-14 01:00


thanks to 黎健东同学的提醒修改一下，
indexOf（“ ”）在不存在“ ”自负串的时候会返回-1，String t=s.substring(0,-1);就会产生数组下表越界
可以将
second=s.indexOf(" ");
String t=s.substring(first,second);
加一个if判定来处理：修改为--->
second=s.indexOf(" ");
if(second!=-1)
{String t=s.substring(first,second); }
else{t=s;}




建议用spit方法直接将语句分解成单词的数组，这样简洁点也不会产生越界问题

import java.io.*;
public class wordtext
{
        public static void main(String[] args)
        {         
                int times=0;
                String s=" ";
                String w=" ";
                System.out.print("input:\n");
                try{
                        BufferedReader in=new BufferedReader(
                        new InputStreamReader(System.in));
                        s=in.readLine();
                } catch(IOException e){}
                System.out.print("input text word!");
                try{
                        BufferedReader in=new BufferedReader(
                        new InputStreamReader(System.in));
                        w=in.readLine();
                }
                catch(IOException e){}
               
                String [] words=s.split(" ");
                for(String word:words){
                        if(word.equalsIgnoreCase(w))times++;
                }//与你得代码不同之处

                if(times==0)System.out.println("there no "+w);
                else System.out.println("there"+times+w);
        }
}

---

作者: 黎健东    时间: 2012-8-14 01:02

周世阳 发表于 2012-8-14 01:00
代码运行无错，不过可以改进一下用spit方法直接将语句分解成单词的数组，这样简洁点

import java.io.*;

你多试几次，数组越界了...............

---

作者: 黑马-郑鹏    时间: 2012-8-14 01:34
import java.io.*;
public class wordtext
{
public static void main(String[] args)
{
String s=" ";
String w=" ";
    System.out.print("input:\n");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
s=in.readLine();}
catch(IOException e){}
System.out.print("input text word!");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
w=in.readLine();}
catch(IOException e){}
int first=0,second=0,end=0;
for(int i=1;i<=s.length();i+=second+1){
//这个s值会随着你这个字串长度的长度不断减小，而你这个i会一直增大，导致你到后面字串的长度小于i值时，i<=s.length()条件已经不满足，虽然你的字串中还有你要查找的字符串，那for循环中的语句也不执行了，导致你的统计结果有时会比实际的偏小
second=s.indexOf(" ");
String t=s.substring(first,second);
s=s.substring(second+1);
if(w.equalsIgnoreCase(t))end++;
}
if(end==0)System.out.println("there no "+w);
else System.out.println("there"+end+w);
}
}
你可以这样：
import java.io.*;
public class wordtext
{
public static void main(String[] args)
{
String s=" ";
String w=" ";
    System.out.print("input:\n");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
s=in.readLine();}
catch(IOException e){}
System.out.print("input text word!");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
w=in.readLine();}
catch(IOException e){}
int first=0,second=0,end=0;
int strLen = s.length();//先把初始字符串的长度保存下来，这样就好了
for(int i=1;i<=strLen;i+=second+1){
second=s.indexOf(" ");
String t=s.substring(first,second);
s=s.substring(second+1);
if(w.equalsIgnoreCase(t))end++;
}
if(end==0)System.out.println("there no "+w);
else System.out.println("there"+end+w);
}
}

---

作者: 黑马-郑鹏    时间: 2012-8-14 01:51
import java.io.*;
public class wordtext
{
public static void main(String[] args)
{
String s=" ";
String w=" ";
    System.out.print("input:\n");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
s=in.readLine();}
catch(IOException e){}
System.out.print("input text word!");
try{
BufferedReader in=new BufferedReader(
new InputStreamReader(System.in));
w=in.readLine();}
catch(IOException e){}
int first=0,second=0,end=0;
int strLen = s.length();//先把初始字符串的长度保存下来，这样就好了
for(int i=1;i<=strLen;i+=second+1){
second=s.indexOf(" ");

if(second==-1)
{
    break;  //加上这段代码，会完善一下，不然当你字符串s的最后面不加空格的话，会报错
}

String t=s.substring(first,second);
s=s.substring(second+1);
if(w.equalsIgnoreCase(t))end++;
}
if(end==0)System.out.println("there no "+w);
else System.out.println("there"+end+w);
}
}

---

作者: 周世阳    时间: 2012-8-14 02:15

黎健东 发表于 2012-8-14 01:02
你多试几次，数组越界了...............

看到了。。。
indexOf(" ")会产生-1，thanks

---

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

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