We Keep Coding

I am facing problem while translating spanish into english. But when i translate english into spansih it works. Here is my code [closed]

Closed. This question needs debugging details. It is not currently accepting answers.
Edit the question to include desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem. This will help others answer the question.
Closed 3 years ago.
Improve this question
I am trying to create a java application that translates spanish into english. I am facing problem while translating spanish into english. But when i translate english into spansih it works. Here is my code. Here is my code. Can you please tell me my error. This code is working right now but when i change the values of fromLang to toLang from en to es it does not work.
import java.io.BufferedReader;
import java.io.OutputStream;
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.URL;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.Arrays;
public class GuiApp1 {
private static final String CLIENT_ID = "FREE_TRIAL_ACCOUNT";
private static final String CLIENT_SECRET = "PUBLIC_SECRET";
private static final String ENDPOINT = "http://api.whatsmate.net/v1/translation/translate";
public static void main(String[] args) throws Exception {
GuiApp1 g = new GuiApp1();
JFrame f=new JFrame();//creating instance of JFrame
f.setAlwaysOnTop( true );
JButton b=new JButton("Translate");//creating instance of JButton
b.setBounds(90,150,100, 40);//x axis, y axis, width, height
f.add(b);//adding button in JFrame
JTextArea t1,t2;
t1=new JTextArea(2,2);
String spanish;
t1.setBounds(50,100, 200,30);
t2=new JTextArea(2,2);
t2.setBounds(50,200, 200,30);
f.add(t1); f.add(t2);
f.setPreferredSize(new Dimension(200, 900));
f.setLayout(null);//using no layout managers
f.setVisible(true);//making the frame visible
b.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent ae){
String text = t1.getText();
text = text.trim();
text = text.toLowerCase();
System.out.println(text);
String fromLang = "en";
String toLang = "es";
//String text = "Cuál es su nombre";
try{
translate(fromLang, toLang, text);
}catch(Exception e){
System.out.println(e);
}
}
});
}
/**
* Sends out a WhatsApp message via WhatsMate WA Gateway.
*/
public static void translate(String fromLang, String toLang, String text) throws Exception {
// TODO: Should have used a 3rd party library to make a JSON string from an object
String jsonPayload = new StringBuilder()
.append("{")
.append("\"fromLang\":\"")
.append(fromLang)
.append("\",")
.append("\"toLang\":\"")
.append(toLang)
.append("\",")
.append("\"text\":\"")
.append(text)
.append("\"")
.append("}")
.toString();
URL url = new URL(ENDPOINT);
HttpURLConnection conn = (HttpURLConnection) url.openConnection();
conn.setDoOutput(true);
conn.setRequestMethod("POST");
conn.setRequestProperty("X-WM-CLIENT-ID", CLIENT_ID);
conn.setRequestProperty("X-WM-CLIENT-SECRET", CLIENT_SECRET);
conn.setRequestProperty("Content-Type", "application/json");
OutputStream os = conn.getOutputStream();
os.write(jsonPayload.getBytes());
os.flush();
os.close();
int statusCode = conn.getResponseCode();
System.out.println("Status Code: " + statusCode);
BufferedReader br = new BufferedReader(new InputStreamReader(
(statusCode == 200) ? conn.getInputStream() : conn.getErrorStream()
));
String output;
while ((output = br.readLine()) != null) {
System.out.println(output);
}
conn.disconnect();
}
}

