Question

我对Java编码非常陌生，而且我遇到了一个问题，这个问题在过去几个小时里一直困扰着我。我正在创建一个非常小而简单的应用程序，它将生成一首诗。我有四个带有“名词”，“动词”，“形容词”和“副词”的数组列表。这首诗将从每个列表中取出一个单词并将其组合成一行四行诗，每行每个单词一个单词，如下所示：

Adverbs Adjectives Nouns Verbs
Adverbs Adjectives Nouns Verbs
Adverbs Adjectives Nouns Verbs
Adverbs Adjectives Nouns Verbs

这将创造一首毫无意义的诗......无论如何......我的问题是结果如下：

public void generatePoem() {
    adjectives.add("adjective1");
    adjectives.add("adjective2");
    adjective.add("adjective3");
    adjective.add("adjective4");
    nouns.add("noun1");
    nouns.add("noun2");
    nouns.add("noun3");
    nouns.add("noun4");
    verbs.add("verb1");
    verbs.add("verb2");
    verbs.add("verb3");
    verbs.add("verb4");
    adverbs.add("adverb1");
    adverbs.add("adverb2");
    adverbs.add("adverb3");
    adverbs.add("adverb4");



    String thepoem = ""; 


        for (int i=0;i<16;i++) { 
            int count = i + 1; 
            int random = (int)(Math.random()*4);



                    if (count % 2 == 0) { 
                        thepoem += adjectives.get(random).toString() + " "; 
                    } 
                    else if (count % 3 == 0) { 
                        thepoem += nouns.get(random).toString() + " ";
                    } 
                    else if (count % 4 == 0) {
                        thepoem += verbs.get(random).toString() + "\n"; 
                    } 
                    else { 
                        thepoem += adverbs.get(random).toString() + " ";
                    }



        }

        JOptionPane.showMessageDialog(null, thepoem);

}
}

结果是随机的，但它总是这样：

Adverb Adjective Noun Adjective(?!?!)
Adverb Adjective Noun Adjective(?!?!)
Adverb Adjective Noun Adjective(?!?!)
Adverb Adjective Noun Adjective(?!?!)

动词在哪里？

我似乎无法找到解决方案......有人请帮帮我吗？同样，我还处于Java编程的早期阶段。

Answer 1

如果count%4==0则count%2==0。因此，您永远不会达到相应的陈述。

if (count % 2 == 0) { 

} 

else if (count % 4 == 0) { // <-- You will never reach this statement.

}

你应该只使用一些模4。它允许你使用switch语句。

switch (count % 4) {
  case 0 : 
    thepoem += verbs.get(random).toString() + "\n";
    break;
  case 1 :
    thepoem += adverbs.get(random).toString() + " ";
    break;
  case 2 :
    thepoem += adjectives.get(random).toString() + " ";
    break;
  case 3 :
    thepoem += nouns.get(random).toString() + " ";
    break;
}

要更进一步，使用StringBuilder而不是+=附加字符串也会更优雅，因为+=会创建一个新的字符串每次都是前2个的副本。

示例：

StringBuilder sb = new StringBuilder();

for (int i=0;i<16;i++) {
  ...

switch (count % 4) {
  case 0 : 
    sb.append(verbs.get(random).toString() + "\n");
    break;
  case 1 :
    sb.append(adverbs.get(random).toString() + " ");
    break;

  ....
  JOptionPane.showMessageDialog(null, sb.toString());

再远一点，

正如@IanMcLaird指出的那样，你也可以（实际上，你真的应该）在没有模数的情况下这样做（参见评论和IanMcLaird的回答）。

如果我将IanMcLaird的答案与StringBuilder结合起来，那么最好的解决方案是恕我直言：

StringBuilder sb = new StringBuilder();
Random random = new Random();
for (int i = 0; i < 4; ++i) {
  sb.append(adverbs.get(random.nextInt(4)) + " ");
  sb.append(adjectives.get(random.nextInt(4)) + " ");
  sb.append(nouns.get((random.nextInt(4)) + " ");
  sb.append(verbs.get((random.nextInt(4)) + "\n");
}

Answer 2

如果count的值为4，那么4％2 == 0将返回true。这就是为什么你得到形容词而不是副词的原因。

Answer 3

或者，您可以通过行进行迭代，而不是在所有中使用modulo 。此外，创建一个真正的随机生成器，以获得更真实的随机结果。

Random random = new Random(); for (int i = 0; i < 4; ++i) { thePoem += adverbs.get(random.nextInt(4)) + " "; thePoem += adjectives.get(random.nextInt(4)) + " "; thePoem += nouns.get((random.nextInt(4)) + " "; thePoem += verbs.get((random.nextInt(4)) + "\n"; }

Answer 4

任何与你的动词匹配的内容（计数％4）也会与你的形容词条件匹配（并被其消耗）（计数％2）

Answer 5

任何可被4整除的东西总是可被2整除，因此您的代码永远不会被4整除，因此无动词。一个解决方法是交换此检查，即首先检查它是否可被4整除，然后如果它可以被2整除

Answer 6

更改此行：

if (count % 2 == 0)

为：

if (count % 2 == 0 && count % 4 != 0)

看看你是否得到了正确的结果。

用于循环输出的Java不显示我想要的方式

6 个答案: