Thread: Tower Defense on Java

then your code here is wrong if a JPanel triggered the event, not a JLabel

I forgot to cast the Component to a JLabel. Nevertheless, I do not intend to apply this MouseListener to a JPanel, so there is not an issue with that.

After some exploring, I found out how it edits all references of the same object.

MouseEvent extends AWTEvent which extends EventObject. EventObject has a protected transient Object source. I assume that transient means that when you edit one reference to an object, all of its references will be updated.

Thanks for your help; I'm going offline for now.

Okay. Now I'm working on saving/deleting files and the GUI's associated with the process. I have a pretty awesome load game GUI (at least I think) and now I'm working on the delete button. The only problem is that sometimes files won't delete.

This is what the load GUI currently looks like. The mouse is invisible, but it is hovering over the "Delete" JLabel.
LoadGameGUI-1.jpg

This is the code I'm currently using for deleting a directory that represents a profile.

   private static void deleteFile(String path)
   {
 
      File file = new File(path);
 
      //If the file is a directory, list all of its files, then use this
      //method to delete them one by one
      if(file.isDirectory())
      {
 
         String[] files = file.list();
 
         for(int a = 0; a < files.length; a++)
         {
 
            deleteFile(path + "/" + files[a]);
 
         }
      }
 
      //If the file is a file, delete it, and assign
      //its return boolean to a boolean called 'success'
      boolean success = file.delete();
 
      //This is for debugging; I use it to tell me why the
      //file couldn't be deleted.
      JOptionPane.showMessageDialog(null, "Deletion of \n" + file.getAbsolutePath() + "\n"
		   + (success ? "Success!" : "Failed: " + (!file.canWrite() ? "Cannot write" : "Other issue")));
 
   }

Often, the files will fail to be deleted, and my debugging helper (the JOptionPane dialog) will tell me "other issue" (see code). Then, one minute and ten tries later, it will suddenly work... Is there a way to make it work the first time, hands down? I've tried using a loop that runs until the deletion succeeds, but then I just get a StackOverflowError...

Are the files being used by other versions of your program?

Well, it's complicated to explain, but I think it may be the GUI itself that is using the files. The GUI I picture above creates the panels inside of it by this code:

   private static String[] getProfileNames()
   {
 
      File profileDirectory = new File(getAppDataDirectory() + "/.WeatherTowerDefense/profiles");
 
      return profileDirectory.list();
 
   }
 
   private static int numPages;
   private static int pageNum = 1;
   private static String[][] pages = organizeProfileNames();
 
   private static void refreshProfileList()
   {
 
      pages = organizeProfileNames();
 
   }
 
   private static String[][] organizeProfileNames()
   {
 
      String[] names = getProfileNames();
 
      int numOfPages = ((names.length - 1) / 5) + 1;
      numPages = numOfPages;
      int numOnLastPage = names.length - ((numOfPages - 1) * 5);
      String[][] tempPages = new String[numOfPages][];
 
      for(int a = 0; a < numOfPages; a++)
      {
 
         tempPages[a] = new String[(a == numOfPages - 1 ? numOnLastPage : 5)];
 
      }
 
      int namesIndex = 0;
 
      for(int a = 0; a < numOfPages; a++)
      {
 
         for(int b = 0; b < (a == numOfPages - 1 ? numOnLastPage : 5); b++)
         {
 
 
            tempPages[a][b] = names[namesIndex];
            namesIndex += 1;
 
         }
      }
 
      return tempPages;
   }

Basically what it does is it gets all of the known profiles that are store in the "/.WeatherTowerDefense/profiles" folder, which is located in the user's Application Data folder (or a similar folder). After all of the profile folders are organized into pages (obviously I can't just keep piling up panels; the window would get too tall), this code creates the panels pictured in the GUI:

      for(int a = 0; a < pages[pageNum - 1].length; a++)
      {
 
         add(new LoadableGamePanel(pages[pageNum - 1][a]));
 
      }

So maybe the program can't delete the files because it thinks it's using them, even though they are no longer necessary for it to function?

However, this is weirder: the deletion attempts will randomly work, after failing a plethora of times... I'm so confused...

You need to close the file when the code is finished using it.

...THANK YOU!!!!! It works GREAT!

It was as simple as adding this:

fr.close();
br.close();

where fr is a FileReader, and br is a BufferedReader!

Thank you so much! And now... my next problem... (get your facepalms out of the way now...)

....and I'm on a roll now. I'll tell you if I get another problem, but I'm working on renaming right now.

Thanks so much, Norm.

YESSSSSS!

I got the rename function to work! Now, I can start working on the actual game GUI! And then, it's the end of the easy stuff, for I have to start thinking about complex tower, bullet, and enemy interactions, along with the correct enemy path, status defects, etc... The task is daunting...