Perhaps you've now exceeded the Free Trial usage limit of 10.
In any case, the code does work. It will translate from English to Spanish OR Spanish to English. All you need to do is make sure you have a means to select which translation direction you want to go with. Currently you do not have this and are translating via a default of en to es. If you change the fromLang variable contents to "es" and the toLang variable contents to en and enter a Spanish word into the JTextArea t1 then hit the Translate button you will receive the translation into the Console Window (instead of the JTextArea, t2). The received string should be placed into JTextArea, t2. In order to place the returned string into the JTextArea t2 you will want to make the tranlate() method return a String type then in the while loop at the bottom of the method, instead of sending the returned data to console do something like this:
public static String translate(String fromLang, String toLang, String text) throws Exception {
// ... Most method code here ...
String output;
StringBuilder sb = new StringBuilder();
while ((output = br.readLine()) != null) {
sb.append(output).append(System.lineSeparator());
System.out.println(output); // Optional - For testing
}
conn.disconnect();
return sb.toString();
}
Now, where the translate() method is called, do this instead:
try {
String translation = translate(fromLang, toLang, text);
t2.setText(translation);
}
catch (Exception e) {
System.out.println(e);
}
Now the returned translation will be placed within the t2 JTextArea. I have taken the above mentioned modifications and applied them to your code provided below:
import java.io.BufferedReader;
import java.io.OutputStream;
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.URL;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.border.Border;
public class Translator {
private static final String CLIENT_ID = "FREE_TRIAL_ACCOUNT";
private static final String CLIENT_SECRET = "PUBLIC_SECRET";
private static final String ENDPOINT = "http://api.whatsmate.net/v1/translation/translate";
#SuppressWarnings("Convert2Lambda")
public static void main(String[] args) throws Exception {
Translator g = new Translator();
JFrame f = new JFrame();//creating instance of JFrame
f.setTitle("Translator");
f.setAlwaysOnTop(true);
f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
f.setPreferredSize(new Dimension(295, 300));
f.setLayout(null); //using no layout manager (*** BAD IDEA!! ***)
String[] transDirection = {"English To Spanish", "Spanish To English"};
JComboBox<String> jc = new JComboBox<>(transDirection);
jc.setSelectedIndex(0);
jc.setBounds(60, 30, 160, 25);
f.add(jc);
JButton b = new JButton("Translate");//creating instance of JButton
b.setBounds(90, 140, 100, 30);//x axis, y axis, width, height
f.add(b);//adding button in JFrame
Border border = BorderFactory.createLineBorder(Color.decode("#33acff"));
JTextArea t1 = new JTextArea(2, 2);
t1.setBounds(40, 80, 200, 50);
t1.setBorder(border);
t1.setLineWrap(true);
t1.setWrapStyleWord(true);
JScrollPane sp1 = new JScrollPane(t1);
sp1.setBounds(40, 80, 200, 50);
// ---------------------
JTextArea t2 = new JTextArea(2, 2);
t2.setBounds(40, 180, 200, 50);
t2.setBorder(border);
t2.setLineWrap(true);
t2.setWrapStyleWord(true);
JScrollPane sp2 = new JScrollPane(t2);
sp2.setBounds(40, 180, 200, 50);
f.add(sp1, BorderLayout.CENTER);
f.add(sp2, BorderLayout.CENTER);
f.pack();
f.setVisible(true);//making the frame visible
f.setLocationRelativeTo(null); // Must be after setVisible().
t1.requestFocus(); // Set focus on t1 when started
// Button Action Listener...
b.addActionListener(new ActionListener() {
#Override
public void actionPerformed(ActionEvent ae) {
String text = t1.getText().trim();
//Make sure there is text to translate...
if (text.equals("")) {
JOptionPane.showMessageDialog(f, "There is no text supplied to translate!",
"Translation Error!", JOptionPane.WARNING_MESSAGE);
return; // Get out of this event.
}
// Get selected language translation direction...
String fromLang = "";
String toLang = "";
switch (jc.getSelectedItem().toString().toLowerCase()) {
case "english to spanish":
fromLang = "en";
toLang = "es";
break;
case "spanish to english":
fromLang = "es";
toLang = "en";
break;
default:
fromLang = "en";
toLang = "es";
}
try {
String translation = translate(fromLang, toLang, text);
t2.setText(translation);
}
catch (Exception e) {
System.out.println(e);
}
}
});
}
/**
* Sends out a WhatsApp message via WhatsMate WA Gateway.<br><br>
* #param fromLang
* #param toLang
* #param text
* #return
* #throws java.lang.Exception
*/
public static String translate(String fromLang, String toLang, String text) throws Exception {
// TODO: Should have used a 3rd party library to make a JSON string from an object
String jsonPayload = new StringBuilder()
.append("{")
.append("\"fromLang\":\"")
.append(fromLang)
.append("\",")
.append("\"toLang\":\"")
.append(toLang)
.append("\",")
.append("\"text\":\"")
.append(text)
.append("\"")
.append("}")
.toString();
URL url = new URL(ENDPOINT);
HttpURLConnection conn = (HttpURLConnection) url.openConnection();
conn.setDoOutput(true);
conn.setRequestMethod("POST");
conn.setRequestProperty("X-WM-CLIENT-ID", CLIENT_ID);
conn.setRequestProperty("X-WM-CLIENT-SECRET", CLIENT_SECRET);
conn.setRequestProperty("Content-Type", "application/json");
// 'Try With Resources' used here to auto-close stream.
try (OutputStream os = conn.getOutputStream()) {
os.write(jsonPayload.getBytes());
os.flush();
}
int statusCode = conn.getResponseCode();
System.out.println("Status Code: " + statusCode);
BufferedReader br = new BufferedReader(new InputStreamReader(
(statusCode == 200) ? conn.getInputStream() : conn.getErrorStream()
));
String output;
StringBuilder sb = new StringBuilder();
while ((output = br.readLine()) != null) {
sb.append(output).append(System.lineSeparator());
System.out.println(output);
}
conn.disconnect();
return sb.toString();
}
}
On a side note: You use no layout manager (null) in the creation of your Form by choice. This is actually a bad idea, here's why (be sure to read the comments as well).

