Android文本函数forcecloses如果给出超过3个单词

时间:2012-05-26 17:48:00

标签: java android hashmap forceclose

我是java的新手,来自c ++,并快速学习将我的聊天机器人转换为java android应用程序的不同实践。这个函数在这里强制关闭,如果它有超过3个不同的单词,我不知道为什么。如果有人能帮助我击败这个谜语,我会很激动!

public void AI(String text) {
// initiate new lists to be on the safe side
List<String>    indexer = new ArrayList<String>();
List<String>    sentence = new ArrayList<String>();
List<String>    explode = new ArrayList<String>();
List<String>    ary = new ArrayList<String>(); 

explode = Arrays.asList(text.split(" "));  

// initiate randint and trigger variable
Random rand = new Random();
int randint = rand.nextInt(explode.size());

// initiate the trigger variable
String trigger = explode.get(randint); 


     // check if word exists in database and add if it not.
for(int i = 0; i < explode.size(); i++) {
    String word = explode.get(i);   

if(common.get(word)==null) {
                        words.add(word);
                        common.put(word, 1);
                        context.put(word, explode);
                        pos.put(word, i);
                        length.put(word, explode.size());

                         }else{
        // increase the weight of common if the word repeats
                         common.put(word, common.get(word)+1 );
                                     }
                                  }
     //check if context with index set as trigger is empty, if not, copy the   arraylist to the ary variable
     if(!context.get(trigger).isEmpty()) {
            Collections.copy( ary, context.get(trigger));}


     // fill the arraylist sentence with words to be used, some in context, some random from the database.
 for(int i2 = 0; i2 < length.get(trigger); i2++ ) {

                  randint = rand.nextInt(length.get(trigger));

if(randint < length.get(trigger)/2) {
                                        //
                                         if(ary.get(i2)!=null) {
                                                                   sentence.add(ary.get(i2));
                                                               }
                                    }else{
                        sentence.add(words.get(rand.nextInt(words.size())));
                                         }

}    

   // use to the pos-hashmap to check at which index the word was detected at, if not     place it at the deliberate index.
for(int i3 = 0; i3 < sentence.size(); i3++) {
                                            if(sentence.get(i3)!=null) {    
                                            indexer.add(pos.get(sentence.get(i3)), sentence.get(i3));   
                                                                       }
                                            }

// compose the final string that is to be passed to the speak function
for(int i4 = 0; i4 < indexer.size(); i4++) {
say = say + indexer.get(i4)+" ";    
    }   

// pass the string to the speak function
mTts.speak(say, TextToSpeech.QUEUE_FLUSH, null);     


// removing final string to be ready for next iteration
    say = "";



// end of AI stuff          
}       

logcat的:

05-26 17:40:08.266: E/AndroidRuntime(490): Uncaught handler: thread main exiting due to  uncaught exception
05-26 17:40:08.276: E/AndroidRuntime(490): java.lang.ArrayIndexOutOfBoundsException
05-26 17:40:08.276: E/AndroidRuntime(490):  at  java.util.Collections.copy(Collections.java:1593)
05-26 17:40:08.276: E/AndroidRuntime(490):  at sarah.namespace.SarahActivity.AI(SarahActivity.java:163)

变量:

// arraylists
public List<String> explode = new ArrayList<String>();
    public List<String> words = new ArrayList<String>();
    public List<String> sentence = new ArrayList<String>();
    public List<String> indexer = new ArrayList<String>();
// hashmaps
    public Map<String, Integer> common = new HashMap<String, Integer>();
public Map<String, List<String>> context = new HashMap<String, List<String>>();
public Map<String, Integer> pos = new HashMap<String, Integer>();
public Map<String, Integer> length = new HashMap<String, Integer>();

 // strings
public String say;

Logcat行指向:

if(!context.get(trigger).isEmpty()) {
            Collections.copy( ary, context.get(trigger));}

1 个答案:

答案 0 :(得分:0)

我没有查明问题,但我相信这种替代方法更快,应该避免。 (老实说,你的代码在第四个单词崩溃了?这很奇怪。我相信它还有更多......无论如何)

在Java for-each loops中更快更简单,所以:

// check if word exists in database and add if it not.
int i = 0;
for(String word : explode) {
    if(common.get(word) == null) {
        if(word.equals(trigger)) {
            ary = new ArrayList<String>(explode);
        }

    ...
    i++;
}

如果trigger中不存在context,则返回null,而不是空List,因此您可以进行简单的更改:

if(context.get(trigger) != null && !context.get(trigger).isEmpty()) {
    Collections.copy(ary, context.get(trigger));
}

但是这应该抛出NullPointerException而不是出界...我仍然不知道为什么第四个单词是致命的。希望这会有所帮助。