Thread: Why is a Frame Not Showing Up When I Run This Class?

I am trying to create a frame with 4 labels on it. When I run this class, it runs with no errors, but there is no frame that shows up? Any suggestions?


I should have a frame with 4 labels that read "test text" of 4 separate lines. Thanks for taking a look - Here is what I have:





import java.awt.Frame;
import java.awt.Label;

public class CustFrame extends Frame {
public CustFrame(Customer cust) {

this.setSize(300, 282);
this.setLayout(null);
this.setVisible(true);

Label custNameLbl = new Label();
Label shipToLbl1 = new Label();
Label shipToLbl2 = new Label();
Label contactInfo = new Label();

custNameLbl.setBounds(62, 65, 176, 23);
shipToLbl1.setBounds(62, 120, 176, 23);
shipToLbl2.setBounds(62, 175, 176, 23);
contactInfo.setBounds(62, 230, 176, 23);

custNameLbl.setText("test text");
shipToLbl1.setText("test text");
shipToLbl2.setText("test text");
contactInfo.setText("test text");

this.add(custNameLbl);
this.add(shipToLbl1);
this.add(shipToLbl2);
this.add(contactInfo);
}

public static void main(String[] args) {
Customer cust = new Customer();
Customer CustFrame = cust;
}
}

I am not sure why JFrame is not being used. This is in alignment with the example provided by my professor. What precisely do you mean when you ask how the code exits the JVM? How do you call the constructor exactly? Sorry, I am a newbie...

I'll take a look. Thanks!

--- Update ---

Okay, I took a look at the tutorials along with my textbook, and I am still obviously missing something because I am not getting a frame displayed. I think I understand the concept of how to create an object and how to instantiate it. My constructor starts with the public CustFrame(Customer cust); right? Isn't that "calling" the constructor? As for creating an instance of CustFrame, hmmmm...wouldn't that look something like: CustFrame cust = new CustFrame();? This would be the equivalent of creating an object named cust, which is an instantiation of the CustFrame class, correct? I'm sorry, but I am missing a basic element here. Thanks for your help and your patience!

Thanks Norm!

--- Update ---

Ugh...still not working...don't know why...probably going to take a break. Been going at this for like 5 hours - not really making progress.

I did this and it works for me. This is my code for a game i'm making.

	public static void main(String[] args) {
 
		JFrame frame = new JFrame("My Game Alpha v0.1"); // Title
 
		frame.add(new Main());
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // Ends process when closed
		frame.setSize(800, 500); // Resolution
		frame.setVisible(true); // Sets visible
		frame.setLocationRelativeTo(null); // Centers window when opened
		frame.setResizable(false);

Cool! Let me soak your code in for a while Cody, thanks! I think I already see a few nuggets that might help me out as far as grasping the concepts. you have everything in the main method...hmmm.

No problem man. And yes! I put it in a seperate class. This is the full source coede for it. If you want just create a class and add this. There are other ways to do it but I find this easiest.

package MyGame;
 
import javax.swing.JFrame;
 
public class Frame {
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("My Game Alpha v0.1"); // Title
 
		frame.add(new Main());
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // Ends process when closed
		frame.setSize(800, 500); // Resolution
		frame.setVisible(true); // Sets visible
		frame.setLocationRelativeTo(null); // Centers window when opened
		frame.setResizable(false);
	}
}

Try and make a new class with that and tell me if it works. Also don't forget to import. (CTRL + Shift + O)

Well, following is my code. Honestly, I think the problem is at least partially located within the main method - I am not sure I completely understanding what is happening there. I am essentially creating new objects but I am not sure to what end?



import java.awt.Frame;
import java.awt.Label;

public class CustFrame extends Frame {



public CustFrame(Customer cust) {

// Label custNameLbl = new Label();
this.setSize(300, 282);
this.setLayout(null);
this.setVisible(true);


Label custNameLbl = new Label();
this.setSize(300, 282);
this.setLayout(null);
custNameLbl.setBounds(62, 65, 176, 23);
custNameLbl.setText("test text");
this.add(custNameLbl);
this.setVisible(true);


Label shipToLbl1 = new Label();
this.setSize(300, 282);
this.setLayout(null);
shipToLbl1.setBounds(62, 120, 176, 23);
shipToLbl1.setText("test text");
this.add(shipToLbl1);
this.setVisible(true);

Label shipToLbl2 = new Label();
this.setSize(300, 282);
this.setLayout(null);
shipToLbl2.setBounds(62, 175, 176, 23);
shipToLbl2.setText("test text");
this.add(shipToLbl2);
this.setVisible(true);

Label contactInfo = new Label();
this.setSize(300, 282);
this.setLayout(null);
contactInfo.setBounds(62, 230, 176, 23);
contactInfo.setText("test text");
this.add(contactInfo);
this.setVisible(true);
}

}
public CustFrame() {
// TODO Auto-generated constructor stub
}

public CustFrame cust = new CustFrame();

public static void main(String[] args) {
Customer cust = new Customer();
CustFrame Customer = new CustFrame();
}
}


As for error messages, I get about a hundred lines of "at CustFrame.<init>(CustFrame.java:70)" followed by one line of "at CustFrame.main(CustFrame.java:74)." I do not, however, generate any frame with the appropriate labels at all. Is there something obvious in my code staring me in the face that I am missing? Thanks!

Originally Posted by Ormolu611