Including JCheckBox in JFrame as and when a client connects

I am trying to write a server application in java and creating a vector v1 to store all the sockets that hit the server.
Next i made an arraylist where i stored all my Client sockets as objects and then created a JCheckBox array where I loop it to add checkbox to the JPanel controlpanel which was initiated in the class contructor but something seems to be going wrong and i can't see any checkbox.
import java.util.*;
import java.net.*;
import javax.swing.*;
import java.awt.event.*;
import java.io.*;
import java.awt.*;
class server extends JFrame implements ActionListener {
JTextArea t1;
PrintWriter pw;
JButton b1, b2;
static Vector v1 = new Vector();
static ArrayList checks = new ArrayList();
static JPanel controlpanel;
server() {
setLayout(new FlowLayout());
t1 = new JTextArea();
add(t1);
b1 = new JButton("Send to All");
b2 = new JButton("Send");
//b1.addActionListener(this);
//b2.addActionListener(this);
add(b2, BorderLayout.SOUTH);
add(b1, BorderLayout.NORTH);
b1.addActionListener(this);
b2.addActionListener(this);
controlpanel = new JPanel();
controlpanel.setVisible(true);
add(controlpanel);
}
public void actionPerformed(ActionEvent ae) {
String m = t1.getText();
if (ae.getSource() == b1) {
try {
Iterator t2 = v1.iterator();
while (t2.hasNext()) {
Socket s = (Socket) t2.next();
pw = new PrintWriter(s.getOutputStream(), true);
pw.println(m);
pw.flush();
System.out.println("send" + m);
}
} catch (Exception e) {
}
}
if (ae.getSource() == b2) {
try {
Iterator itr = checks.iterator();
while (itr.hasNext()) {
Socket schecked = (Socket) itr.next();
pw = new PrintWriter(schecked.getOutputStream(), true);
pw.println(m);
pw.flush();
System.out.println("sent " + m);
}
} catch (Exception e) {
}
}
t1.setText("");
}
public static void main(String z[]) {
try {
new server().setVisible(true);
ServerSocket s = new ServerSocket(2000);
while (true) {
System.out.println("waiting...");
Socket c = s.accept();
// System.out.print(c);
v1.add(c);
for (int i = 0; i < v1.size(); i++) {
checks.add(v1.get(i));
// checks[i].setText()=v1[i];
}
JCheckBox checkbox[] = new JCheckBox[checks.size()];
for (int i = 0; i < checks.size(); i++) {
// Object o=checks.get(i);
checkbox[i] = new JCheckBox();
// checkbox[i].setText()=checks.get(i).name;
controlpanel.add(checkbox[i]);
checkbox[i].setSelected(true);
}
}
} catch (Exception e) {
e.printStackTrace();
}
}
}

As you are adding UI elements after the JFrame has been made visible, you need to call revalidate() on the panel. Also you should never add directly from a thread which is not the Event Dispatch Thread. Finally, it appears every time a new client connects, you're going to re-add all the checkbox you already have added, plus one (i.e. when the first client connects you will have 1 checkbox, when the second connects you'll have 1 + 2 = 3, when the third connects you'll have 1 + 2 + 3 = 6, etc.).
I strongly suggests you mentally run through your code, instruction by instruction, to see what happens. Alternatively, if you're using a decent IDE, you can step through the code in the debugger and inspect the content of controlPanel. If you're not using an IDE... you should.
I don't quite understand what you need check[] and v1[] for but assuming that you do need them, instead of maintaining three arrays you can define a class containing a variable for a check, one for a v1, and one for a checkbox, and maintain a single array of instances of such objects.

