Method Needs to return a value from a string array

Question

I need to create a method for my Television class that will receive an integer, but will return a string value (channel name) from an array. I am having trouble in the toString() are where I am receiving an error "error- '.class' expected' for the part that is supposed to display the channel name. Thanks and here is my code.

public class Television

{

private int channel;
private int currentChannel;
private int volume;
private boolean power;
private String[] channelName = {"CBS", "FOX", "DISCOVERY", "PBS", "HBO", "CNN", "DISNEY", "CNN", "TBS", "USA"};



//No argument constructor
public Television()
{
    currentChannel = 1;
    volume = 0;
    power = false;

}


public void powerChange()
{
    this.power = !this.power;
}

public void setVolume(int vol)
{
    if (vol>10)
    {
    volume = 10;
    }else
    {
        volume = vol;
    }
    if (vol<0)
    {
    volume = 0;
    }

}

public void increaseVolume()
{
    volume++;
}

public void decreaseVolume()
{
    volume--;
}

public int getVolume()
{
    return volume;
}

public void setChannel(int ch)
{
    if(ch>10)
    {
    channel = 10;
    }else
    {
        channel = ch;
    }
    if(ch<1)
    {
    channel = 1;
    }
}

public void increaseChannel()
{
    channel++;
}

public void decreaseChannel()
{
    channel--;
}

public int getChannel()
{
    return channel;
}

public String getChannelName(int channel)
{

    if (channel==1)
    {
        return channelName[0];
    }
    else if (channel == 2)
    {
        return channelName[1];
    }   
    else if (channel == 3)
    {
        return channelName[2];
    }   
    else if (channel == 4)
    {
        return channelName[3];
    }   
    else if (channel == 5)
    {
        return channelName[4];
    }   
    else if (channel == 6)
    {
        return channelName[5];
    }   
    else if (channel == 7)
    {
        return channelName[6];
    }
    else if (channel == 8)
    {
        return channelName[7];
    }   
    else if (channel == 9)
    {
        return channelName[8];
    }   
    else if (channel == 10)
    {
        return channelName[9];
    }   


}


public String toString()
{
    if(!power)
    {
        return String.format("%s :%s\n%s :%d\n %s :%s\n%s :%d", "TV State", "OFF", "Channel No", channel, "Channel Name", getChannelName(channel), "Volume", volume);
    }
    else if(power)
    {
        return String.format("%s :%s\n%s :%d\n %s :%s\n%s :%d", "TV State", "ON", "Channel No", channel, "Channel Name", getChannelName(channel), "Volume", volume);
    }


}

}

What do you think this piece of code does: getchannelName[]? — Sotirios Delimanolis
– Sotirios Delimanolis, Commented Nov 12, 2013 at 3:10
I did not have getchannelName[] in my original code, but I was trying to play with the toString() just to see if anything would help it. I was trying to invoke the method I had created earlier but I realize that this did not work. — user2981579
– user2981579, Commented Nov 12, 2013 at 3:18

Paul Samsotha · Accepted Answer · 2013-11-12 03:21:01Z

2

TVChannel[] channelName = {"CBS", "FOX", "DISCOVERY", "PBS", "HBO", 
                           "CNN", "DISNEY", "CNN", "TBS", "USA"};

Should be

String[] channelName = {"CBS", "FOX", "DISCOVERY", "PBS", "HBO", 
                        "CNN", "DISNEY", "CNN", "TBS", "USA"};

Your array is of TVChannel type and should be of String type. TVChannel is a class that does not even exist.

Edit: also consider

channelName = new TVChannel[MaxChannel];  // delete this from your constructor.
                                          // it is not needed. Your instance of
                                          // the class already give you array

Also consider your getChannelName() method. Maybe you swant take an int argument?

public String getChannelName(int channel){ // instead of using the instance var
                                           // the logic really doesn't make sense
                                           // for your situation

}

edited Nov 12, 2013 at 3:21

answered Nov 12, 2013 at 3:11

Paul Samsotha

210k39 gold badges503 silver badges756 bronze badges

Sign up to request clarification or add additional context in comments.

10 Comments

MadProgrammer Over a year ago

Or it needs to list instances of TVChannel, which ever the OP actually needs ;)

user2981579 Over a year ago

Thanks. I have changed this but I am still receiving the same error message.

Paul Samsotha Over a year ago

@user2981579, you don't need this piece of code in your constructor channelName = new TVChannel[MaxChannel];

user2981579 Over a year ago

@peeskillet thank you, that does make more sense. Any tips on how to refer to it in the toString method? I am still getting and error from this.

Paul Samsotha Over a year ago

@user2981579 getchannelName[]. You want getChannel(channel) in your toString(). What error are you getting? Is it just in the toString()?

|

Collectives™ on Stack Overflow

Method Needs to return a value from a string array

1 Answer 1

10 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

10 Comments

Your Answer

Sign up or log in

Post as a guest

Related