Funktions Programmierung

[Guest]

Hallo liebes Java-Team ,

Ich bin ein absoluter Java Anfänger...

Ich versuche z.Z. einen Chat zu Programmieren habe kriege die Privat funktion einfach nicht hin.

Mal ein ausschnitt aus meinem Code in der Client.java:

if(type.equals("privatein"))
{
String message = msg;
String from = "Jefrey";
if(msg.indexOf(":") != -1)
{
from = msg.substring(0, msg.indexOf(":"));
message = msg.substring(msg.indexOf(":")+1);
}
if(chatframe.isVisible())
{
chatframe.chat.append("Privat (von " + from + "): " + message + "\n");
}
}
else if(type.equals("privateout"))
{
String message = msg;
String to = "Jefrey";
if(msg.indexOf(":") != -1)
{
to = msg.substring(0, msg.indexOf(":"));
message = msg.substring(msg.indexOf(":")+1);
}
if(chatframe.isVisible())
{
chatframe.chat.append("Privat (an " + to + "): " + message + "\n");
}
}

------------------------------------------------------------------------------------------------------------------------------------------

Das ist der ausschnitt aus der server.java:

else if(command.equals("/p"))
{
String[] infos = params.split(":");

if(infos.length > 1)
{
System.out.println("[" + getTimeString() + "] Command \"" + command + "\" (Parameters \"" + params + "\") from " + client + " (" + loggedUsers.get(client) + ")");

String nick = infos[0];
String msg = infos[1];

if(userOnline(nick))
{
sendToUser(getClientWithNick(nick), "privatein:" + loggedUsers.get(client) + ":" + msg);
sendToUser(client, "privateout:" + nick + ":" + msg);
}
}
else
sendToUser(client, "privatein:Funktion nicht ausgeführt!");
}
--------------------------------------------------------------------------------------------------------------------------------------

Es kommt zwar keine Fehlermeldung nur kann man die Funktion nicht nutzen.

Habt ihr eine Idee?

MfG

 

[SlaterB]

es ist nicht klar, was du mit 'Funktion' meinst,
wer sie wann wo nutzt, was daran gehen soll/ könnte/ aktuell nicht geht/ wie sie erkennbar ist/ ..

es ist nicht klar wo irgendwas mit 'Privat' zu tun hat, was der Code mit der Sache zu tun hat (außer dass da auch privat auftaucht),
usw usf..

edit:
wenn alles nur von Buttons, Aktionen, übertragenen Strings usw. abhängt,
dann baue in dein Programm doch ein Log ein:
System.out.println("hier Client, Button x gedrückt, starte Aktion y");

System.out.println("hier Server, empfange Nachricht 17 mit Inhalt a von Client b");
System.out.println("hier Server, in Nachricht 17 erkenne ich folgende Kommandos c, schalte daher Aktion d ein");

usw.

alles was in deinem Programm passiert kannst du auf den kleinsten Millimeter genau verfolgen

 

[Guest]

es ist nicht klar, was du mit 'Funktion' meinst,
wer sie wann wo nutzt, was daran gehen soll/ könnte/ aktuell nicht geht/ wie sie erkennbar ist/ ..

es ist nicht klar wo irgendwas mit 'Privat' zu tun hat, was der Code mit der Sache zu tun hat (außer dass da auch privat auftaucht),
usw usf..

edit:
wenn alles nur von Buttons, Aktionen, übertragenen Strings usw. abhängt,
dann baue in dein Programm doch ein Log ein:
System.out.println("hier Client, Button x gedrückt, starte Aktion y");

System.out.println("hier Server, empfange Nachricht 17 mit Inhalt a von Client b");
System.out.println("hier Server, in Nachricht 17 erkenne ich folgende Kommandos c, schalte daher Aktion d ein");

usw.

alles was in deinem Programm passiert kannst du auf den kleinsten Millimeter genau verfolgen

Naja ich würde mal sagen das man das Privatschreiben nutzen kann wann man will mit:
/p NICK:text nur mein Problem ist das es nicht im Chatframe angezeigt wird...
kann mir jmd. helfen?

 

[SlaterB]

wie gesagt, ist ein denkbares Vorgehen zum Finden des Problems das was ich beschrieben habe,
was spricht dagegen?

ansonsten ist aus so wenig Code kaum die Interaktion von Client/ Server zu überblicken

 

[Guest]

Ich kann deinem Beispiel leider nicht ganz folgen...


Warum soll ich das noch zusätzlich einbauen?

ich dachte vllt. schon das anstatt from bzw. to = "Jefrey"; durch getClientWithNick o.ä. ersetzt werden muss?

MfG

 

[SlaterB]

du sollst das einbauen, damit du verstehst, was dein Programm tut,
jedes if/ else ist einfacher zu verstehen als das Gesamtprogramm

zu Jefrey/ getClientWithNick kann ich nix sagen