0

Делаю чат на javа, но столкнулся с проблемой. Мне нужно, чтобы сервер пользователя получал сообщение от централизованного сервера и выводил все сообщения на экран. По некоторым обстоятельствам, я не могу в классе сервера обращаться к методам/полям контроллера чата, поэтому мне пришла идея передавать какую-то статичную переменную, по типу Arraylist, контроллеру. Я еще не очень опытен в Java, поэтому вопрос в том, настолько это эффективно и как лучше всего это сделать?
Код моего сервера:

package sample;

import java.awt.*;
import java.io.IOException;
import java.net.ServerSocket;
import java.net.Socket;
import java.net.URLClassLoader;
import java.util.Scanner;

import static sample.chatcontroller.ClientPort;
import static sample.chatcontroller.ServerPort;

class ClientServer implements Runnable {

    @Override
    public void run() {
        try (ServerSocket serverSocket = new ServerSocket(ClientPort)) {
        while (true) {
                Socket socket = serverSocket.accept();
                Scanner scanner = new Scanner(socket.getInputStream());
                ServerPort = Integer.parseInt(scanner.nextLine());
                String msg = scanner.nextLine();//само сообщение
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Код моего контроллера:

package sample;

import java.io.IOException;
import java.io.PrintWriter;
import java.net.*;

import javafx.application.Platform;
import javafx.fxml.FXML;
import javafx.fxml.FXMLLoader;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.TextArea;
import javafx.scene.control.TextField;
import javafx.stage.Stage;

public class chatcontroller{
    public static String name = "client";
    public static int ClientPort = 6777;
    public static int ServerPort = 7777;
    @FXML
    private TextField messagefield;

    @FXML
    private Button sendbutton;

    @FXML
    private Button exitbutton;

    @FXML
    private Button configbutton;

    @FXML
    private Button onlyadminbutton;

    @FXML
    public TextArea textarea;

    @FXML
    void initialize() {
        new Thread(new ClientServer()).start();
        exitbutton.setOnAction(event -> {
            exitbutton.getScene().getWindow().hide();
            FXMLLoader loader1 = new FXMLLoader();             //выход из чата
            loader1.setLocation(getClass().getResource("sample.fxml"));
            try {
                loader1.load();
            } catch (IOException e) {
                e.printStackTrace();
            }
            Parent root1 = loader1.getRoot();
            Stage stage1 = new Stage();
            stage1.setScene(new Scene(root1));
            stage1.showAndWait();
        });
        sendbutton.setOnAction(e -> {
            String input = messagefield.getText();
            if (!input.isEmpty()) {
                SendMSG(input);
                Platform.runLater(()-> textarea.appendText("Пользователь " + name + " написал " + input + "\n"));
                messagefield.clear();
            }
        });
    }
}
10
  • Так же буду благодарен за советы по улучшению моего стиля программирования. 5 янв 2020 в 13:59
  • Грязное решение которое мне помогло когда-то, передать контроллер как переменную, в другой класс, где она являлась статической и я мог обратиться к ней из любого пакета и класса. Это будет работать, но я знаю что это неправильно. 5 янв 2020 в 14:03
  • Добавьте гетер и сетер для этих переменных 5 янв 2020 в 14:05
  • @PerfectVoyage, пытался так сделать еще давно, все так же NullPointerExecption.... И даже не в классе сервера, а в классе контроллера :/ 5 янв 2020 в 14:06
  • 1
    @Shad0wCloud покажите пример передачи ? По поводу кода - Название класса ВСЕГДА с большой буквы. Помнится вы вчера задавали этот же вопрос, а ошибки у вас остались 5 янв 2020 в 14:08

1 ответ 1

0

Решение проблемы - передать контроллер классу сервера,

  1. В классе сервера объявить поле static Chatcontroller ccr;
  2. В методе initialize() у контроллера, передать контроллер через класс ClientServer.ccr = this
    В результате передачи контроллера, можно будет обратиться к TextArea в классе контроллера ccr.textarea.appendText(msg);

Спасибо пользователю @PerfectVoyage за решение проблемы.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.