when calling a jframe class the second time it duplicates it but with no jbuttons,jtextarea,jtextfield etc just the jframe background

I'm currently working on a school project and I'm experiencing a problem that occurs when I call a JFrame class, Whenever I call the simSimi.java in mainMenu.java at first it will call it perfectly without any problem but the second time I call simSimi.java or mainMenu.java it produce a replica of the frame but no buttons, textarea, etc just the frame and the background it would really help me alot if you help me with this occuring problem thanks :)
here's the code for the mainMenu where I call simSimi
//if(e.getSource()==play){
simSimi ss = new simSimi();
ss.setVisible(true);
ss.setSize(420,500);
ss.setLocation(300, 154);
ss.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
ss.setResizable(false);
this.setVisible(false);
dispose();
}
here's the code for the simSimi where i call the mainMenu when exit button is pressed
//if(e.getSource==exitButton){
mainMenu MM = new mainMenu();
MM.setVisible(true);
MM.setSize(719,480);
MM.setLocation(300, 154);
MM.setResizable(false);
MM.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setVisible(false);
this.dispose();
}
here's the simSimi
.
here's the mainMenu
here's the simSimi source code
/*
* To change this license header, choose License Headers in Project Properties.
* To change this template file, choose Tools | Templates
* and open the template in the editor.
*/
package aichat;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JRootPane;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;
import javax.swing.ScrollPaneConstants;
import javax.swing.SwingUtilities;
/**
*
* #author MASTER
*/
public class simSimi extends JFrame implements ActionListener, KeyListener {
static JTextArea chatBox = new JTextArea();
static JTextField msgBox = new JTextField();
static ImageIcon restartIcon = new ImageIcon("restartImage.png");
static Icon sendButtonImage = new ImageIcon("sendImage.png") {};
static Icon teachImage = new ImageIcon("teachImage.gif");
static Icon exitImage = new ImageIcon("exitImage.png");
static JButton sendButton = new JButton(sendButtonImage);
static JButton exitButton = new JButton(exitImage);
static JButton restartButton= new JButton(restartIcon);
static JButton teachButton = new JButton(teachImage);
static Boolean AnswerFound;
static JScrollPane scroll = new JScrollPane(chatBox);
public simSimi() throws IOException{
super("AIBot");
setLayout(null);
JRootPane rootPnae = SwingUtilities.getRootPane(sendButton);
rootPane.setDefaultButton(sendButton);
mainMenu.whoOpenWho=0;
chatBox.setEditable(false);
chatBox.setLineWrap(true);
chatBox.setWrapStyleWord(true);
setContentPane(new JLabel(new ImageIcon
("simSimiBackground.png")));
scroll.setBounds(5,5,320,420);
msgBox.setBounds(5,430,320,30);
teachButton.setBounds(330,5,73,75);
restartButton.setBounds(330,85,73,75);
exitButton.setBounds(330,165,73,75);
sendButton.setBounds(330,430,73,30);
scroll.setVerticalScrollBarPolicy(ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS);
add(exitButton);
add(msgBox);
add(sendButton);
add(scroll);
add(teachButton);
add(restartButton);
msgBox.addKeyListener(this);
sendButton.addActionListener(this);
restartButton.addActionListener(this);
teachButton.addActionListener(this);
exitButton.addActionListener(this);
}
#Override
public void keyTyped(KeyEvent e){
//not used
}
#Override
public void keyPressed(KeyEvent e){
int keyCode = e.getKeyCode();
if(keyCode==KeyEvent.VK_ENTER){
if(e.getSource()==sendButton){
int i=0;
int x=0;
String Question[]= new String[9999999];
String Answer[]=new String[9999999];
String UserMessage;
String UserName="JohnAlbert";
String BotName="Piyaya";
if(msgBox.getText().isEmpty()){
JOptionPane.showMessageDialog(this, "msgBox must not be empty!"
, "WARNING", JOptionPane.WARNING_MESSAGE);
}
else{
UserMessage = msgBox.getText();
msgBox.setText("");
chatBox.setText(chatBox.getText()+UserName+" : "+UserMessage+"\n");
try{//file handling stufff e.g. opening files and assigning some pre defined function
BufferedReader QuestionBR = new BufferedReader(new FileReader("Questions.txt"));
BufferedReader AnswerBR= new BufferedReader(new FileReader("Answer.txt"));
File QuestionFile =new File("Questions.txt");
File AnswerFile = new File("Answer.txt");
FileWriter QfileWritter = new FileWriter(QuestionFile.getName(),true);
BufferedWriter QbufferWritter = new BufferedWriter(QfileWritter);
FileWriter AfileWritter = new FileWriter(AnswerFile.getName(),true);
BufferedWriter AbufferWritter = new BufferedWriter(AfileWritter);
//transfer the question.txt to array
while((Question[i] = QuestionBR.readLine())!=null){
i++;
}
i=0;
//transfer the answer.txt to array
while((Answer[i] = AnswerBR.readLine())!=null){
i++;
}
i=0;
//find if question has a answer
while(Question[x]!=null){
if(Question[x].toLowerCase().equals(UserMessage.toLowerCase())){
AnswerFound=true;
break;
}
else{
AnswerFound=false;
}
x++;
}
if(AnswerFound==true){
chatBox.setText(chatBox.getText()+BotName+" : "+Answer[x]+"\n");
}
else{
chatBox.setText(chatBox.getText()+BotName+" : What?\n");
//simismi will response beyond its queries
if(JOptionPane.showConfirmDialog(this, "Simsimi: I dont know about "+UserMessage+
".\nDo you want me to learn this?", "Teach Me", JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE)==0){
AbufferWritter.write("\r\n"+JOptionPane.showInputDialog(null,"And my answer is..."));
QbufferWritter.write("\r\n"+UserMessage);
QbufferWritter.close();
AbufferWritter.close();
msgBox.setText("");
}
}
//catch all errors
}catch(FileNotFoundException FF){
JOptionPane.showMessageDialog(null,FF);
}catch(IOException IO){
JOptionPane.showMessageDialog(null,IO);
}
}
}// if(e.getSource()==sendButton){
}//if(keyCode==KeyEvent.VK_ENTER){
}// public void keyPressed(KeyEvent e){
#Override
public void keyReleased(KeyEvent txt){
//not used
}
#Override
public void actionPerformed(ActionEvent e){
try{//file handling stufff e.g. opening files and assigning some pre defined function
if(e.getSource()==sendButton){
int i=0;
int x=0;
String Question[]= new String[9999999];
String Answer[]=new String[9999999];
String UserMessage;
String UserName="JohnAlbert";
String BotName="Piyaya";
if(msgBox.getText().isEmpty()){
JOptionPane.showMessageDialog(this, "msgBox must not be empty!"
, "WARNING", JOptionPane.WARNING_MESSAGE);
}
else{
UserMessage = msgBox.getText();
msgBox.setText("");
chatBox.setText(chatBox.getText()+UserName+" : "+UserMessage+"\n");
BufferedReader QuestionBR = new BufferedReader(new FileReader("Questions.txt"));
BufferedReader AnswerBR= new BufferedReader(new FileReader("Answer.txt"));
File QuestionFile =new File("Questions.txt");
File AnswerFile = new File("Answer.txt");
FileWriter QfileWritter = new FileWriter(QuestionFile.getName(),true);
BufferedWriter QbufferWritter = new BufferedWriter(QfileWritter);
FileWriter AfileWritter = new FileWriter(AnswerFile.getName(),true);
BufferedWriter AbufferWritter = new BufferedWriter(AfileWritter);
//transfer the question.txt to array
while((Question[i] = QuestionBR.readLine())!=null){
i++;
}
i=0;
//transfer the answer.txt to array
while((Answer[i] = AnswerBR.readLine())!=null){
i++;
}
i=0;
//find if question has a answer
while(Question[x]!=null){
if(Question[x].toLowerCase().equals(UserMessage.toLowerCase())){
AnswerFound=true;
break;
}
else{
AnswerFound=false;
}
x++;
}
if(AnswerFound==true){
chatBox.setText(chatBox.getText()+BotName+" : "+Answer[x]+"\n");
}
else{
chatBox.setText(chatBox.getText()+BotName+" : What?\n");
//simismi will response beyond its queries
if(JOptionPane.showConfirmDialog(this, "Simsimi: I dont know about "+UserMessage+
".\nDo you want me to learn this?", "Teach Me", JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE)==0){
AbufferWritter.write("\r\n"+JOptionPane.showInputDialog(null,"And my answer is..."));
QbufferWritter.write("\r\n"+UserMessage);
QbufferWritter.close();
AbufferWritter.close();
msgBox.setText("");
}
}
}
}// if(e.getSource()==sendButton){
else if(e.getSource()==restartButton){
chatBox.setText(" ");
}else if(e.getSource()==teachButton){
/*
teach tc = new teach();
tc.setVisible(true);
tc.setSize(325,450);
tc.setLocation(300,154);
tc.setResizable(false);
tc.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
simSimi ss = new simSimi();
ss.setVisible(false);
ss.dispose();
*/
}else if(e.getSource()==exitButton){
mainMenu MM = new mainMenu();
MM.setVisible(true);
MM.setSize(719,480);
MM.setLocation(300, 154);
MM.setResizable(false);
MM.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setVisible(false);
this.dispose();
}
//catch all errors
}catch(FileNotFoundException FF){
JOptionPane.showMessageDialog(null,FF);
}catch(IOException IO){
JOptionPane.showMessageDialog(null,IO);
}
}
public static void main(String args[]) throws IOException{
simSimi ss = new simSimi();
ss.setVisible(true);
ss.setSize(420,500);
ss.setLocation(300, 154);
ss.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
ss.setResizable(false);
}
}
here's the mainMenu sourcecode
/*
* To change this license header, choose License Headers in Project Properties.
* To change this template file, choose Tools | Templates
* and open the template in the editor.
*/
package aichat;
import java.awt.event.*;
import java.io.IOException;
import java.net.MalformedURLException;
import javax.swing.*;
/**
*
* #author MASTER
*/
public class mainMenu extends JFrame implements ActionListener {
static JButton play = new JButton("Play");
static JButton settings = new JButton("Settings");
static JButton teach = new JButton("Teach");
static JButton exit = new JButton("Exit");
static int whoOpenWho=1;
public mainMenu()throws MalformedURLException{
super("Main Menu");
setLayout(null);
setContentPane(new JLabel(new ImageIcon
("mainMenuBackground.jpg")));
play.setBounds(200,150,100,50);
teach.setBounds(200,220,100,50);
settings.setBounds(400,150,100,50);
exit.setBounds(400,220,100,50);
add(play);
add(settings);
add(teach);
add(exit);
play.addActionListener(this);
teach.addActionListener(this);
settings.addActionListener(this);
exit.addActionListener(this);
}
#Override
public void actionPerformed(ActionEvent e) {
try{
if(e.getSource()==play){
simSimi ss = new simSimi();
ss.setVisible(true);
ss.setSize(420,500);
ss.setLocation(300, 154);
ss.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
ss.setResizable(false);
this.setVisible(false);
dispose();
}
else if(e.getSource()==settings){
}
else if(e.getSource()==teach){
/*
teach tc = new teach();
tc.setVisible(true);
tc.setSize(325,450);
tc.setLocation(300,154);
tc.setResizable(false);
tc.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setVisible(false);
dispose();
*/
}
else if(e.getSource()==exit){
System.exit(0);
}
}catch(IOException IO){
JOptionPane.showMessageDialog(null,IO);
}
}
public static void main(String args[])throws MalformedURLException{
mainMenu MM = new mainMenu();
MM.setVisible(true);
MM.setSize(719,480);
MM.setLocation(300, 154);
MM.setResizable(false);
MM.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
}

The first thing you need to do is stop making everything static. Get rid of all the static keywords that you can in any class that extends JFrame.

JTextArea saving to txt

I'm making a program that requires to save user input. So I would like to know how to save JTextArea to text file and when you close and re-open program text is still in JTextArea.
Also sorry for my bad grammar.
package main;
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;
import javax.swing.border.EtchedBorder;
public class main extends JFrame {
JLabel statusbar;
public main() {
initUI();
}
public final void initUI() {
JPanel panel = new JPanel();
statusbar = new JLabel("");
statusbar.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.RAISED));
panel.setLayout(null);
JTextArea area1;
area1 = new JTextArea(90, 25);
area1.setBounds(20, 20, 200, 25);
area1.setBackground(Color.white);
area1.setForeground(Color.BLACK);
area1.setText("");
panel.add(area1);
add(panel);
add(statusbar, BorderLayout.SOUTH);
setTitle("Viskis");
setSize(300, 200);
setLocationRelativeTo(null);
setDefaultCloseOperation(EXIT_ON_CLOSE);
}
class ButtonListener implements ActionListener {
public void actionPerformed(ActionEvent e) {
JButton o = (JButton) e.getSource();
String label = o.getText();
statusbar.setText("");
} }
public static void main(String[] args) {
SwingUtilities.invokeLater(new Runnable() {
public void run() {
main ms = new main();
ms.setVisible(true);
new main();
}
});
}
}

