Frage wegen Chatbefehlen...

[Akkon]

Hallo ihr lieben,

ich versuche gerade krampfhaft Chatbefehle für meinen Chat einzubauen, was momentan Serverseitig noch so aussieht:

        for (i=0; i<connections.size(); i++)
        {
            you = (connection) connections.elementAt(i);
			if(msg.startsWith("/")) {
			you.out.println("Die Funktion "+msg+" steht dir hier nicht zur Verfügung.");
			} else if(msg.startsWith("/w")) {
			you.out.println("TESTUNG");
			} else {
            you.out.println(msg);
			}
        }

Das Problem ist dabei nur, dass er IMMER auf die erste Variante zurückgreift, dass heißt, "Die Fuktion..... zur Verfügung."

Wie schaffe ich es, dass das System bei "/w" TESTUNG ausgibt oder ein Popup öffnet?

Danke schonmal für Eure Hilfe

Gruß Akkon

 

[troll]

einfachste wäre wohl RegEx ... warum du das allerdings in der broadcast-metode machst verstehe ich nicht

 

[Akkon]

Ich gebe zu, es war ein vorgefertigter Chat, jedoch möchte ich aus diesem eben einen vernünftigen Chat bauen.

Wie setze ich denn regEx ein? Würdest Du mir das bitte einmal erklären?

 

[Gastredner]

Du musst die Reihenfolge umkehren, in der du den Beginn des Strings analysierst. Der String "/w" beginnt mit einem "/", deshalb schlägt in deiner Implementierung direkt die erste Bedingung zu und du erhälst die immerselbe Ausgabe.

 

[Nardian]

oder du überprüfst zuerst die längeren befehle, und erst wenn keiner von ihnen zugetroffen ist, dann den "/"

        for (i=0; i<connections.size(); i++)
        {
            you = (connection) connections.elementAt(i);
			if(msg.startsWith("/w")) {
			you.out.println("TESTUNG");
			} else if(msg.startsWith("/")) {
			you.out.println("Die Funktion "+msg+" steht dir hier nicht zur Verfügung.");
			} else {
		        you.out.println(msg);
			}
        }

oooder wenn du java7 verwendest könnte sowas hier auch funktionieren

String cmd = eingabe.split(" ")[0];
switch(cmd){
   case "/": 
      break;
   case "/w":
      break;
   default:
      break;
}

falls du kein java7 verwendest, kannst du statt dem switch natürlich bei deinen ifs bleiben, mit dem unterschied dass du nicht .startsWith verwendest, sondern .equals

MfG

 

[Pentalon]

Wie Gastredner schon sagte, ist die Reihenfolge hier entscheidend.
Wenn ich zuerst "/" prüfen trifft das auch zu wenn ich ein "/w" habe,
also muss ich das vorher prüfen, auch wenn ich es mit SWITCH/CASE mache.

switch(cmd){
   case "/w": 
      break;
   case "/":
      break;
   default:
      break;
}

Pentalon

 

[Nardian]

bei nem switch / case ist die sache etwas anders...

jeder case MUSS einzigartig sein. das heißt wenn der String "/" ist, dann matcht das "/", wenns ein "/w" ist, dann matcht das NUR "/w", und nicht "/" - kommt also einem equals, statt einem startswith gleich.

daher auch die spielerei mit dem

String cmd = eingabe.split(" ")[0];

 

[Mujahiddin]

Ich mach das in etwa so:

/** executes a command 
@return true if the command was executed successfully, false otherwise */
public boolean command(String text) {
	final String text = "/doSomething";
	final String command = text.toLowerCase();
	getTextPane().append(">" + text + "\n");
	
	boolean success = false;
	
	if( command.startsWith( "/name " ) ) // "/name " mit Leerzeichen, weil der Befehl "/name" 	ungültig ist (zumindest in meinem Programm)
		success = name( text );
	else if( command.equals( "/clear" ) )
		success = clear( text );
	else if( command.equals( "/quit" ) )
		quit( text );
	else if( command.equals( "/ping" ) )
		success = ping( text );
	else if( command.equals( "/time" ) )
		success = time( text );
	else if( command.startsWith( "/help" ) )
		success = help( text );
	else if( command.equals( "/log" ) )
		success = log( text );
	else
		getTextPane().append( "console: " + getString( "NO_SUCH_COMMAND" ) );
	return success;
}

getString

ist hierbei eine Methode, die auf properties-Dateien zugreift. Du kannst natürlich auch sowas machen wie

getTextPane().append( "console: Dieser Befehl existiert nicht." );

 

[Pentalon]

bei nem switch / case ist die sache etwas anders...

jeder case MUSS einzigartig sein. das heißt wenn der String "/" ist, dann matcht das "/", wenns ein "/w" ist, dann matcht das NUR "/w", und nicht "/" - kommt also einem equals, statt einem startswith gleich.

daher auch die spielerei mit dem

String cmd = eingabe.split(" ")[0];

Ah, gut zu wissen. Das sah ich bisher anders.

Danke sagt,
Pentalon

 

[Akkon]

Im Moment habe ich die Textausgabe auf den Client beschränkt..... sieht so aus:

    public boolean action(Event e, Object what)
    {
        if (e.target==inputfield)
        {
            String inp=(String) e.arg;
			if (inp.equals("/clear")) {
            outputarea.setText("");
            inputfield.setText("");
            return true;
			} else if (inp.equals("/list "+inp)) {
            userList.add(inp);
            inputfield.setText("");
            return true;
			} else {
			out.println(inp);
			inputfield.setText("");
			return true;
			}
        }
 
        return false;
    }

Wie baue ich das denn in Eure Vorschläge um? Kann ich von der Konsole nicht iwie ganz weggehen? Weil ich kann ja auch kein "teilfett" schreiben für einzelne Worte ermöglichen

Mag mir jemand einzelunterricht geben?

 

[Akkon]

Wie ist das eigentlich besser?
Die ChatText Kreationskomponente Clientseitig oder Serverseitig?

 

[troll]

vorsicht bei switch w/ string ... der compiler wandelt das

switch(string1)
{
case "x":
...
break;
case "y":
...
break;
}

in sowas hier um

switch(string1.hashCode())
{
case 123456:
...
break;
case 654321:
...
break;
}

ist ähnlich sinnlos wie generics und type-earasure ... und entspricht einem String.equals(String)

 

[Mujahiddin]

if (inp.equals("/list " + inp)

Das wird immer

false

zurückliefern, denn ein String "xyz" wird niemals einem String "/list xyz" entsprechen.

Es gibt hier im Forum irgendwo doch einen Chatserver-Code in Java - den kannst du dir doch mal anschauen, vielleicht wirst du dann ja schlauer.

 

[Akkon]

Ich kann mit den Methoden nicht umgehen, ich kriege es einfach nicht hin, die BroadCast Methode umzuschreiben auf eine andere....