But, I think I'll rest for tonight, to gather my thoughts. Thanks again for your help, Norm!

Alright I have a new problem, this time with a KeyListener.

Here is the code I'm using for the KeyListener:

   //KeyListener for NewGameGUI
   private class KListener implements KeyListener
   {
 
      public void keyTyped(KeyEvent e)
      {
 
         if(e.getKeyCode() == KeyEvent.VK_ESCAPE)
         {
 
            dispose();
            new MainGUI();
 
         }
         else if(e.getKeyCode() == KeyEvent.VK_ENTER)
         {
 
            if(validName)
            {
 
               createNewGame(nameField.getText());
               dispose();
               new MainGUI();
 
            }
            else
            {
 
               createLabel.setForeground(textColor_inactive);
               createLabel.setBackground(backgroundColor_inactive);
               createLabel.setBorder(border_inactive);
               createLabel.setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
 
            }
         }
      }
 
      public void keyPressed(KeyEvent e)
      {
 
      }
 
      public void keyReleased(KeyEvent e)
      {
 
      }
   }

And then this code adds it to a JTextField:

   //Add the KeyListener to only the nameField (input field for file name)
   //because adding it to the JFrame didn't work
   nameField.addKeyListener(new KListener());

When I type enter or escape, nothing happens. I have also tried this code combination:

   //KeyListener for NewGameGUI
   private class KListener implements KeyListener
   {
 
      public void keyTyped(KeyEvent e)
      {
 
      }
 
      public void keyPressed(KeyEvent e)
      {
 
      }
 
      public void keyReleased(KeyEvent e)
      {
 
         if(e.getKeyCode() == KeyEvent.VK_ESCAPE)
         {
 
            dispose();
            new MainGUI();
 
         }
         else if(e.getKeyCode() == KeyEvent.VK_ENTER)
         {
 
            if(validName)
            {
 
               createNewGame(nameField.getText());
               dispose();
               new MainGUI();
 
            }
            else
            {
 
               createLabel.setForeground(textColor_inactive);
               createLabel.setBackground(backgroundColor_inactive);
               createLabel.setBorder(border_inactive);
               createLabel.setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
 
            }
         }
      }
   }

   //Add the KeyListener to the entire JFrame
   addKeyListener(new KListener());

I'm pretty sure this issue has something to do with the JTextField somehow blocking the event, and if that is the problem I don't know how to bypass it. And then it could also be something totally different as well. Any help would be appreciated.

Your code doesn't have an printlns to show you what is happening.

printlns? Why would it have those? None of my program is done in the terminal.

Please make a small simple complete program (SSCCE) that compiles and executes and shows your problem and post it here.

I don't completely understand... Do you want me just to copy the code for the NewGameGUI (from which the above snippets were collected) here?

Originally Posted by snowguy13

printlns? Why would it have those? None of my program is done in the terminal.


I don't completely understand... Do you want me just to copy the code for the NewGameGUI (from which the above snippets were collected) here?

Suggested Reading:
http://www.javaprogrammingforums.com...t-println.html
http://www.javaprogrammingforums.com...why-sscce.html

I have to go now... Sorry for the abruptness. I'll work on that code and put it here when I finish.

Okay. Here's the SSCCE I made.

package GUIs;
 
import java.awt.GridBagLayout;
import java.awt.GridBagConstraints;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.JLabel;
import java.awt.event.KeyListener;
import java.awt.event.KeyEvent;
import javax.swing.SwingConstants;
import javax.swing.JOptionPane;
 
public class KLSSCCE extends JFrame
{
 
   public static void main(String[] args)
   {
 
      new KLSSCCE();
 
   }
 
   //Create final integers to represent the window's size
   private final int WINDOW_WIDTH = 300;
   private final int WINDOW_HEIGHT = 200;
 
   public KLSSCCE()
   {
 
      super("Key Listener SSCCE");
      JOptionPane.showMessageDialog(
              null, 
              "This is a simplified example of my problem.\n"
              + "Pressing Enter or Escape should terminate this program.\n"
              + "Neither works. I believe the JTextField is the culprit.");
      System.out.print("Initializing frame and size...");
      setSize(WINDOW_WIDTH, WINDOW_HEIGHT);
      setResizable(false);
      setLocationRelativeTo(null);
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      System.out.print(" Done.\n");
      addPanel();
      System.out.print("Adding key listener..."); 
      textField.addKeyListener(new KListener());
      System.out.print(" Done.\n");
      System.out.print("Displaying window...");
      setVisible(true);
      System.out.print(" Done.\n");
 
   }
 
   private JPanel panel;
   private JTextField textField;
   private JLabel infoLabel;
 
