Thread: Improper Way of Achieving Output?

Hey everyone,

Quick Question Posed to ME



----------------------------------------------------------------------------------------------------------------------


The question:

Create a Television store that can hold three Television objects in an
array. Use the Television class below:

class Television
{
boolean isOn;
} // end class Television

Use a for loop to print the isOn instance variable for each Television in the
TelevisionStore. Use a second for loop to change the isOn instance variable for
each Television to “true”. Finally, use a third for loop to print the isOn instance
variable for each Television in the TelevisionStore array.

Sample output:

Openening the tv store for the day... tv status:
Television 0 is on? False.
Television 1 is on? False.
Television 2 is on? False.

Turning the tv's on...
Television 0 is on? True.
Television 1 is on? True.
Television 2 is on? True.






----------------------------------------------------------------------------------------------------------------------

So I made a program that does create this output, with this code + the Television Class I was supposed to create:

public class TelevisionDriver
    {
    public static void main( String[] args )
        {
        String[] Television = { "Television 0", "Television 1", "Television 2", "Television 0", "Television 1", "Television 2" };
        String[] Boolean = { "False", "False", "False", "True", "True", "True"};
 
 
        System.out.println( "Openening the tv store for the day... tv status:");
        for( int x = 0; x < 3; x++ )
            {
            System.out.println( Television[x] + " is on? " + Boolean[x] + "." );
 
            }
        System.out.println( "\n" + "Turning the tv's on..." );
        for( int y = 3; y < 6; y++)
            {
 
            System.out.println( Television[y] + " is on? " + Boolean[y] + "." );
            }
        }
    }

----------------------------------------------------------------------------------------------------------------------

Is this the proper way to do it? I think I was supposed to do something different, even though the output is correct.

I don't believe that is what your instructor will want to see.

Is that the whole question? Does it mention any thing about creating a constructor for the television class? Does it ask you to create some form of getter or setter for the instance variable isOn (yep). Can you override the toString method?

You shouldn't be making an array of Strings. How do you create a new object?

No, this is not the correct way. There was a purpose in asking you to create the Television class, it's a baby step to understanding Object Oriented Programming, or OOP. You will not do well if you turn in the above program.

Well he did create some Objects just the wrong kind(Strings). The assignment does not call for an array of Strings but for what? I am guessing the instructor provided you with that class. Now when you create an object normally you have getters and setters as mentioned in post #2 which is the correct way of doing things but in this instance on how it is setup it seems you won't really need them.

If you have any questions or are getting errors come back and ask for help. Make sure to include the full details of the error you are receiving.

Thank you. Would this be proper then? Just making sure I am approaching this right

 
public class TelevisionDriver
    {
    public static void main( String[] args )
        {
         boolean isOn[] = new boolean[3];
         isOn[0] = false;
         isOn[1] = false;
         isOn[2] = false;
 
         System.out.println( "Opening the tv store for the day... tv status:" + "\n" );
 
         for( int x = 0; x < 3; x++ )
            {
            System.out.println( "Television" + x + " on? " + isOn[x] );
            }
 
         System.out.println( "\n" + "Turning the tv's on..." + "\n" );
 
         for( int y = 0; y < 3; y++ )
            {
            isOn[0] = true;
            isOn[1] = true;
            isOn[2] = true;
 
            System.out.println( "Television" + y + " on? " + isOn[y] );
            }
        }
    }

Keep in mind, I have only been using java for a week or two VERY lightly. I am not educated in advanced mechanics. I appreciate all constructive, but positive responses.

Im afraid you do not yet get the concept. Take some time and read through this tutorial, about 20 pages worth. (It is large print, it should not take long)
Hopefully you will see the way an object saves it's own state independent of any other object, and that saving the values the way done in the two previous code samples is not the way to go. Each object should be able to tell you if it is on or off, and not have a permanent value hard coded in somewhere.

No you need to create TV objects. That class is provided to you by the instructor and you are not using it at all. I'm not sure if it's provided to you as a separate file or he/she expects you to use it in your TelevisionDriver file. Notice that the status of the TV is a property of the TV itself, it is the class level variable called isOn. You will access this property by referring to an instance of each TV you create.

Start your code by creating these three objects very similarly to how you've created your booleans. Then think about how you will get their isOn status.

EDIT: Follow the link jps gave you. It's perfect.