javac утверждает, что я не переопределяю метод в реализации абстрактного класса, когда я явно - PullRequest
6 голосов
/ 08 июля 2010

Я сделаю это как можно более кратким и конкретным, но это довольно сложный вопрос.Я пишу на Java на платформе Linux, чего бы это ни стоило.

Краткая версия цели: я хочу иметь абстрактный класс Client, который действует как универсальный контейнер для клиентских подключений.Client должен пронизывать каждое из его соединений.У меня также есть некоторый полу-протестированный код, который воспроизводит аналог сервера на аналогичном коде.Резюме Client должно быть реализовано во что-то более осязаемое и инстанцируемое.В моем случае у меня есть класс FileClientGui, который расширяет Client и переопределяет все абстрактные методы Client с помощью базового метода получения содержимого файла с сервера и его отображения.Это еще более осложняется тем фактом, что абстрактный Client сам по себе является расширением java.lang.Thread.

Итак, вот моя файловая структура в общих терминах:

/ class / path/lib/client/Client.java

/ class / path / com / fileclient / FileClientGui.java

Существует несколько других пользовательских классов, на которые ссылаются оба этих файла, но яЯ не получаю никаких ошибок из них.Если мне нужно отправить код для этих элементов, дайте мне знать, и я опубликую их.

Поэтому я запускаю эту длинную команду javac на терминале, настраивая каталог classpath и build, а также все соответствующие файлы.это также должно быть скомпилировано.Единственная ошибка, которую я получаю для любого из этого кода, заключается в следующем:

com/fileclient/FileClientGui.java:26: com.fileclient.FileClientGui is not abstract and does not override abstract method cleanClients() in lib.client.Client

Мой код (см. Ниже) четко реализует метод и все другие абстрактные методы, определенные в Client.java.Я искал в Интернете, и кажется, что большинство людей, которые сталкиваются с этой ошибкой, пытаются сделать что-то вроде реализации ActionListener и путаются с этой реализацией, и часто это просто орфография или заглавные буквы.Я перебираю свой код, чтобы убедиться, что это не простая проблема типа «упс».Я подозреваю, что это на самом деле какая-то коллизия между именем моего класса и именем какого-то другого класса, который каким-то образом попал в мой путь к классам или в собственные фреймворки / библиотеки Java, но я не могу найти ничего очевидного.

В любом случае, вот мой код.

Client.java:

package lib.client;

import lib.clientservercore.Connection;
import lib.simplefileaccess.Logger;
import java.io.IOException;
import java.net.Socket;
import java.util.ArrayList;
import java.lang.Thread;

/**
 *
 * @author Ryan Jung
 */
public abstract class Client extends Thread {

    ArrayList<Connection> connections;
    boolean isRunning;
    Logger log;

    public Client (String logFile) {
        log = new Logger(logFile);

        log.write("Initializing client...");

        connections = new ArrayList<Connection>(50);

        log.write("Client initialized.");
    }

    public void logOut(String contents) {
        log.write(contents);
    }

    public Logger getLogger() {
        return this.log;
    }

    public ArrayList<Connection> getConnections() {
        return connections;
    }

    public void addConnection(Connection c) {
        connections.add(c);
    }

    public void removeConnection(Connection c) {
        connections.remove(c);
    }

    public boolean getIsRunning() {
        return isRunning;
    }

    public void setIsRunning(boolean r) {
        isRunning = r;
    }

    public Connection connect(String host, int port) {
        log.write("Creating new connection...");

        Socket s;
        Connection c = null;

        // Validate port
        if (port <= 1024 || port > 65536) {
            log.write("Invalid server port: " + port + ".  Using 12321.");
            port = 12321;
        }

        try {
            s = new Socket(host, port);
            c = connectClient(s);
        } catch (IOException exIo) {
            log.write("Could not connect to the server at " + host + ":" + port + ".  Exception: " + exIo.getMessage());
            exIo.printStackTrace();
        }

        log.write("Connected client to " + host + ":" + port);

        return c;
    }

    @Override
    public void run() {
        log.write("Running client.");
        runClient();
        log.write("Client finished running.");
    }

    abstract Connection connectClient(Socket sock);

    abstract void runClient();

    abstract void cleanClients();

}

FileClientGui.java:

package com.fileclient;

import lib.client.Client;
import lib.clientservercore.Connection;
import lib.clientservercore.Connection.ConnectionStatus;
import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.io.IOException;
import java.net.Socket;
import java.net.UnknownHostException;
import java.util.Iterator;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTabbedPane;
import javax.swing.JTextField;
import java.lang.Thread;

/**
 *
 * @author Ryan Jung
 */
public class FileClientGui extends Client {

    JFrame frmMain;
    JPanel pnlMain;
    JPanel pnlConnect;
    JTabbedPane tabConnections;
    JLabel lblHost;
    JLabel lblPort;
    JTextField txtHost;
    JTextField txtPort;
    JButton btnConnect;

