JAVA Problem med at sætte JMenuBar ude for class

Den er ikke nem at opdage, men det er fordi du aldrig instantierer undermenuerne. Menuen bliver tilført, problemet er at menuen er tom. På intet tidspunkt bliver Menu.Menu() kaldt. Den er erklæret statisk så det er ikke en konstruktor. Det vil sige at linje 53 i TSudio1TE.java ikke har nogen effect, siden den ikke bliver brugt til noget. Hvis du før linje 55 skriver Menu.Menu(); så kommer hele menuen frem.

Som de andre rigtigt siger, så initialiserer du ikke din menu ordenligt.

1. Niveau menubar
2. Niveau menu
3. Niveau menuitem


Du skal huske, at for at en menubar vises, skal der som minimum være en menu i den, med en tekst, der er forskellige fra tom. Derudover så skal man sætte menubaren sammen med sin JFrame og ikke Menuen.

Hvis man sætter dette ind i din kode vil det ca. se sådanne her ud.

 /*
   * To change this template, choose Tools | Templates
   * and open the template in the editor.
   */
  package tsudio.pkg1te;
  
  import GUI.Menu;
  import java.awt.BorderLayout;
  import java.awt.Color;
  import java.awt.Component;
  import java.awt.Frame;
  import java.awt.Image;
  import java.awt.Toolkit;
  import java.io.IOException;
  import java.util.logging.Level;
  import java.util.logging.Logger;
  import javax.imageio.ImageIO;
  import javax.swing.JFrame;
  import javax.swing.JMenu;
  import javax.swing.JMenuBar;
  import javax.swing.JMenuItem;
  import javax.swing.JPanel;
  import javax.swing.JPopupMenu;
  import javax.swing.JSeparator;
  import javax.swing.JTabbedPane;
  import javax.swing.JTextPane;
  import static tsudio.pkg1te.TSudio1TE.GUI;
  import static tsudio.pkg1te.TSudio1TE.jtab;
  import static tsudio.pkg1te.TSudio1TE.jtp;
  import static tsudio.pkg1te.TSudio1TE.jtp1;
  
  /**
   *
   * @author Mathias
   */
  public class TSudio1TE {
  
      /**
       * @param args the command line arguments
       */
      public static JTabbedPane jtab;
      public static JFrame GUI;
      public static JTextPane jtp;
      public static JTextPane jtp1;
      
      public static void main(String[] args) {
          GUI = new JFrame();
          GUI.setTitle("TSudio 1TE");
          GUI.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          GUI.setExtendedState(Frame.MAXIMIZED_BOTH);
          GUI.setSize(860, 440);
          
 
          //Her er de vigtige ændringer
          MenuBar menubar = new MenuBar()
          Menu menu = new Menu("File");
          menubar.add(menu);
 
          
          GUI.SetJMenuBar(menubar);
          //Nu slutter de vigtige ændringer
          
          jtab = new JTabbedPane();
          jtp = new JTextPane();
          jtp1 = new JTextPane();
          jtab.setForeground(Color.BLACK);
          jtp.setBackground(Color.BLACK);
          jtp.setForeground(Color.WHITE);
          jtab.add("New file (1)", jtp);
          
          GUI.setIconImage(Toolkit.getDefaultToolkit().getImage("T1TE.png"));
          GUI.add(BorderLayout.CENTER,jtab);
          GUI.setVisible(true);
      }
      
      
  }

Hvor initialiserer du menuBar henne? Prøv at ændre linjen:

 public static JMenuBar menubar = new JMenuBar();

Mange Swing metoder accepterer gerne null værdier uden at smide en exception, og i det her tilfælde vil en null-værdi fjerne menubar'en, hvis der var en.

Som altid, så kan vi ikke bruge "den gider ikke" til ret meget. Får du fejl eller hvad sker? Post evt. lidt mere kode.

Altså vis den sagde fejl så vil jeg skrive det.

Men her er hele min kode :

