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
  • Так же буду благодарен за советы по улучшению моего стиля программирования. Commented 5 янв. 2020 в 13:59
  • Грязное решение которое мне помогло когда-то, передать контроллер как переменную, в другой класс, где она являлась статической и я мог обратиться к ней из любого пакета и класса. Это будет работать, но я знаю что это неправильно. Commented 5 янв. 2020 в 14:03
  • Добавьте гетер и сетер для этих переменных Commented 5 янв. 2020 в 14:05
  • @PerfectVoyage, пытался так сделать еще давно, все так же NullPointerExecption.... И даже не в классе сервера, а в классе контроллера :/ Commented 5 янв. 2020 в 14:06
  • 1
    @Shad0wCloud покажите пример передачи ? По поводу кода - Название класса ВСЕГДА с большой буквы. Помнится вы вчера задавали этот же вопрос, а ошибки у вас остались Commented 5 янв. 2020 в 14:08

1 ответ 1

0

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

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

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

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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