Here are some helper methods to help you read and write from Files. Look in the JavaDoc for getText() and setText() for getting and setting the text of the JTextArea.
I recommend reading the SwingWorker tutorials and using these methods in a SwingWorker, lest you lock up your GUI while saving/reading the Files
/**
* Writes a String text to a File specified by filename
*
* #param filename The filename/path of the File we would like to write the text to
* #param text The text to write to the File
*
*/
public static void writeToFile(String filename, String text) {
BufferedWriter writer = null; // This could go in a try-with-resources if you wanna get fancy
try {
writer = new BufferedWriter(new FileWriter(new File(filename))); // Open a File for writing
writer.write(text); // write the text to the file
} catch ( IOException e) {
/* We could not open the File for writing, or could not write to the File */
} finally {
try {
if (writer != null) {
writer.close(); // we are done writing to the File, close the connection
}
} catch (IOException e) {
/* We could not close the connection to the File */
}
}
}
/**
* Reads all lines from a File specified by filename
*
* #param filename The filename/path of the File we would like to read text from
*
* #return An list of Strings containing each line of the File
*/
public static List<String> readFromFile(String filename) {
BufferedReader reader = null; // This could go in a try-with-resources if you wanna get fancy
List<String> lines = new ArrayList<String>();
try {
reader = new BufferedReader(new FileReader(new File(filename))); // Open a File for writing
while ((line = reader.readLine()) != null) {
lines.add(line);
}
} catch (IOException e) {
/* We could not open the File for reading, or could not read from the File */
} finally {
try {
if (reader != null) {
reader.close(); // we are done reading from the File, close the connection
}
} catch (IOException e) {
/* We could not close the connection to the File */
}
}
return lines;
}

