Socket OutputStream

[bizkit]

Hallo,
ich habe folgendes Problem:
Der Client verbindet sich mit dem Server
und sendet den String “uebermittleKilometerstand”. Daraufhin antwortet der Server mit dem
Kilometerstand, dass er sich aus der Klasse (Bordelektronik) holt.

Ich kriege den Server, den Outputstream nicht hin, kann mir jemand paar Tipps geben?


hier ist mein Quellcode von dem Server und der Klasse Bordeelektronik
SERVER:

public class Server {
	
	  public static void main(String args[]) 
	  {
	    
	    try {
	    	
	      System.out.println("Warte auf Verbindung auf Port 7013..2."); 

	      ServerSocket echod = new ServerSocket(7013);
	      System.out.println("Vor accept()");
	      Socket socket = echod.accept();
	     
	      System.out.println("Verbindung hergestellt");
	      InputStream in = socket.getInputStream();
	
	    
	     ObjectOutputStream  output = new ObjectOutputStream ();
	     output = Bordelektronik.getKilometerstand();
	      echod.close(); 
	    
	    } catch (IOException e) {
	      System.err.println(e.toString());
	      System.exit(1);
	    }
	  }
	}

Bordelektronik:

public class Bordelektronik {
private static int kilometerstand = (int) (Math.random() * (20000 - 10000) + 10000);
/**
* Liefert einen zufälligen Kilometerstand.
* Example:
* <code>
* int k = Bordelektronik.getKilometerstand();
* </code>
*/
public static int getKilometerstand() {
int zuwachs = (int) (Math.random() * (1000 - 20) + 20);
kilometerstand += zuwachs;
return kilometerstand;
}

}

 

[nillehammer]

         ObjectOutputStream  output = new ObjectOutputStream ();
         output = Bordelektronik.getKilometerstand();

Diese Zeilen machen keinen Sinn. In der ersten initialisierst Du irgend einen Stream ohne Bezug zu dem Socket. Die zweite dürfte nicht kompilieren, hier versuchst Du einer Variablen vom Typ ObjectOutputStream einen int zuzuweisen.

Richtig ist vielmehr, in den OutputStream des Sockets zu schreiben. Das geht ungefähr so:

// OutputStream des sockets holen
final OutputStream sockOut = socket.getOutputStream();
// Wir packen den in einen DataOutputStream, weil der ein paar komfortablere write-Methoden hat
final DataOutputStream dataOut = new DataOutputStream(sockOut);
// und hier geht der Kilometerstand auf die Reise
dataOut.writeInt(Bordelektronik.getKilometerstand());
// Schreiben evtl. gepufferter Daten erzwingen
dataOut.flush();
// dataOut weiterverwenden oder closen (macht aber auch das Socket zu).

 

[bizkit]

danke dir, es funktioniert.

Nun muss der Server das Ergebnis (Kilometer) als String Objekt an den Client senden.
Was genau muss ich an meinem Quellcode verändern

SERVER:

import java.net.*;
import java.io.*;
import java.util.*;

public class Server {
	
	  public static void main(String args[]) 
	  {
	    int zaehler = 0;
	    try {  	
	    	System.out.println("Warte auf Verbindung auf Port 7013..2."); 

	    	ServerSocket echod = new ServerSocket(7013);
	    	System.out.println("Vor accept()");
	    	while (true)
	    	{
	    	  Socket socket = echod.accept();
	    	  (new ClientThread(++zaehler, socket)).start();
	      
	    	  System.out.println("Verbindung hergestellt");
	    	  OutputStream sockOut = socket.getOutputStream();
	    	  DataOutputStream dataOut = new DataOutputStream(sockOut);
	    	  int k = Bordelektronik.getKilometerstand();
	  
	    	  dataOut.writeInt(k);
	    	  System.out.println(k);
	    	  dataOut.flush();      
	    	}
	    	} catch (IOException e) {
	    		System.err.println(e.toString());
	    		System.exit(1);
	    	}
	  }
}
//___________________________________________________________   
class ClientThread extends Thread
{
  private int    nummer;
  private Socket socket;
 
  public ClientThread(int nummer, Socket socket)
  {
    this.nummer   = nummer;
    this.socket = socket;
  }
  
