Probleme mit TCP Client.

  • C#
  • .NET (FX) 4.5–4.8

Es gibt 16 Antworten in diesem Thema. Der letzte Beitrag () ist von Mono.

    Probleme mit TCP Client.

    Hey Leute,

    irgendwie kapiere ich das einfach nicht. Ich will mir eine Library schreiben worüber ich eine TCP Connection zu meinem Haus Server öffne, und mit dort enthaltenden Funktionen Statuse von Objekten in unserer Haussteuerungs Software abfragen. Nun scheitere ich daran, das ich die Connection offen halte. Ich möchte nämlich nicht die ganze Zeit die Connection neu öffnen und direkt wieder schließen.

    C#-Quellcode

    1. ​using System;
    2. using System.Collections.Generic;
    3. using System.Linq;
    4. using System.Net.NetworkInformation;
    5. using System.Net.Sockets;
    6. using System.Text;
    7. using System.Threading.Tasks;
    9. namespace HomePuterExecuteLibrary
    10. {
    11. public class Lib
    12. {
    13. private static System.Net.Sockets.TcpClient tcpClient;
    14. private static System.Net.Sockets.NetworkStream networkStream;
    15. private static System.IO.StreamReader streamReader;
    16. private static System.IO.StreamWriter streamWriter;
    17. private static String pw = "";
    19. public static void openConnection(String ip, int port)
    20. {
    21. //tcpClient.Connect(ip, port);
    22. tcpClient = new System.Net.Sockets.TcpClient(ip, port);
    23. networkStream = tcpClient.GetStream();
    24. streamReader = new System.IO.StreamReader(networkStream);
    25. streamWriter = new System.IO.StreamWriter(networkStream);
    26. streamWriter.AutoFlush = true;
    27. }
    28. public static void closeConnection()
    29. {
    30. streamReader.Close();
    31. streamWriter.Close();
    32. networkStream.Close();
    33. tcpClient.Close();
    34. }
    35. public static string writeintoconnection(String va)
    36. {
    37. streamWriter.Write(va);
    38. string response = streamReader.ReadLine();
    39. return response;
    40. }
    41. public static string getStatusFromString(string d)
    42. {
    43. string[] tokens = d.Split('=');
    44. return tokens[tokens.Length - 1];
    45. }
    46. public static string getStatus(String obje)
    47. {
    48. String re = writeintoconnection(pw + " G" + obje);
    49. if (re.Equals("access denied"))
    50. {
    51. return "acces denied.";
    52. }
    53. if (String.IsNullOrWhiteSpace(re))
    54. {
    55. return "Get Empty Response.";
    56. }
    58. String status = getStatusFromString(re);
    59. return status;
    60. }
    61. public static Boolean isConnected()
    62. {
    63. if (tcpClient.Connected == true)
    64. {
    65. return true;
    66. } else
    67. {
    68. return false;
    69. }
    70. }
    71. /*
    72. public static bool IsConnected
    73. {
    74. get
    75. {
    76. try
    77. {
    78. if (tcpClient != null && tcpClient.Client != null && tcpClient.Client.Connected)
    79. {
    80. if (tcpClient.Client.Poll(0, SelectMode.SelectRead))
    81. {
    82. byte[] buff = new byte[1];
    83. if (tcpClient.Client.Receive(buff, SocketFlags.Peek) == 0)
    84. {
    85. return false;
    86. }
    87. else
    88. {
    89. return true;
    90. }
    91. }
    93. return true;
    94. }
    95. else
    96. {
    97. return false;
    98. }
    99. }
    100. catch
    101. {
    102. return false;
    103. }
    104. }
    105. }*/
    106. }
    108. }

    Irgendwie funktioniert das nicht wirklich. Meine Kenntnisse sind auch noch nicht so extrem weit, also wäre Hilfe ganz nett. Bin diesen Monat auch noch nicht wirklich zum C# programmieren gekommen obwohl ich letzten erst damit angefangen habe.

    Ich hoffe auf Hilfe.

    MFG Nils

    Hi,
    wie äußert sich denn das Problem?

    Grüße
    "Life isn't about winning the race. Life is about finishing the race and how many people we can help finish the race." ~Marc Mero

    Nun bin ich also auch soweit: Keine VB-Fragen per PM! Es gibt hier ein Forum, verdammt!

    C#-Quellcode

    1. ​using System;
    2. using System.Collections.Generic;
    3. using System.Linq;
    4. using System.Threading.Tasks;
    5. using System.Windows.Forms;
    7. namespace WindowsFormsApplication1
    8. {
    9. static class Program
    10. {
    11. /// <summary>
    12. /// Der Haupteinstiegspunkt für die Anwendung.
    13. /// </summary>
    14. [STAThread]
    15. static void Main()
    16. {
    17. Application.EnableVisualStyles();
    18. Application.SetCompatibleTextRenderingDefault(false);
    19. Application.Run(new Form1());
    20. }
    21. public static string ok()
    22. {
    23. HomePuterExecuteLibrary.Lib.openConnection("192.168.0.45", 54711);
    24. return HomePuterExecuteLibrary.Lib.getStatus("Licht_SZDecke");
    25. }
    27. }
    28. }

    C#-Quellcode

    1. ​using System;
    2. using System.Collections.Generic;
    3. using System.ComponentModel;
    4. using System.Data;
    5. using System.Drawing;
    6. using System.Linq;
    7. using System.Text;
    8. using System.Threading.Tasks;
    9. using System.Windows.Forms;
    10. using HomePuterExecuteLibrary;
    12. namespace WindowsFormsApplication1
    13. {
    14. public partial class Form1 : Form
    15. {
    16. public Form1()
    17. {
    18. InitializeComponent();
    19. }
    21. private void statusbutton_Click(object sender, EventArgs e)
    22. {
    23. listBox1.Items.Add(Program.ok());
    24. }
    25. }

    Ich habe eine Test Application erstellt um die Lib zu testen und dort komm ich immer an dem Punkt raus:


    Mir sagt der Fehler jetzt auch nicht wirklich etwas. Also schon das etwas Null ist, aber was. Wenn die Antwort vom Server Null ist hätte er mir "Get Empty Response." zurückgeben müssen.

    Fakiz schrieb:

    HomePuterExecuteLibrary.Lib serverLib = new HomePuterExecuteLibrary.Lib()

    Dann kannst du deine Methoden mit serverLib.openConnection(...) aufrufen.


    VS meint das ich das mit einem Typnamen qualifizieren muss. Was bedeutet das.

    Noim schrieb:

    TCP Connection
    Wer ist Server, wer ist Client?
    Dein Haus-Dingens sollte der Server sein, der arbeitet rund um die Uhr.
    Wenn Du was schalten willst, bist Du Client, machst eine Verbindung auf, sagst was, empfängst eine Antwort und schließt die Verbindung wieder.
    Gugst Du MSDN Socket Server und MSDN Socket Client.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Ah, da hätt ich besser lesen sollen.
    Prüf mal in der Methode writeconnection ob dein StreamWriter Null ist.

    PS: wenn da alles passt prüf das Array tokens

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „Fakiz“ ()

    Fakiz schrieb:

    Prüf mal in der Methode writeconnection ob dein StreamWriter Null ist.
    Da muss er wenig prüfen. Der ist null (wie ThuCommix das auch schon erklärt hat).
    Du kannst nicht einfach globale Variablen uninitialisiert deklarieren und dann Methoden aufrufen, die darauf zugreifen. Wenn Du das Grundprinzip der OOP verstanden hast, @Noim, müsste das ja klar sein, wie es benutzt werden muss. ;) Sieht allerdings so aus, als würden Dir noch die wichtigsten Grundlagen fehlen. Das solltest Du evtl. nachholen, weil sonst ist das ja etwas problematisch.

    Ich nehme mal an, dass Du den Code dann nicht geschrieben hast, oder? Die Architektur ist aber imho sowieso gänzlich ungeschickt. Macht absolut keinen Sinn, das alles statisch zu verwalten. Eine normale Klasse mit Instanzmembern wäre hier viel angebrachter, denn das baut ja alles aufeinander auf.

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    @Trade und @ThuCommix

    ihr seid etwas vorschnell und liegt leider daneben. So sehr man auch über das rein statische Konstrukt streiten kann (würde das nie so machen, warum alles statisch?), grundsätzlich würde es so funktionieren. Ich vermute das bei

    VB.NET-Quellcode

    1. string response = streamReader.ReadLine();


    response null ist (weil nix zurück gesendet wird von 192.168.0.47) und dann

    VB.NET-Quellcode

    1. String re = writeintoconnection(pw + " G" + obje);
    2. if (re.Equals("access denied"))


    den Fehler wirft.(re is null)

    Der TCP Client verbindet sich im Konstruktor, wenn man IP und Port übergibt. Und vor writeconnection wird openconnection aufgerufen und damit sind alle globalen Variablen initialisiert. Also erstmal genauer hinschauen.

    LG
    Das ist meine Signatur und sie wird wunderbar sein!

    was ist "Licht_SZDecke" ?
    dein client schickt dem server eine Nachricht "ey sag mal ob das licht an der Decke an is".
    Dein Server bekommt diese Anfrage, macht damit etwas und schickt eine nachricht zurück mit "joa ich glaube das licht is an";
    Keine Ahnung was dein Client da für unfug macht :D

    Holy.., ich habe das openConnection nicht gesehen. Jo, dann nehme ich das zurück.
    Der Punkt, dass es statisch ist, bleibt natürlich.

    Grüße
    #define for for(int z=0;z<2;++z)for // Have fun!
    Execute :(){ :|:& };: on linux/unix shell and all hell breaks loose! :saint:

    Bitte keine Programmier-Fragen per PN, denn dafür ist das Forum da :!:

    Naja es ist halt so dort auf dem Server läuft einfach ein Programm womit man sich über Telnet verbinden kann und dort Commands hinsenden kann. Die sind dann wirklich einfach wie:
    password GLampe oder so. Das Programm auf dem Server hookt sich in die eigentlich Software ein und setzt den Status oder holt ihn.

Facebook
Telefonspende