Write and read file on a swing thread is not recommanded. Use mvc pattern. Create a data model with field binded to your component. Create an adapter to update model and write file
Then your component will always up to date with your model
Use model to write data(with the adapter) and read file will update your model with the adapter
For exemple, Focus listener will call adapter to do the task. Or you can use an obseevee pattern when you Object if modified.

This code worked for me with 'nam' as the current date and 'name' an input in a jTextField.
try {
con=Connect.ConnectDB();
pst.execute();
Date date=new Date();
SimpleDateFormat sd=new SimpleDateFormat("dd-mm-yyyy-h-m-s");
String nam= sd.format(date);
String name=CaseNoField.getText();
CaseNoField.setText("");
FileWriter writer=new FileWriter( "C:\\path"+name+"("+nam+")"+".txt");
BufferedWriter bw=new BufferedWriter( writer );
JCaseArea.write( bw );
bw.close();
JCaseArea.setText("");
JCaseArea.requestFocus();
JOptionPane.showMessageDialog(null, "Case Saved");
}
catch(Exception e){
JOptionPane.showMessageDialog(null, e);
}

JLabel's text is not changing

EDIT: Huge code re-organisation, old question here: http://pastebin.com/Mbg4dYiY
I have created a basic program that is designed to show the weather in a window using Swing. I am developing using IntelliJ and I have used the UI builder in that. I am attempting to fetch some information from the Weather Underground servers and then make a JLabel called weatherlabel display this information. However, the JLabel doesn't actually change in the window; it just stays as 'Weather Will Go Here'. How do I fix this?
Here is my main.java:
public class main {
public static void main(String[] args) {
System.out.println("Hello World!");
Display d = new Display();
d.getandsetWeather();
}
}
Here is my Display.java:
public class Display {
Display disp = this;
public JPanel myPanel;
public JLabel weatherfield;
private JButton button1;
public void init() {
JFrame frame = new JFrame("Display");
frame.setContentPane(new Display().myPanel);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setMinimumSize(new Dimension(480, 234));
frame.pack();
frame.setVisible(true);
}
public void getandsetWeather() {
String editedline = null;
init();
try {
// Construct data
String data = URLEncoder.encode("key1", "UTF-8") + "=" + URLEncoder.encode("value1", "UTF-8");
data += "&" + URLEncoder.encode("key2", "UTF-8") + "=" + URLEncoder.encode("value2", "UTF-8");
// Send data
URL url = new URL("http://api.wunderground.com/api/772a9f2cf6a12db3/geolookup/conditions/q/UK/Chester.json");
URLConnection conn = url.openConnection();
conn.setDoOutput(true);
OutputStreamWriter wr = new OutputStreamWriter(conn.getOutputStream());
wr.write(data);
wr.flush();
// Get the response
BufferedReader rd = new BufferedReader(new InputStreamReader(conn.getInputStream()));
String line;
while ((line = rd.readLine()) != null) {
if( line.contains("\"weather\":")) {
System.out.println(line);
editedline = line.replace("\"weather\":\"", "");
editedline = editedline.replace("\",", "");
System.out.println(editedline);
weatherfield.setText(editedline);
}
}
wr.close();
rd.close();
weatherfield.setText(editedline);
System.out.println(weatherfield.getText());
weatherfield.repaint();
weatherfield.revalidate();
} catch (Exception e) {
System.out.println("Error!" + e);
}
}
}
When I run the program, this is printed to the log:
Hello World!
"weather":"Scattered Clouds",
Scattered Clouds
Scattered Clouds