   private void addPanel()
   {
 
      System.out.print("Building panel and text field...");
      panel = new JPanel(new GridBagLayout());
      GridBagConstraints c = new GridBagConstraints();
      c.gridx = 1;
      c.gridy = 1;
      c.weightx = 0.5;
      c.weighty = 0.5;
      c.ipadx = 100;
      c.ipady = 10;
      textField = new JTextField(WINDOW_WIDTH - 10);
      panel.add(textField, c);
      infoLabel = new JLabel("Pressing Enter or Escape should exit the program.", SwingConstants.CENTER);
      c.gridy = 0;
      panel.add(infoLabel, c);
      add(panel);
      System.out.print(" Done.\n");
 
   }
 
   private class KListener implements KeyListener
   {
 
      public void keyTyped(KeyEvent e)
      {
 
         if(e.getKeyCode() == KeyEvent.VK_ESCAPE)
         {
 
            System.out.print("Escape key pressed. Disposing of window...");
            dispose();
            System.out.print(" Done.\nExiting program...");
            System.exit(0);
 
         }
         else if(e.getKeyCode() == KeyEvent.VK_ENTER)
         {
 
            System.out.print("Enter key pressed. Disposing of window...");
            dispose();
            System.out.print(" Done.\nExiting program...");
            System.exit(0);
 
         }
      }
 
      public void keyPressed(KeyEvent e)
      {
 
      }
 
      public void keyReleased(KeyEvent e)
      {
 
      }
   }
}

Oh. I think I'm misunderstanding you. By printlns, I thought you meant this (which is what I did):

         if(e.getKeyCode() == KeyEvent.VK_ESCAPE)
         {
 
            System.out.print("Escape key pressed. Disposing of window...");
            dispose();
            System.out.print(" Done.\nExiting program...");
            System.exit(0);
 
         }

Note the print statements at the top and bottom of the code block. That's what I thought you meant. What did you actually mean?

Also note that those statements never printed, which means that the KeyEvent either a) doesn't occur, b) is blocked, or c) is wrongly coded by me.

Well firstly how did you get the program to display this:

sP e=java.awt.event.KeyEvent[KEY_PRESSED,keyCode=10,keyText=Enter,keyChar=Enter ,keyLocation=KEY_LOCATION_STANDARD,rawCode=13,prim aryLevelUnicode=13,scancode=28] on javax.swing.JTextField[,95,107,104x30,layout=javax.swing.plaf.basic.Basic TextUI$UpdateHandler,alignmentX=0.0,alignmentY=0.0 ,border=javax.swing.plaf.BorderUIResource$Compound BorderUIResource@169e11,flags=296,maximumSize=,min imumSize=,preferredSize=,caretColor=sun.swing.Prin tColorUIResource[r=51,g=51,b=51],disabledTextColor=javax.swing.plaf.ColorUIResourc e[r=184,g=207,b=229],editable=true,margin=javax.swing.plaf.InsetsUIRes ource[top=0,left=0,bottom=0,right=0],selectedTextColor=sun.swing.PrintColorUIResource[r=51,g=51,b=51],selectionColor=javax.swing.plaf.ColorUIResource[r=184,g=207,b=229],columns=290,columnWidth=11,command=,horizontalAli gnment=LEADING]
sT e=java.awt.event.KeyEvent[KEY_TYPED,keyCode=0,keyText=Unknown keyCode: 0x0,keyChar=Enter,keyLocation=KEY_LOCATION_UNKNOWN ,rawCode=0,primaryLevelUnicode=0,scancode=0] on javax.swing.JTextField[,95,107,104x30,layout=javax.swing.plaf.basic.Basic TextUI$UpdateHandler,alignmentX=0.0,alignmentY=0.0 ,border=javax.swing.plaf.BorderUIResource$Compound BorderUIResource@169e11,flags=296,maximumSize=,min imumSize=,preferredSize=,caretColor=sun.swing.Prin tColorUIResource[r=51,g=51,b=51],disabledTextColor=javax.swing.plaf.ColorUIResourc e[r=184,g=207,b=229],editable=true,margin=javax.swing.plaf.InsetsUIRes ource[top=0,left=0,bottom=0,right=0],selectedTextColor=sun.swing.PrintColorUIResource[r=51,g=51,b=51],selectionColor=javax.swing.plaf.ColorUIResource[r=184,g=207,b=229],columns=290,columnWidth=11,command=,horizontalAli gnment=LEADING]

..?

And if you say the listener is being called, why aren't my statements printing?

Because your if tests are not true.

-_- Well yes, I surmised that much. However, how is the enter key's code not equal to KeyEvent.VK_ENTER? I don't know what else it would be...

Did you look at the contents of the Event object?

I did now. Problem fixed. Thanks.