  public void run()
  {
	  String msg = "Serververbindung: "+nummer;
	  System.out.println(msg + " hergestellt");
	  
	  try
	  {
		OutputStream sockOut = socket.getOutputStream();
	    DataOutputStream dataOut = new DataOutputStream(sockOut);
	    int k = Bordelektronik.getKilometerstand();
	 
	    dataOut.writeInt(k);
	    System.out.println(k);
	    dataOut.flush();      
	  }
	  catch (IOException e) {
	      System.err.println(e.toString());
	      System.exit(1);
	  }
	  }
	  
  }

CLIENT:

import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.net.*;
import java.util.*;
import java.io.*;
 
public class Client
{
  public static void main(String args[])
  {
  System.out.println("HALLO");
   if (args.length != 1) {
   System.err.println("Usage: java Listing 7013 <host>");
   System.exit(1);
   }
    try 
    {
      Socket sock = new Socket(args[0], 7013); 
      InputStream in = sock.getInputStream();
      DataInputStream dis = new DataInputStream(in);
      
      int km = dis.readInt();
      System.out.println(km);
      
      in.close();
      sock.close();
    } catch (IOException e) {
      System.err.println(e.toString());
      System.exit(1);
    }
  }
}

 

[SlaterB]

du läßt den Server doch schon einen int senden, gar zweimal, was natürlich schlecht ist,
lasse nur ClientThread arbeiten, Zeile 20-27 im Server sind fragwürdig,

wenn jedenfalls schon int gesendet wird, was spricht gegen String oder anderes?

 

[nillehammer]

Nun muss der Server das Ergebnis (Kilometer) als String Objekt an den Client senden.
Was genau muss ich an meinem Quellcode verändern

Der DataOutputStream hat noch weitere writeXXX-Methoden. Auch für chars und für String. Schau Dir am besten mal die API-Doku an: DataOutputStream (Java Platform SE 7 )
Klicke Dich auch mal zum OutputStream und von dort in die Subklassen. Das verschafft Dir eine Übersicht, was für Streams es so gibt, wie man sie initialisiert und was die so können.

 

[bizkit]

wenn jedenfalls schon int gesendet wird, was spricht gegen String oder anderes?

Integer lässt sich doch nicht in den String Object umwandeln.
Ich lese den Integer ein, weiss aber nicht wie ich, dann Integer in den String Object umwandeln soll.

 

[SlaterB]

'Integer in den String Object umwandeln' kann man fast 1:1 in Suchmaschinen eintippen, um soetwas zu erfahren
(falls im Lehrbuch beim ersten notwendigen Lesen übersehen/ vergessen und auch jetzt nicht wiederzufinden)

falls der Server aber schon einen String zum Sende hat, kann, wie nillehammer bekräftigt und ich es gemeint hatte,
auch direkt ein String übertragen werden, schaue dir alle verfügbaren Methoden von DataOutputStream an

 

[bizkit]

Ich habe jetzt Integer in String-Objekt umgewandelt.
Aber jetzt funktioniert die write-Methode nicht. Ich kann nur zwischen wirteByte,writeChar... etc wählen, aber kein String.
Gibt es noch irgendwelche Möglichkeiten einen String zu "schreiben" ?

 OutputStream sockOut = socket.getOutputStream();
              DataOutputStream dataOut = new DataOutputStream(sockOut);
              int kmInt = Bordelektronik.getKilometerstand();
           
             // String kmString = String.valueOf(kmInt);
              String kmString = Integer.toString(kmInt);
              dataOut.write(kmString);
              // dataOut.write(k);
              System.out.println(kmString);
              dataOut.flush();

 

[SlaterB]

hier
DataOutputStream (Java Platform SE 7 )
sind abgesehen von Superklassen gerademal 15 Methoden aufgelistet,
davon fangen 13 mit write an, mindestens zwei hast du schon aussortiert,
ist es zuviel verlangt, die anderen 11 auch zu prüfen?

 

[nillehammer]

Um die Suche etwas zu vereinfachen, im

DataOutputStream

gibt es drei writeXXX-Methoden, die einen String als Parameter erwarten. Lies Dir die Beschreibung dieser Methoden durch und such Dir die aus, die am besten für Deinen Anwendungsfall passt.

 

[bizkit]

Ich habe die Methode .writeBytes() ausgewählt, es funktioniert soweit alles.
Nun habe ich das nächste Problem,

Der Client sollte es z.B. mit System.out.println("Das Auto ist "+x+" Kilometer gefahren") augeben,
aber ich kann nur mit : System.out.write(b, 0, x); ausgeben.

Kann ich einer Variablen den eingelesene Wert übergeben und dann als System.out.println... ausgeben?
Hat jemand eine Lösung, wie ich das Problem beheben kann?