You seem to have a strange understanding of OOP and code-flow
You have to main methods, one of which you are trying to call the other main method. Don't do that. A program should only have one main method. You should never have to this
Display.main(new String[]{});
Why even have this ChangeWeatherLabelText class? There is only one method, that doesn't seem to be needed in it's own class. Your instantiation if Display in that method does nothing to the rest of the program. So you call has no effect on the label.
Instead of 3, put that method in the class that actually has the label and just reference the label field in the method.
Also, GetWeather just seems like a helper class with a helper method. "Helper" class methods are useless if they don't return something.
IMHO, you should restructure your entire program. Some things may work now, but there's a lot of bad practice going on in your code
If I were to write this program, all the code would be in one file. If you insist on them being in separate files, you need to learn how to use constructors and how to pass objects to them. That is how you are going to manipulate objects from other classes. That is unless you understand the MVC model, which may be a little advance at this point for you.
UPDATE with OP update of code
Test this out and be sure to read the comments so you can see what I did. Let me know if you have any question.
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.OutputStreamWriter;
import java.net.URL;
import java.net.URLConnection;
import java.net.URLEncoder;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
public class main {
public static void main(String[] args) {
System.out.println("Hello World!");
new Display(); // <-- just instantiate
}
}
class Display {
Display disp = this;
public JPanel myPanel; // <--------- Haven't been initialized
public JLabel weatherfield;
private JButton button1;
public Display() { // you need constructor to call init
init();
}
public void init() {
myPanel = new JPanel(new BorderLayout()); // initialize
weatherfield = new JLabel(" "); // initialize
button1 = new JButton("Button"); // initialize
myPanel.add(weatherfield, BorderLayout.CENTER);
myPanel.add(button1, BorderLayout.SOUTH);
button1.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent e) {
getandsetWeather(); // <-------- add listener to call getandsetweather
}
});
JFrame frame = new JFrame("Display");
frame.setContentPane(myPanel); // <--------------------- fix 1
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setMinimumSize(new Dimension(480, 234));
frame.pack();
frame.setVisible(true);
}
public void getandsetWeather() {
String editedline = null;
init();
try {
// Construct data
String data = URLEncoder.encode("key1", "UTF-8") + "=" + URLEncoder.encode("value1", "UTF-8");
data += "&" + URLEncoder.encode("key2", "UTF-8") + "=" + URLEncoder.encode("value2", "UTF-8");
// Send data
URL url = new URL("http://api.wunderground.com/api/772a9f2cf6a12db3/geolookup/conditions/q/UK/Chester.json");
URLConnection conn = url.openConnection();
conn.setDoOutput(true);
OutputStreamWriter wr = new OutputStreamWriter(conn.getOutputStream());
wr.write(data);
wr.flush();
// Get the response
BufferedReader rd = new BufferedReader(new InputStreamReader(conn.getInputStream()));
String line;
while ((line = rd.readLine()) != null) {
if( line.contains("\"weather\":")) {
System.out.println(line);
editedline = line.replace("\"weather\":\"", "");
editedline = editedline.replace("\",", "");
System.out.println(editedline);
weatherfield.setText(editedline);
}
}
wr.close();
rd.close();
weatherfield.setText(editedline);
System.out.println(weatherfield.getText());
weatherfield.repaint();
weatherfield.revalidate();
} catch (Exception e) {
System.out.println("Error!" + e);
}
}
}