TSudio1TE

 /*
  * To change this template, choose Tools | Templates
  * and open the template in the editor.
  */
 package tsudio.pkg1te;
 
 import GUI.Menu;
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Component;
 import java.awt.Frame;
 import java.awt.Image;
 import java.awt.Toolkit;
 import java.io.IOException;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 import javax.imageio.ImageIO;
 import javax.swing.JFrame;
 import javax.swing.JMenu;
 import javax.swing.JMenuBar;
 import javax.swing.JMenuItem;
 import javax.swing.JPanel;
 import javax.swing.JPopupMenu;
 import javax.swing.JSeparator;
 import javax.swing.JTabbedPane;
 import javax.swing.JTextPane;
 import static tsudio.pkg1te.TSudio1TE.GUI;
 import static tsudio.pkg1te.TSudio1TE.jtab;
 import static tsudio.pkg1te.TSudio1TE.jtp;
 import static tsudio.pkg1te.TSudio1TE.jtp1;
 
 /**
  *
  * @author Mathias
  */
 public class TSudio1TE {
 
     /**
      * @param args the command line arguments
      */
     public static JTabbedPane jtab;
     public static JFrame GUI;
     public static JTextPane jtp;
     public static JTextPane jtp1;
     
     public static void main(String[] args) {
         GUI = new JFrame();
         GUI.setTitle("TSudio 1TE");
         GUI.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         GUI.setExtendedState(Frame.MAXIMIZED_BOTH);
         GUI.setSize(860, 440);
         
         Menu men = new Menu();
         
         Menu.SetJMenuBar();
         
         jtab = new JTabbedPane();
         jtp = new JTextPane();
         jtp1 = new JTextPane();
         jtab.setForeground(Color.BLACK);
         jtp.setBackground(Color.BLACK);
         jtp.setForeground(Color.WHITE);
         jtab.add("New file (1)", jtp);
         
         GUI.setIconImage(Toolkit.getDefaultToolkit().getImage("T1TE.png"));
         GUI.add(BorderLayout.CENTER,jtab);
         GUI.setVisible(true);
     }
     
     
 }

