所以我有这个程序可以生成贵公司的随机描述。
这主要用于链接建设网站等，因此您没有重复数据。我在JFrame添加了一个按钮，这样当按下它时会重新加扰句子而不是退出程序并再次填写所有信息。
我一直在尝试使用群组，但似乎无法弄明白。有什么想法吗？
我不希望程序再次询问信息，所以我需要循环从随机变量开始的地方开始。
if (actionCommand.equals("Product")) {
JButton Pbutton;
Pbutton = new JButton("Generate");
final String name = JOptionPane.showInputDialog(null, "Enter your Business name: \n \n ex: We at 'your business' deliver...");
final String productP = JOptionPane.showInputDialog(null, "Enter your Business product(plural): \n \n ex: take pride in our 'pizza slicers'");
final String productS = JOptionPane.showInputDialog(null, "Enter your Business product(singular): \n \n ex: we make the highest quality 'pizza slicer' in...");
final String location = JOptionPane.showInputDialog(null, "Where is your business located? \n \n ex: Cityville, Virginia.");
final String keywords = JOptionPane.showInputDialog(null, "Enter 6 keywords for your product: \n \n Ex: pizza, pizza slicer, cityville, Virginia, tools, restaurant equipment");
final String URL = JOptionPane.showInputDialog(null, "Enter your website URL: \n \n ex: www.yourwebsite.com");
Random rand1 = new Random();
int number1 = rand1.nextInt(3);
if (number1 == 2)
v1 = "We at ";
else if (number1 == 1)
v1 = "The staff at ";
else if (number1 == 0)
v1 = "The professionals at ";
Random rand2 = new Random();
int number2 = rand2.nextInt(5);
if (number2 == 4)
v2 = " take great pride in our ";
else if (number2 == 3)
v2 = " know the importance of having great ";
else if (number2 == 2)
v2 = " bring you only the best ";
else if (number2 == 1)
v2 = " offer only the greatest ";
else if (number2 == 0)
v2 = " are bringing you top-of-the-line ";
Random rand3 = new Random();
int number3 = rand3.nextInt(5);
if (number3 == 4)
v3 = " We offer the highest quality " + productS + " ";
else if (number3 == 3)
v3 = " Let us be the ones to supply you with " + productP + " ";
else if (number3 == 2)
v3 = " No one can compete with our high quality " + productP + " ";
else if (number3 == 1)
v3 = " Only we can offer these a " + productS + " of this quality ";
else if (number3 == 0)
v3 = " Turn to us if you need a " + productS + " ";
JFrame.setDefaultLookAndFeelDecorated(true);
final JFrame frame = new JFrame(name);
frame.getContentPane().setBackground(JMUpurple);
frame.setLayout(new GridLayout(2, 3));
String text = v1 + name + v2 + productP + "." + v3 + "in " + location + ".";
String text2 = name + " of " + location + "| High quality " + productP;
String text3 = keywords;
String text4 = URL;
Font font = new Font("Georgia", 0, 12);
JTextArea textArea1 = new JTextArea(text, 5, 25);
JTextArea textArea2 = new JTextArea(text2, 5, 25);
JTextArea textArea3 = new JTextArea(text3, 10, 25);
JTextArea textArea4 = new JTextArea(text4, 2, 25);
textArea2.setPreferredSize(new Dimension(100, 100));
textArea3.setPreferredSize(new Dimension(50, 50));
JScrollPane scrollPane = new JScrollPane(textArea1, 22,
32);
JScrollPane scrollPane2 = new JScrollPane(textArea2, 22,
32);
JScrollPane scrollPane3 = new JScrollPane(textArea3, 22,
32);
JScrollPane scrollPane4 = new JScrollPane(textArea4, 22,
32);
textArea1.setWrapStyleWord(true);
textArea1.setLineWrap(true);
textArea2.setWrapStyleWord(true);
textArea2.setLineWrap(true);
textArea3.setWrapStyleWord(true);
textArea3.setLineWrap(true);
textArea4.setWrapStyleWord(true);
textArea4.setLineWrap(true);
frame.add(scrollPane);
frame.add(scrollPane2);
frame.add(scrollPane3);
frame.add(scrollPane4);
frame.add(Pbutton);
frame.pack();
frame.setVisible(true);
textArea1.setFont(font);
textArea1.setBackground(JMUgold);
textArea2.setFont(font);
textArea2.setBackground(JMUgold);
textArea3.setFont(font);
textArea3.setBackground(JMUgold);
textArea4.setFont(font);
textArea4.setBackground(JMUgold);
}
答案 0 :(得分：0)
您的代码主要是以程序风格编写的。你应该将它分成小块并将它们放入方法中。
此处的最低要求是将随机生成代码包装在方法中，并将JTextBox es作为参数。
此方法应在其上调用setText而不是创建它们。您应该在开始时创建它们而不使用任何文本，并在类中保留一个私有全局引用。
即使从将UI实例与任何业务逻辑分开的观点来看，这也是有意义的。您现在可以随时安全地调用此方法。
注意：
如果你要重构它，考虑将所有字符串移到源代码之外可能是个好主意。将它们放在xml文件或其他内容中通常被认为是一种很好的做法。
它们不包含逻辑，它们不应该在源头。它也极大地影响了你的代码的可读性（因为你有很多代码）。您甚至可以提高代码的可维护性。
祝你好运。答案 1 :(得分：0)
这是您第一次使用面向对象编程语言吗？虽然您可以在本地存储所有变量，但正如您所尝试的那样，创建对象并将相关信息存储在单独的对象中几乎总是一个更好的主意。此外，正如Tanmay在另一个答案中所提到的，将工作分解为方法会增加可读性和可重用性。
要解决这个问题，我会使用三个类：
1）创建一个类来存储适当的字符串。在这个类中有一个方法（或四个），为每个框返回适当的随机字符串。 （我将这称为BusinessInfo类。）如果你确实选择保持字符串硬编码（正如Tanmay所提到的那样，通常不是一个好主意），那么你可以将它们保留在这里。
2）创建一个覆盖JPanel的类。让这个类存储四个JTextBox个对象。在构造函数中，使用这些JTextBox es来显示信息。添加四种方法来设置JTextBox个对象中的文本。
3）在JFrame类中，在显示框架之前，创建BusinessInfo对象。 （在这里使用JOptionPane并不是一个坏主意。）创建自定义JPanel并调用其四种设置方法。将该面板和包含重置按钮的另一个面板添加到JFrame。添加按下重置按钮时调用的新randomize()方法。让randomize方法在自定义JPanel上调用四组方法。
这是一个关于我如何分解它的一个基本的，袖手旁观的想法。这只是许多可行的一个例子。
希望这个和其他答案能让你朝着正确的方向前进。如果您认为自己已经充分重构了代码，并希望其他人对其进行审核，请尝试使用code review stack exchange site.