    public FileClientGui(String logFile) {
        super(logFile);

        logOut("Initializing client controller...");

        frmMain = new JFrame("Client");
        pnlMain = new JPanel(new BorderLayout());
        pnlConnect = new JPanel(new FlowLayout());
        tabConnections = new JTabbedPane();
        lblHost = new JLabel("Host:");
        lblPort = new JLabel("Port:");
        txtHost = new JTextField("localhost", 10);
        txtPort = new JTextField("12321", 5);
        btnConnect = new JButton("Connect");

        frmMain.setSize(450, 600);
        frmMain.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frmMain.add(pnlMain);
        pnlMain.add(pnlConnect, BorderLayout.NORTH);
        pnlMain.add(tabConnections, BorderLayout.CENTER);
        pnlConnect.add(lblHost);
        pnlConnect.add(txtHost);
        pnlConnect.add(lblPort);
        pnlConnect.add(txtPort);
        pnlConnect.add(btnConnect);

        btnConnect.addActionListener(
            new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    String host = txtHost.getText();
                    int port = Integer.parseInt(txtPort.getText());
                    try {
                        Socket sock = new Socket(host, port);
                        FileClientConnectionGui c = (FileClientConnectionGui)(connectClient(sock));
                        tabConnections.addTab(c.getInetAddress().toString(), c.getMainPanel());
                    } catch (UnknownHostException ex) {
                        logOut("Can't find host: " + host + ".  Exception: " + ex.getMessage());
                        ex.printStackTrace();
                    } catch (IOException ex) {
                        logOut("Exception: " + ex.getMessage());
                        ex.printStackTrace();
                    }
                }
            }
        );

        frmMain.setVisible(true);

        logOut("Client controller initialized.");

    }

    public void removeConnection(FileClientConnectionGui c) {
        logOut("Removing connection: " + c.getInetAddress().toString());
        tabConnections.remove(c.getMainPanel());
        logOut("Removed connection.");
    }

    Connection connectClient(Socket sock) {
        logOut("Client controller is creating a new connection...");
        FileClientConnectionGui c = new FileClientConnectionGui(sock, getLogger(), this);
        addConnection(c);
        c.start();
        logOut("Client controller created a new connection.");
        return c;
    }

    void runClient() {
        setIsRunning(true);
        logOut("Client controller is running.");

        while (getIsRunning()) {
            cleanClients();
            try {
                sleep(500);
            } catch (InterruptedException ex) {
                logOut("Sleep interrupted.  Exception: " + ex.getMessage());
                ex.printStackTrace();
            }
        }

        logOut("Client controller stopped running.");
    }

    void cleanClients() {
        Iterator i = getConnections().iterator();
        try {
            while (i.hasNext()) {
                FileClientConnectionGui c = (FileClientConnectionGui)(i.next());
                if (c.getStatus() == ConnectionStatus.CLOSED) {
                    logOut("Removing dead client at " + c.getInetAddress().toString());
                    tabConnections.remove(c.getMainPanel());
                    removeConnection(c);
                }
            }
        } catch (Exception ex) {
            logOut("cleanClients Exception: " + ex.getMessage());
        }
    }

}

Я приму любую помощь, которую смогу получить,и я заранее благодарю вас за любые предложения, которые вы предоставляете.Я полностью сбит с толку этим.

Возможно, что больше всего сбивает с толку это (и, возможно, это дает ключ к решению проблемы?), Что я могу закомментировать другие реализации абстрактных методов (например, runClient илиconnectClient), и у меня не возникает никаких дополнительных проблем, просто такая же.Кроме того, если я добавлю директиву @Override к одной из этих команд, например:

@Override
Connection connectClient(Socket sock) {
    logOut("Client controller is creating a new connection...");
    FileClientConnectionGui c = new FileClientConnectionGui(sock, getLogger(), this);
    addConnection(c);
    c.start();
    logOut("Client controller created a new connection.");
    return c;
}

, я получу дополнительную ошибку:

com/fileclient/FileClientGui.java:96: method does not override or implement a method from a supertype

Очевидно, что равно переопределение метода из его супертипа (который является клиентом).Я попытался заменить «Клиент» на полный путь к классу (lib.client.Client), и ни одна из ошибок не изменилась вообще.

Есть что-то, что я пропустил?Что-то я не пытаюсь?

1 Ответ

9 голосов
/ 08 июля 2010

Я полагаю, это потому, что у вас есть абстрактные методы уровня пакета, которые не видны в вашем подклассе. Попробуйте сделать их защищенными.

Вот простая пара классов, которые воспроизводят задачу:

package x1;

public abstract class P1
{
    abstract void foo();
}

А потом:

package x2;

public class P2 extends x1.P1
{
    void foo() {}
}

Компиляция их дает:

P2.java:3: P2 is not abstract and does not override abstract method foo() in P1
public class P2 extends x1.P1
       ^
1 error

Обеспечение защиты foo в обоих классах устраняет проблему.

...