0

これは、3 つのクラス ファイルを含む Java 割り当てです。問題は、メインで「doOperations」メソッドを呼び出すときです。if ステートメントはファイルを正しく読み取りますが、オブジェクト メソッドは機能しません (例: tv.on、tv.volumeUp tv.setChannel(scan.nextInt() など))。

誰が私が間違っているのかを見ることができますか? ありがとう。

TV クラス ファイル。

import java.io.*;
import java.util.Scanner;

public class TV {
    boolean on;
    boolean subscribe;
    int currentchannel;
    int indexChan;
    int volume;
    File chanList;
    Scanner chanScan;
    TVChannel[] channels;
    final int MaxChannel = 100;

    public TV(){
        on = false;
        currentchannel = 1;
        volume = 1;
        channels = new TVChannel[MaxChannel];   //Create object of an array, default value = null.
        subscribe = false;

    }

    public void turnOn(){
        on = true;
    }
    public void turnOff(){
        on = false;
    }
    public void channelUp(){
        currentchannel++;
    }
    public void channelDown(){
        currentchannel--;
    }
    public void volumeUp(){
        volume++;
    }
    public void volumeDown(){
        volume--;
    }
    public TVChannel getChannel(){
        return channels[currentchannel];
    }
    public void setChannel(int c){
        if(channels[c]!= null)
        currentchannel = c;
    }
    public boolean subscribe(String provider) throws IOException{
        chanList = new File(provider);
        chanScan = new Scanner(chanList);

        if(chanList.exists()){
            while(chanScan.hasNext()){

                indexChan = chanScan.nextInt();
                channels[indexChan] = new TVChannel(indexChan, chanScan.nextLine());

            }
            chanScan.close();
            return true;
        }
        else return false;
    }

    public void printAll(){
        for(int i = 0; i < MaxChannel; i++){

            if(channels[i]!= null)
            System.out.println(channels[i].channelNumber+"\t"+channels[i].channelName);
        }
    }

    public void displayChannel(int c){
        if(channels[c]!= null)
        System.out.println(channels[c].getChannel()+"\t"+channels[c].getName());
    }
    public void displayCurrentChannel(){
        if(channels[currentchannel] != null)
        System.out.println(channels[currentchannel].getChannel() +" "+ channels[currentchannel].getName());
    }
}

主な機能

import java.io.File;
import java.io.IOException;
import java.util.Scanner;


public class TestTV {

    public static void main(String[] args) throws IOException{


        TV tv = new TV();
        tv.subscribe("chan.txt");

        if(tv.subscribe = true){
            tv.printAll();
            doOperations("operations.txt", tv);

        }

    }
    //Static means only one instance of object. 
    public static void doOperations(String opFileName, TV tv) throws IOException{
            File op = new File(opFileName);
            Scanner scan = new Scanner(op);
            String opa = scan.next();

            while(scan.hasNext()){
                System.out.println(scan.next());

                if(opa.equals("on")) tv.turnOn();
                else if(opa.equals("off")) tv.turnOff();
                else if(opa.equals("channelUp")) tv.channelUp();
                else if(opa.equals("channelDown")) tv.channelDown();
                else if(opa.equals("volumeUp")) tv.volumeUp();
                else if(opa.equals("volumeDown")) tv.volumeDown();
                else if(opa.equals("showChannel")) tv.displayCurrentChannel();
                else if(opa.equals("setChannel"))tv.setChannel(scan.nextInt()); //Error
            }
            scan.close();
        }

}
4

2 に答える 2

1

String opa = scan.next();これを while ループの中に入れるのを忘れていました 。もちろん、各トークンに結果を保存する必要があります。

while(scan.hasNext()){
    opa = scan.next();
    System.out.println(opa); //don't just call scan.next() 
                             //and discard the result. Unless that's really what you want?
    .....
}
于 2013-01-24T17:30:09.450 に答える
0

コードに問題はありません。operation.txt が次のようになっていることを確認してください showChannel setChannel 2

注：あなたはscan.next（）;なので while ループの前に、ループ内で System.out.println(scan.next()); として再度実行されます。operation.txt の最初の行を無視すると、チャネルが正しく設定されていることがわかります。

提案：次のように変更します

     String opa= "";// = scan.next();
        while(scan.hasNext()){
            System.out.println(scan.next());
于 2013-01-24T18:18:19.290 に答える