2016-09-20 138 views
2

我在循环中比较两个字符串时遇到了问题。比较ArrayList中的两个字符串

说我想把两个单词放在ArrayList中。我决定通过inputDialogeBox写两个单词并将它们放入列表中。如果两个单词相同,则第二个单词不会被添加到列表中。

但是,当我比较两个字符串程序似乎不关心,如果他们是相同的,并且都在列表中结束。

继承人我的代码:

package testing; 

import javax.swing.*; 

public class Testing { 

    public static void main(String[] args) { 
     Word c = new Word(); 

     // word 1 
     String word; 
     word = JOptionPane.showInputDialog("Write a word: "); 
     System.out.println("word1 = " + word); 
     c.setWord(word); 

     // word 2 
     word = JOptionPane.showInputDialog("Write a new word: "); 
     System.out.println("word2 = " + word); 
     c.setWord(word); // sätt kortet≤ 

     System.out.println("words = " + c.getWord().size() + " " + c.getWord()); 
    } 
} 

而我的等级:

package testing; 

import java.util.ArrayList; 

public class Word { 

    ArrayList<String> words = new ArrayList<>(); 

    public void setWord(String word) { 
     // set first value in arraylist to start the loop 
     if (words.isEmpty()) { 
      words.add("default"); 
     } 

     for (int i = 0; i < words.size(); i++) { 
      if (!words.get(i).equals(word)) { 
       words.add(word); 
       System.out.println("words.get(i): " + words.get(i) + " word: " + word); 
       break; 
      } 
     } 
    } 

    public ArrayList getWord() { 
     return words; 
    } 
} 

我的猜测是,这个问题是我周围添加一个默认值,只是为了找点与循环。否则,如果ArrayList为空,我不能启动我的循环。但也许有更好的方法?

感谢

+2

如果列表中有*任何*字不是单词,您正在将该单词添加到列表中。 – Andreas

+0

对不起。我是一个初学者:-)
1)如果ArrayList为空 - 将第一个值添加到列表 2)我遍历该唯一值。 3)如果字符串(默认)与第一个字不同,它将第一个字添加到ArrayList 4)如果字符串(默认和word1)不是第二个字,它将它添加到ArrayList 但我想我搞砸了:-) – acroscene

+2

如果你想防止重复,使用一套,而不是一个列表。 –

回答

1

的问题是,你不检查所有的值,只有一个接一个。对于第一个字,你检查'默认'并添加到列表中。对于第二个单词,首先检查“默认”,并且由于这不相同,所以也将该单词添加到列表中。

public void setWord(String word) { 

    boolean exists = false; 
    for (int i = 0; i < words.size(); i++) { 
     if (words.get(i).equals(word)) { 
      exists = true; 
      break; 
     } 
    } 
    if (!exists) { 
     words.add(word); 
     System.out.println("words.get(i): " + words.get(i) + " word: " + word); 
    } 
} 

或者你可以使用ArrayList包含:

public void setWord(String word) { 

    if(!words.contains(word)) { 
     words.add(word); 
     System.out.println("words.get(i): " + words.get(i) + " word: " + word); 
    } 
} 
1

在这些类型,你应当使用的,而不是ArrayList的案件。 仍然下面的代码修改应该可以帮到你。

public void setWord(String word) {' 
     boolean found = false;  
     for (int i = 0; i < words.size(); i++) { 
      if (words.get(i).equals(word)) { 
       found=true; 
       break; 
      } 
     } 
     if(!found){ 
      words.add(word); 
     } 
    } 
+0

谢谢!这样可行!我如何使用Set编写它? – acroscene

+0

Set words = new HashSet(); set.add(单词)....它会照顾一切 – Shiva