Why is this a really bad code and how would you fix it? public class HelloCh8{ private String message; private static int counter; public HelloCh8(){ message = new String("Hello World"); counter++; } public HelloCh8(String str){ this.message = str; counter++; } public void setMessage(String str){ this.message = str; } public String toString(){ return this.message; } } ========================================= public class HelloCh8Bad{ private HelloCh8 hello; public HelloCh8Bad(){ hello=new HelloCh8(); } public HelloCh8 getHello(){ return this.hello; } public String toString(){ return this.hello.toString(); } } ========================================== public class HelloCh8Driver{ public static void main(String args[]){ HelloCh8Bad hello=new HelloCh8Bad
Why is this a really bad code and how would you fix it?
public class HelloCh8{
private String message;
private static int counter;
public HelloCh8(){
message = new String("Hello World");
counter++;
}
public HelloCh8(String str){
this.message = str;
counter++;
}
public void setMessage(String str){
this.message = str;
}
public String toString(){
return this.message;
}
}
=========================================
public class HelloCh8Bad{
private HelloCh8 hello;
public HelloCh8Bad(){
hello=new HelloCh8();
}
public HelloCh8 getHello(){
return this.hello;
}
public String toString(){
return this.hello.toString();
}
}
==========================================
public class HelloCh8Driver{
public static void main(String args[]){
HelloCh8Bad hello=new HelloCh8Bad();
System.out.println(hello);
}
}
Step by step
Solved in 2 steps with 1 images
