Listview не отображает в реальном времени наблюдаемые изменения списка правильно - PullRequest
0 голосов
/ 22 апреля 2019

У меня есть проблема с частью моего приложения, которая работает неправильно, как я хочу, у меня есть список просмотра Java, который похож на журнал, я пытаюсь создать простой журнал с ним. посмотрите на код,

Я попытался добавить Задачу, которая записывает журнал с функцией writeLog каждую секунду, и это сработало (просмотр списка обновляется правильно с новой строкой). Второй код такой же, но без Задачи, и он не работает так, как я хочу, в этом случае writeLog будет вызываться другими классами. При вызове writeLog («любая строка ..») из других классов список Observable изменится правильно, и файл (log) также изменится, но просмотр списка не хочет показывать новую строку, добавленную в журнал.

public class ControllerServer implements Initializable {

    @FXML
    public ListView myListView;

    private ObservableList<String> logList = FXCollections.observableArrayList();

    @Override
    public void initialize(URL location, ResourceBundle resources) {
        try {
            fillLogList();
            MyTask task = new MyTask();
            Timer timer = new Timer(true);
            timer.scheduleAtFixedRate(task,4000,4000);
        } catch (IOException e) {
            e.printStackTrace();
        }

        if (logList!= null) {
            for (int i = 0; i < logList.size(); i++) {
                System.out.println(logList.get(i));
            }
         logList.addListener((javafx.beans.Observable observable)->{
             System.out.println("changed list");
         });
        }
        myListView.setItems(logList);
    }
    //METHODS
    public void fillLogList() throws IOException {
        BufferedReader reader = new BufferedReader(new FileReader("samplefile1.txt"));
        String line;
        while((line = reader.readLine())!=null) {
            logList.add(line);
        }
    }
    public void writeLog(String log) throws IOException {
        BufferedWriter writer = new BufferedWriter(new FileWriter("samplefile1.txt", true));
        writer.write(log);
        if(logList!=null){
            logList.add(log);
            System.out.println("i'm writing this log inside the file, log say : " + log);
        }else
        writer.newLine();
        writer.close();
    }

    class MyTask extends TimerTask {
        @Override
        public void run(){
            try {
                writeLog("Hello, i'm working inside this task, why?");
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
}
public class ControllerServer implements Initializable {

    @FXML
    public ListView myListView;

    private ObservableList<String> logList = FXCollections.observableArrayList();

    @Override
    public void initialize(URL location, ResourceBundle resources) {
        try {
            fillLogList();
        } catch (IOException e) {
            e.printStackTrace();
        }

        if (logList!= null) {
            for (int i = 0; i < logList.size(); i++) {
                System.out.println(logList.get(i));
            }
         logList.addListener((javafx.beans.Observable observable)->{
             System.out.println("changed list");
         });
        }
        myListView.setItems(logList);
    }
    //METHODS
    public void fillLogList() throws IOException {
        BufferedReader reader = new BufferedReader(new FileReader("samplefile1.txt"));
        String line;
        while((line = reader.readLine())!=null) {
            logList.add(line);
        }
    }
    public void writeLog(String log) throws IOException {
        BufferedWriter writer = new BufferedWriter(new FileWriter("samplefile1.txt", true));
        writer.write(log);
        if(logList!=null){
            logList.add(log);
            System.out.println("i'm writing this log inside the file, log say : " + log);
        }else
        writer.newLine();
        writer.close();
    }

}
public class Server extends Application {
    private static ServerLog serverLog = new ServerLog();
    private static ControllerServer  controllerServer = new ControllerServer();


    public static void main(String[] args) throws IOException {
        Runnable r = () -> {
            new File("users").mkdir();
            try {
                ServerSocket socket = new ServerSocket(8189);

                while(true) {
                    Socket s = socket.accept();
                    ServiceHandler client = new  ServiceHandler(s);
                }

            }catch(IOException e){e.printStackTrace();}
        };
        new Thread(r).start();
        controllerServer.writeLog("server starting..");
        launch(args);
    }

    @Override
    public void start(Stage primaryStage) throws Exception {
        Parent root = FXMLLoader.load(getClass().getResource("View/server.fxml"));
        primaryStage.setScene(new Scene(root, 800, 600));
        primaryStage.setTitle("Server Log");
        primaryStage.show();
    }

}

package server;

import server.Controller.ControllerServer;
import server.Model.*;

import java.io.*;
import java.net.Socket;

import static common.ServiceID.*;
import static common.ServiceID.FAILURE_RESPONSE;

public class ServiceHandler extends Thread{

    private static Object lock = new Object();
    private static ServerLog serverLog = new ServerLog();
    private static ControllerServer controllerServer = new ControllerServer();
    private Socket s;

    public ServiceHandler(Socket s){
        this.s = s;

        this.start();
    }

    public void run()
    {
        try
        {
            controllerServer.writeLog("client address "+s.getInetAddress());
            InputStream inputStream = s.getInputStream();
            int servId = inputStream.read();
            OutputStream outputStream = s.getOutputStream();
            Thread t;
            switch(servId) {
                case WRITE_EMAIL:
                    controllerServer.writeLog("Writing Email, id : " + WRITE_EMAIL);
                    outputStream.write(SUCCESS_RESPONSE);
                    t = new Thread(new EmailReceiver(s,lock, servId));
                    t.start();
                    break;

                case READ_ALL_EMAILS:
                    controllerServer.writeLog("Loading users list, id : " + READ_ALL_EMAILS);
                    outputStream.write(SUCCESS_RESPONSE);
                    t = new Thread(new EmailReader(s));
                    t.start();
                    break;

                case USER_LIST:
                    controllerServer.writeLog("Caricamento lista utenti, id : " + USER_LIST);
                    outputStream.write(SUCCESS_RESPONSE);
                    t = new Thread(new UserGetter(s));
                    t.start();
                    break;

                case LOGIN:
                    controllerServer.writeLog("Login, id " + LOGIN);
                    outputStream.write(SUCCESS_RESPONSE);
                    t = new Thread(new UserLogin(s));
                    t.start();
                    break;

                case DELETE_EMAIL:
                    controllerServer.writeLog("delete email, id " + DELETE_EMAIL);
                    outputStream.write(SUCCESS_RESPONSE);
                    t = new Thread(new EmailDelete(s));
                    t.start();
                    break;
                default:
                    controllerServer.writeLog("Error");
                    outputStream.write(FAILURE_RESPONSE);
                    break;

            }


        }
        catch(Exception e)
        {
            e.printStackTrace();
        }
    }

}

1 Ответ

1 голос
/ 22 апреля 2019

Проблема в том, что вы создаете новый ControllerServer в других ваших классах.

Когда вы звоните controllerServer.writeLog() из этих классов, вы на самом деле пишете на совершенно другой List, чем тот, который привязан к вашему ListView.


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

Например, вы можете обновить класс ServiceHandler следующим образом:

public class ServiceHandler extends Thread {
  private static Object lock = new Object();
  private static ServerLog serverLog = new ServerLog();
  private Socket s;
  private ControllerServer controllerServer;  // Do NOT instantiate it here!

  // Pass your single ControllerServer to the constructor
  public ServiceHandler (Socket socket, ControllerServer controllerServer) {
    this.s = socket;
    this.controllerServer = controllerServer;

    this.start();
  }
}

Теперь, когда вы позвоните controllerServer.writeLog() позже, вы будете делать это на правильном экземпляре ControllerServer.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...