Help with reprinting downloaded text from FTP site onto GUI!

[Big Girl]

*This has already been posted on the Sun Java Forum, but I need help quickly, so I am cross-posting. Thank you!*

Basically, I have all of the data download from the FTP site but I can't get it to be reprinted to the GUI after the GUI is already opened initially. I'm pretty sure I have to make a new method at the very bottom of the GUIWindow.java class but I don't know what goes there. If anyone needs to look at the other classes, let me know. By the way, I'm handing this in for my final project in Computer Science tomorrow, so I need help quick! Any help is greatly appreciated! Thanks!

Scroll down for code...

[pete stein]

Cross-posted on the Sun Java Forum. To the OP, please have a look here: Be Forthright When Cross Posting To Other Sites

This is also general policy at the Sun fora and just good net etiquette.

Good luck!

/Pete

[Big Girl]

Sorry, I didn't realize that...I will edit the first post to reflect that. Thank you!

[pete stein]

The mods will also send you a private message about your screen name, but you can beat them to their game by reading the other FAQs here.

A question I have for you is this: have you sprinkled your code with println statements to see what the state of variables are in various locations? Either that or have you run it with a debugger? This can solve your whole problem, or at the very least show you exactly where your error is located.

For instance,

  public void word() {
    Writer output = new PrintWriter(System.out, true);
    DecodeMETAR formatter = new DecodeMETAR(output);

    try {
      System.out.println("createMETAR == null: " + (createMETAR == null));
      String[] words = createMETAR.split("\\s+");
      // ... code deleted

    } catch (IOException e) {
      System.out.println("Something went wrong with the formatting.");
    }
  }

and

public class CreateMETAR {

  private String createMETAR;

  public CreateMETAR(String createMETAR) {
    this.createMETAR = createMETAR;
    System.out.println("createMETAR: " +createMETAR);
    word();
  }

  public void word() {
    Writer output = new PrintWriter(System.out, true);
    DecodeMETAR formatter = new DecodeMETAR(output);

    try {
      System.out.println("createMETAR == null: " + (createMETAR == null));
      String[] words = createMETAR.split("\\s+");
      for (int i = 1; i < words.length; i++) {
	String word = words[i];
	// .... code deleted

[Big Girl]

Ok, I narrowed down what I could, without taking too much out. My problem really only involves these two classes...I'm pretty sure it's a simple problem (since it's something I have missing, not something I need to fix) but I honestly don't know what to do. And I've only put a few println statements in there, but like I said, it's just something that needs to be added. By the way, to see if you did fix it, type in 'KLGA' (without the quotes of course) into the blank box before you hit convert. Thanks for any help!

DownloadMETAR.java:

import java.io.*;
import java.net.*;

public class DownloadMETAR {
    
    public String allTheText;
    public StringBuffer buff;

    public DownloadMETAR(String temp)
    {
        getHTML(temp);
        CreateMETAR formatter = new CreateMETAR(allTheText);
    }

    public void getHTML(String temp)
    {
        URL metarURL = null;
  
        try
        {
            metarURL = new URL("ftp://tgftp.nws.noaa.gov/data/observations/metar/stations/" + temp + ".TXT");
        }
            catch(MalformedURLException e)
            {
                System.out.println("Error in given URL");
                return;
            }
        
        try
        {
            URLConnection connection = metarURL.openConnection();
            BufferedReader br = new BufferedReader(new InputStreamReader(connection.getInputStream()));
            
            buff = new StringBuffer();
            
            String line = "";
            
            while ((line = br.readLine()) != null)
                     buff.append(line);
                     br.close();

                allTheText = buff.toString();
        }
            catch(UnknownHostException e)
            {
                System.out.println("Unknown Host");
                return;
            }
            catch(IOException e)
            {
                System.out.println("Error in opening URLConnection, Reading or Writing");
                return;
            }
    }
}

GUIWindow.java:

import javax.swing.*;
import javax.swing.text.JTextComponent;
import java.awt.*;
import java.awt.event.*;
import java.lang.reflect.InvocationTargetException;

public class GUIWindow{
    
    public static JTextArea tempCelsiuscopy = new JTextArea();
    private JPanel panelOne = new JPanel();
    private JPanel panelTwo = new JPanel();
    private JPanel introPanel = new JPanel();
    private JButton convertTemp;
    private JButton introButton;
    private JTextField tempCelsius;
    private JFrame intro;
    private JFrame conversion;
    private DownloadMETAR METARLoader;
    
    public static void main(String[] args)
    {
        JFrame.isDefaultLookAndFeelDecorated();
        GUIWindow conversionFinal = new GUIWindow();
    }
    
    private void setTempCelsiusCopyText(String text)
    {
        tempCelsiuscopy.setText(text);
    }
    
    private void setTempCelsiusCopyVisibility(boolean visibility)
    {
        tempCelsiuscopy.setVisible(visibility);
    }
    
    public GUIWindow()
    {
        conversion = new JFrame();
        conversion.setTitle("METAR Conversion Program");
        conversion.setSize(600, 250);
        conversion.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        
        JPanel panelOne = new JPanel();
        panelOne.setBackground(Color.white);
        JPanel panelTwo = new JPanel();
        panelTwo.setBackground(Color.LIGHT_GRAY);
        
        conversion.getRootPane().setDefaultButton(convertTemp);
        
        Container pane1 = conversion.getContentPane();
        
        panelOne.setSize(600, 75);
        panelTwo.setSize(600, 180);
        
        //add buttons, text, etc into GUI
        
        tempCelsius = new JTextField(15);
        JTextArea tempCelsiuscopy = new JTextArea(4,60);
        JLabel celsiusLabel = new JLabel("Metar:");
        JButton convertTemp = new JButton("Convert");
        
        convertTemp.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                System.out.println("actionPerformed");
                downloadMETAR();
            }
        });
        
        pane1.add(panelOne);
        pane1.add(panelTwo);
        
        panelOne.add(celsiusLabel);
        panelOne.add(tempCelsius);
        panelOne.add(convertTemp);
        panelTwo.add(tempCelsiuscopy);
        tempCelsiuscopy.setVisible(true);

        celsiusLabel.setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
        
        conversion.setVisible(false);    
    }
    
    public void downloadMETAR()
    {
        String temp = tempCelsius.getText();
        setTempCelsiusCopyText("");                    
      
        System.out.println("instantiating METARLoader");
        METARLoader = new DownloadMETAR(temp);
        System.out.println("done instantiating METARLoader");
        setTempCelsiusCopyVisibility(true);
        conversion.setVisible(true);
    }
    
    public void updateMETAR()
    {
    	//what goes here?
    }
}

CreateMETAR.java:

import java.io.*;
import javax.swing.*;

public class CreateMETAR {

    private String createMETAR;

    public CreateMETAR(String createMETAR)
    {
        this.createMETAR = createMETAR;
        word();
    }

    public void word()
    {
         Writer output = new PrintWriter(System.out, true);
         DecodeMETAR formatter = new DecodeMETAR(output);
        
        try
        {
                    String[] words = createMETAR.split("\\s+");
                    for(int i = 1; i < words.length; i++)
                    {
                        String word = words[i];
                        
                        formatter.addWord(word);                  
              
                     }
          
            formatter.flushNoFill();
      
        }
        catch(IOException e){
            System.out.println("Something went wrong with the formatting.");
        }        
    }    
}

DecodeMETAR.java:

import java.io.IOException;
import java.io.Writer;
import java.util.ArrayList;
import java.util.Calendar;
import javax.swing.JTextArea;

public class DecodeMETAR
{
    private Writer destination;
    private ArrayList line;
    private ArrayList lines;
    private JTextArea file;
    
    public DecodeMETAR(Writer destination)
    {                  
        this.destination = destination;
        line = new ArrayList();
        lines = new ArrayList();
        file = GUIWindow.tempCelsiuscopy;
    }

    public void addWord(String word) throws IOException
    {
        line.add(word);
        lines.add(word);
    }
    
    public void finalm() throws IOException
    {        
        file.append("\n");  
        file.append("\n");  
        file.append("Test Metar: ");  
        
        int x = 1;
        while(x < lines.size())
        {      
          String weather = (String) lines.get(x);
          
          file.append(weather);  
          file.append("");
          x++;            
        }
    }
    
    public void airport() throws IOException
    {
          String weather = (String) line.remove(0);
          String temp = weather.substring(5, 9);
          
          file.append(temp);
          file.append(" Airport");
          file.append("\n");
          file.append("Last Weather Observation Taken");
          file.append("");        
    }
    
    public void time() throws IOException
    {
          String weather = (String) line.remove(0);
          String temp = weather.substring(0, 2);
          
          file.append(temp);
          file.append("");
          file.append(Month.getMonth());
          file.append("");
          temp = weather.substring(2, 7);
          file.append(temp);
          file.append("\n");        
    }
    
    public void winds() throws IOException
    {
          String weather = (String) line.remove(0);
          String temp = weather.substring(0, 3);
          
          file.append("Winds from ");
          file.append(temp);
          file.append(" degrees at ");
          temp = weather.substring(3, 5);
          temp = remove(temp);
          file.append(temp);
          file.append("kts ");
            
             if(weather.startsWith("G", 5))
             {
                 temp = weather.substring(6, 8);
                 file.append("Gusting ");
                 file.append(temp);
                 file.append("kts ");
             }
    }
    
    public void metar() throws IOException
    {
        airport();
        time();
        winds();        
        
            if(checkCAVOK() == true)  // CAVOK = Ceiling and Visibility OK
            {
                variableWinds();
                visibility();
                temp();
                baroPressure();
                clouds();
            }
                      else
                      {
                          variableWinds();
                          file.append("\n");
                          file.append("9999 Metres Visibilty");
                          file.append("\n");
                          temp();
                          baroPressure();
                          file.append("Sky conditions: Clear ");
                    }
        finalm();        
    }
    
    public void variableWinds() throws IOException
    {
           String weather = (String) line.get(0);
          
           if(weather.startsWith("V", 3))
           {
             String temp = weather.substring(0,3);
            
             file.append("Variable from ");
             file.append(temp);
             file.append(" to ");
             temp = weather.substring(4, 7);
             file.append(temp);
             file.append(" degrees");
             line.remove(0);
           }
    }
    
    public void visibility() throws IOException
    {
          file.append("\n");
          String weather = (String) line.remove(0);
          String temp = weather;
          
          file.append(temp);
          file.append(" Metres Visibilty");
          file.append("\n");
    }

    public void temp() throws IOException
    {
          int testing = checks();
          
          String weather = (String) line.remove(testing);
          String temp = weather.substring(0, 2);
          
          file.append("Temperature: ");
          temp = remove(temp);
          file.append(temp);
          temp = weather.substring(3, 5);
          file.append("c Dew Point: ");
          temp = remove(temp);
          file.append(temp + "c");    
     }

    public void baroPressure() throws IOException
    {
          int testing = check();
          
          String weather = (String) line.remove(testing);
          String temp = weather.substring(1, 4);
          
          if(weather.length() > 4)
          {
              temp = weather.substring(1, 5);
          }
          
          file.append(" Barometer: ");
          temp = remove(temp);
          file.append(temp);
          file.append("mb");
          file.append("\n");      
    }

    public void clouds() throws IOException
    {
          int x = 0;
          
          file.append("Sky conditions: ");
          
          while(x < line.size())
          {
              String weather = (String) line.get(x);
              
                  if(weather.startsWith("B"))
                  {
                      String temp = weather.substring(3, 6);
                      
                      temp = temp + "00";
                      file.append("Broken at ");
                      temp = remove(temp);
                      file.append(temp + "ft" + " ");
                  }
              
                  if(weather.startsWith("S"))
                  {
                      String temp = weather.substring(3, 6);
                      
                      temp = temp + "00";
                    file.append("Scatered at ");
                    temp = remove(temp);
                    file.append(temp + "ft" + " ");
                  }
              
                  if(weather.startsWith("O"))
                  {
                      String temp = weather.substring(3, 6);
                      
                    temp = temp + "00";
                    file.append("Overcast at ");
                    temp = remove(temp);
                    file.append(temp + "ft" + " ");
                  }
              
                  if(weather.startsWith("F"))
                  {
                      String temp = weather.substring(3, 6);
                      
                    temp = temp + "00";
                    file.append("Few at ");
                    temp = remove(temp);
                    file.append(temp + "ft" + " ");
                  }
                  
                  x++;
           }
    }
    
     public String remove(String temp)
     {
         while(temp.startsWith("0"))
         {
             temp = temp.substring(1);
         }
        
         return temp;
      }

    public boolean checkCAVOK()
    {
         int x = 0;
          
          while(x < line.size())
          {
              String weather = (String) line.get(x);
              
              if(weather.equals("CAVOK"))
              {
                  return false;
              }
              
              x++;
           }
            
          return true;        
    }

        public int check()
        {
            int x = 0;
            
            while(x < line.size())
            {
                String weather = (String) line.get(x);
                
                if(weather.startsWith("Q"))
                {
                    return x;
                }
                
             x++;
            }
            
            return 0;
        }
        
        public int checks()
        {
            int x = 0;
            
            while(x < line.size())
            {
                String weather = (String) line.get(x);
                
                if(weather.startsWith("/", 2))
                {
                    return x;
                }
                
             x++;  
            }
            
            return 0;
        }
        
    public void flushNoFill() throws IOException
    {
        metar();
        destination.flush();
    }
}

Edit: I just realized that the CreateMETAR.java class and DecodeMETAR.java class are both necessary to compile...I will post them, but you don't need to look in either class since I know for a fact that the problem isn't cause be either class.

[pete stein]

Please see the changes made to my post above. Also, your code does not compile. Non-compiling code without comments is extremely hard if not impossible to debug. You really should consider creating an SSCCE as recommended in your cross-post.

[Maneesh Godbole]

pete stein wrote:The mods will also send you a private message about your screen name, but you can beat them to their game by reading the other FAQs here.

"Big Girl",
Please check your private messages for an important administrative matter.