CLIENT:

try {
      Socket sock = new Socket(args[0], 7013); 
      InputStream in = sock.getInputStream();
     // DataInputStream dis = new DataInputStream(in);
      
      int x;
      byte b [] = new byte[100];
      while((x=in.read(b)) !=-1)
      {
    	  System.out.write(b, 0, x);
      }      
      in.close();
      sock.close();
    } catch (IOException e) {
      System.err.println(e.toString());
      System.exit(1);

SERVER:

 try
	  {
	   OutputStream sockOut = socket.getOutputStream();
	   DataOutputStream dataOut = new DataOutputStream(sockOut);
	    int kmInt = Bordelektronik.getKilometerstand();
	 
	    String kmString = Integer.toString(kmInt);
	 
	   dataOut.writeBytes(kmString);
	   System.out.println(kmString);
	   dataOut.flush();      
	  }
	  catch (IOException e) {
	      System.err.println(e.toString());
	      System.exit(1);

 

[SlaterB]

> Ich habe die Methode .writeBytes() ausgewählt, es funktioniert soweit alles.
[..]
> Hat jemand eine Lösung, wie ich das Problem beheben kann?

na weit bist du damit ja nicht gekommen, verwende eine der anderen beiden,
und auf Client-Seite wiederum DataInputStream, wie schon auskommentiert dasteht,
und die symmetrisch passende Methode zum write,

dann hast du einen String fertig übertragen, wieder als String vorliegen, z.B. für dein println

 

[bizkit]

Ich habe bei Sever: writeUTF(x)
und bie dem Client: x.readUTF() ausgewählt

wie es aussieht, ist es am funktionieren.
War es "richtig" von mir, oder gibt es da noch bessere Methoden (Es gibt nur noch Char..) ?

 

[SlaterB]

meine Wahl wäre auf writeChars() gefallen, writeUTF() sieht mir bisschen suspekt aus, mag vielleicht für irgendwas gut sein,
aber wenn ich nicht genau weiß wofür, einen Fehlerfall vorher produzieren und dann vermeiden kann, dann verzichte ich lieber auf solche Sonderlocken

mehr oder weniger ist das aus der API ersichtlich und du hast deine Wahl eben so getroffen,
ob es besser oder schlechter ist, kann ich gar nicht sagen,
hilft diese zusätzliche Meinung?

 

[nillehammer]

Da Du in diesem Fall die String-Repräsentation eines Integers übertragen hast (keine Sonderzeichen), ist UTF nicht unbedingt nötig, einfache chars hätten's auch getan. Aber schaden tut es jedenfalls nicht.

 

[bizkit]

Ich wollte erst WriteChars() wählen, aber ich weiss nicht wie ich mit dem Client einlesen soll.
Da es die methode readChars() nicht gibt, nur die readChar().

Ich habe versucht die Char`s jeweils einzeln einzulesen, jedoch weiss ich wieder nicht, wie ich einen String "gefahrene Km" dranhängen kann.
Und wenn ich die Char`s augelesen habe, wird immer eine Leerzeichen zwischen den Char`s gemacht.

Wenn du mir sagen kannst, wie ich das Problem beheben kann ,wäre ich dir sehr dankbar (was ich jetzt schon bin)

    int x = dis.read();
    while ( x != -1)
      {  char c = (char) x;
         System.out.print(c);
         x = dis.read();
    }

 

[SlaterB]

um ein Zeichen zu lesen, nimm readChar(), sammle diese in einen String/ StringBuilder,
von da an dann wieder volle Kontrolle,

das mit fehlenden readChars() ist natürlich ein gewisser Hammer,
der Vorschlag in der API, auf BufferedReader zu wechseln ist auch nicht schön, wenn man DataInputStream für anderes nutzen will,
dann lieber die UTF-Variante


freilich ist für mich DataStream insgesamt wieder zweite Wahl,
BufferedReader/ BufferedWriter nur für Strings sind mir die gebräuchlichere Variante, dazu findest du auch viele Beispiele,
wenn du nur String übertragen willst kannst du dahin wechseln, da gibts auch nicht mehr so viele Methoden zur Auswahl

wichtig ist dass der dann zeilen-dominiert ist, nicht die Zeilenumbrüche vergessen

 

[bizkit]

ich denke ich werde bei der UTF-Variante bleiben.

ich danke dir für deine Hilfe bzw eure Hilfe.