Well, following is my code. Honestly, I think the problem is at least partially located within the main method - I am not sure I completely understanding what is happening there. I am essentially creating new objects but I am not sure to what end?



import java.awt.Frame;
import java.awt.Label;

public class CustFrame extends Frame {



public CustFrame(Customer cust) {

//Label custNameLbl = new Label();
this.setSize(300, 282);
this.setLayout(null);
this.setVisible(true);


Label custNameLbl = new Label();
this.setSize(300, 282);
this.setLayout(null);
custNameLbl.setBounds(62, 65, 176, 23);
custNameLbl.setText("test text");
this.add(custNameLbl);
this.setVisible(true);


Label shipToLbl1 = new Label();
this.setSize(300, 282);
this.setLayout(null);
shipToLbl1.setBounds(62, 120, 176, 23);
shipToLbl1.setText("test text");
this.add(shipToLbl1);
this.setVisible(true);

Label shipToLbl2 = new Label();
this.setSize(300, 282);
this.setLayout(null);
shipToLbl2.setBounds(62, 175, 176, 23);
shipToLbl2.setText("test text");
this.add(shipToLbl2);
this.setVisible(true);

Label contactInfo = new Label();
this.setSize(300, 282);
this.setLayout(null);
contactInfo.setBounds(62, 230, 176, 23);
contactInfo.setText("test text");
this.add(contactInfo);
this.setVisible(true);
}

}
public CustFrame() {
// TODO Auto-generated constructor stub
}

public CustFrame cust = new CustFrame();

public static void main(String[] args) {
Customer cust = new Customer();
CustFrame Customer = new CustFrame();
}
}


As for error messages, I get about a hundred lines of "at CustFrame.<init>(CustFrame.java:70)" followed by one line of "at CustFrame.main(CustFrame.java:74)." I do not, however, generate any frame with the appropriate labels at all. Is there something obvious in my code staring me in the face that I am missing? Thanks!

I see your issue. Your calling the wrong constructor. You have two of them. One that accepts a parameter of type customer and another that has no para maters. The one that take son parameters is being called but does nothing! Pass in cust inside of the constructor call and it will take off. also don't instantiate twice.

 
 import java.awt.Frame;
 import java.awt.Label;
 
 public class CustFrame extends Frame {
 
 
 
 public CustFrame(Customer cust) {
 
	 //Label custNameLbl = new Label();
	 this.setSize(300, 282);
	 this.setLayout(null);
	 this.setVisible(true);
 
 
	 Label custNameLbl = new Label();
	 this.setSize(300, 282);
	 this.setLayout(null);
	 custNameLbl.setBounds(62, 65, 176, 23);
	 custNameLbl.setText("test text");
	 this.add(custNameLbl);
	 this.setVisible(true);
 
 
	 Label shipToLbl1 = new Label();
	 this.setSize(300, 282);
	 this.setLayout(null);
	 shipToLbl1.setBounds(62, 120, 176, 23);
	 shipToLbl1.setText("test text");
	 this.add(shipToLbl1);
	 this.setVisible(true);
 
	 Label shipToLbl2 = new Label();
	 this.setSize(300, 282);
	 this.setLayout(null);
	 shipToLbl2.setBounds(62, 175, 176, 23);
	 shipToLbl2.setText("test text");
	 this.add(shipToLbl2);
	 this.setVisible(true);
 
	 Label contactInfo = new Label();
	 this.setSize(300, 282);
	 this.setLayout(null);
	 contactInfo.setBounds(62, 230, 176, 23);
	 contactInfo.setText("test text");
	 this.add(contactInfo);
	 this.setVisible(true);
 	}
 
/*
 	public CustFrame() {
 	// TODO Auto-generated constructor stub
 	}
*/
 	//public CustFrame cust;
 
 	public static void main(String[] args) {
 		Customer cust = new Customer();
 		CustFrame Customer = new CustFrame(cust);
 	}
 }

try that for your code

Rockking8350, it works! Fantastic! Thank you. So, to review, the constructor that accepts a parameter of type customer in my code is:

Customer cust = new Customer();

Conversely, the constructor that was doing nothing was:

CustFrame Customer = new CustFrame();

Is this accurate? So my understanding is that with the first constructor, I was instantiating an object of type Customer called cust. The second constructor, I was essentially not "passing" anything to that object, and that was my problem - is that right? So by adding cust in the parenthesis, I was "passing" the previously created cust to Customer....right? I am still just a little shaky about what is happening here and just want to make sure. Thanks!

you have another issue. you have the same name variable. cust is two different types of variables. you really need to watch how you instantiate the variables, and make sure not to double do it.

Customer cust = new Customer();

all that does is makes a new customer named cust

CustFrame Customer = new CustFrame();

this is the call to the CustFrame constructor. but by property of polymorphism you are doing nothing. you need to pass in a variable of type customer to actually do anything because the constructor that accepts a parameter is what has all the code in it. so you were making a call to an empty constructor which did nothing. also, watch your naming. your frame is the same name as a class type. that should be changed. now the reason why you didn't get an error is because you had a constructor that could be called but didn't require a parameter. had you not had the empty constructor, you would have been told that it required a parameter of type customer.

Thanks guys! I am going to play with it a bit and try to comment out alternate code snippets to see the effect. It looks like some of my coding is superfluous and unnecessary...I'll also try to be aware of the double dipping variable issue...thanks again!

we all started somewhere.