Menu

 /*
  * To change this template, choose Tools | Templates
  * and open the template in the editor.
  */
 package GUI;
 
 import javax.swing.JMenu;
 import javax.swing.JMenuBar;
 import javax.swing.JMenuItem;
 
 /**
  *
  * @author Mathias
  */
 public class Menu {
     public static JMenuBar menubar = new JMenuBar();
     public static JMenu filemenu;
     public static JMenu editmenu;
     public static JMenu insertmenu;
     public static JMenu ShowMenu;
     public static JMenu LanguageMenu;
     public static JMenu OptionsMenu;
     
    //Insert menu
     public static JMenu HTML;
     public static JMenu CSS;
     public static JMenu PHP;
     public static JMenu JavaScript;
     
     //
     
     //File menu
     public static JMenuItem Open;
     public static JMenuItem New;
     public static JMenuItem Save;
     public static JMenuItem SaveAs;
     public static JMenu RecentFiles;
     public static JMenuItem Print;
     public static JMenuItem Exit;
     //Edit menu
     
     //Show menu
     public static JMenuItem TextEditor;
     
     public static JMenuItem Undo;
     public static JMenuItem Redo;
     public static JMenuItem Copy;
     public static JMenuItem Paste;
     public static JMenuItem Cut;
     public static JMenuItem Select;
     public static JMenuItem SelectAll;
     public static JMenuItem Replacetw;
     public static JMenuItem Find;
     //Language menu
     
     public static JMenuItem Danish;
     public static JMenuItem English;
     public static JMenuItem Sweden;
     public static JMenuItem Norway;
     public static JMenuItem Germany;
     public static JMenuItem French;
     public static JMenuItem Spanish;
     public static JMenuItem Italian;
     public static JMenuItem Japanese;
     public static JMenuItem Chinese;
     public static JMenuItem Finnish;
     public static JMenuItem African;
     public static JMenuItem Greenland;
     public static JMenuItem Icelandic;
     public static JMenuItem Polish;
     public static JMenuItem Greek;
     public static JMenuItem Turkish;
     public static JMenuItem Dutch;
     public static JMenuItem Irish;
     public static JMenuItem Russian;
     public static JMenuItem Korean;
     public static JMenuItem Portogisks;
     public static JMenuItem Latin;
     public static JMenuItem Estonian;
     
     //Options
     
     public static JMenuItem Options;
     public static JMenu Layouts;
     public static void Menu() {
         filemenu = new JMenu("File");
         editmenu = new JMenu("Edit");
         insertmenu = new JMenu("Insert");
         ShowMenu = new JMenu("Show");
         LanguageMenu = new JMenu("Language");
         OptionsMenu = new JMenu("Options");
         
         //FileMenu
         Open = new JMenuItem("Open");
         New = new JMenuItem("New");
         Save = new JMenuItem("Save");
         SaveAs = new JMenuItem("Save as");
         RecentFiles = new JMenu("Recent files");
         Print = new JMenuItem("Print");
         Exit = new JMenuItem("Exit");
         
         //Edit menu
         Undo = new JMenuItem("Undo");
         Redo = new JMenuItem("Redo");
         Copy = new JMenuItem("Copy");
         Cut = new JMenuItem("Cut");
         Paste = new JMenuItem("Paste");
         Select = new JMenuItem("Select");
         SelectAll = new JMenuItem("Select all");
         Find = new JMenuItem("Find");
         Replacetw = new JMenuItem("Replace all with");
         
         //Insert Menu
         HTML = new JMenu("HTML");
         CSS = new JMenu("CSS");
         PHP = new JMenu("PHP");
         JavaScript = new JMenu("JavaScript");
         
         //Language
         Danish = new JMenuItem("Dansih - Dansk");
         English = new JMenuItem("English");
         Sweden = new JMenuItem("Sweden - Svenska");
         Norway = new JMenuItem("Norway - Norsk");
         Germany = new JMenuItem("Germany - Deutch");
         French = new JMenuItem("French - français");
         Spanish = new JMenuItem("Spanish - español");
         Italian = new JMenuItem("Italian - italiano");
         Japanese = new JMenuItem("Japanese - ???");
         Chinese = new JMenuItem("Chinese Traditional - ???");
         Finnish = new JMenuItem("Finnish - suomalainen");
         African = new JMenuItem("African - Afrikaanse");
         Greenland = new JMenuItem("Greenland - Kalaallit Nunaat");
         Icelandic = new JMenuItem("Icelandic - Icelandic");
         Polish = new JMenuItem("Polish - polski");
         Greek = new JMenuItem("Greek - e???????");
         Turkish = new JMenuItem("Turkish - Türk");
         Dutch = new JMenuItem("Dutch - Nederlands");
         Irish = new JMenuItem("Irish - Gaeilge");
         Russian = new JMenuItem("Russian - ???????");
         Korean = new JMenuItem("Korean - ???");
         Portogisks = new JMenuItem("Portogisks - ?????");
         Latin = new JMenuItem("Latin - Latine");
         Estonian = new JMenuItem("Estonian - eesti");
         
         
         //Show menu
         TextEditor = new JMenuItem("Text Editor");
         
         //Options
         Options = new JMenuItem("Options");
         Layouts = new JMenu("Layouts");
         
         filemenu.add(New);
         filemenu.add(Open);
         filemenu.add(Save);
         filemenu.add(SaveAs);
         filemenu.addSeparator();
         filemenu.add(RecentFiles);
         filemenu.addSeparator();
         filemenu.add(Print);
         filemenu.addSeparator();
         filemenu.add(Exit);
         
         editmenu.add(Undo);
         editmenu.add(Redo);
         editmenu.addSeparator();
         editmenu.add(Copy);
         editmenu.add(Cut);
         editmenu.add(Paste);
         editmenu.addSeparator();
         editmenu.add(Select);
         editmenu.add(SelectAll);
         editmenu.add(Find);
         editmenu.add(Replacetw);
         
         insertmenu.add(HTML);
         insertmenu.add(CSS);
         insertmenu.add(PHP);
         insertmenu.add(JavaScript);
         
         
         LanguageMenu.add(African);
         LanguageMenu.add(Chinese);
         LanguageMenu.add(Danish);
         LanguageMenu.add(Dutch);
         LanguageMenu.add(English);
         LanguageMenu.add(Estonian);
         LanguageMenu.add(French);
         LanguageMenu.add(Finnish);
         LanguageMenu.add(Germany);
         LanguageMenu.add(Greek);
         LanguageMenu.add(Greenland);
         LanguageMenu.add(Icelandic);
         LanguageMenu.add(Irish);
         LanguageMenu.add(Italian);
         LanguageMenu.add(Japanese);
         LanguageMenu.add(Korean);
         LanguageMenu.add(Latin);
         LanguageMenu.add(Norway);
         LanguageMenu.add(Polish);
         LanguageMenu.add(Portogisks);
         LanguageMenu.add(Russian);
         LanguageMenu.add(Turkish);
         
         menubar.add(filemenu);
         menubar.add(editmenu);
         menubar.add(insertmenu);
         menubar.add(ShowMenu);
         menubar.add(LanguageMenu);
         menubar.add(OptionsMenu);
     }
     
     public static void SetJMenuBar() {
         tsudio.pkg1te.TSudio1TE.GUI.setJMenuBar(menubar);
     }
 }

Ok, så menuen vises bare ikke eller?

Giver det en forskel, hvis du i din Main metode laver

 GUI.setJMenuBar(menubar);

lige efter at du instantierer din Menu? I stedet for at kaldet Menu klassens SetJMenuBar?

Ingen ændringer

Den er ikke nem at opdage, men det er fordi du aldrig instantierer undermenuerne. Menuen bliver tilført, problemet er at menuen er tom. På intet tidspunkt bliver Menu.Menu() kaldt. Den er erklæret statisk så det er ikke en konstruktor. Det vil sige at linje 53 i TSudio1TE.java ikke har nogen effect, siden den ikke bliver brugt til noget. Hvis du før linje 55 skriver Menu.Menu(); så kommer hele menuen frem.

Tak skal du havde.