[C#] Switch kan niet correcte case kiezen

[AeroXbird]

Hallo,

Ik ben sinds een paar dagen bezig met een hobbyprojectje, een soort server-client communication projectje, om een beetje te trainen met sockets etc in het .net framework.
Ik kan nu successvol mijn string van mijn client naar server versturen, maar zodra ik de server een response laat sturen via een switch, gaat hij in plaats van de case, naar default, wat dus niet hoort.
Hier is de code:

code:

requestCount = requestCount + 1;
                    NetworkStream networkStream = clientSocket.GetStream();
                    networkStream.Read(bytesFrom, 0, (int)clientSocket.ReceiveBufferSize);
                    dataFromClient = System.Text.Encoding.ASCII.GetString(bytesFrom);
                    
                    
                    
                    Console.WriteLine(" [NET] " + "From client-" + clNo + dataFromClient);
             

                    switch (dataFromClient)
                    {
                        case "help":
                            Console.WriteLine("HELP TRIGGERED");
                            sendBytes = Encoding.ASCII.GetBytes("Welcome to the help menu!");
                            networkStream.Write(sendBytes, 0, sendBytes.Length);
                            sendBytes = null;
                            networkStream.Flush(); 
                            break;
                        default:
                            Console.WriteLine("DEFAULT MODE KICKED IN");
                            sendBytes = Encoding.ASCII.GetBytes("Default mode for command: " + dataFromClient);
                            networkStream.Write(sendBytes, 0, sendBytes.Length);
                            sendBytes = null;
                            networkStream.Flush(); 
                            break;
                    }
                    networkStream.Flush();

[asfaloth_arwen]

Als je een debugpoint zet op regel 11 en de waarde van dataFromClient bekijkt, wat is die dan?

[AeroXbird]

asfaloth_arwen schreef op zaterdag 15 oktober 2011 @ 21:26:
Als je een debugpoint zet op regel 11 en de waarde van dataFromClient bekijkt, wat is die dan?

Dan krijg ik netjes "help" terug (zonder quotes), wel met een boel spaties, wat voor mij nog een raadsel blijft.

[ Voor 3% gewijzigd door AeroXbird op 15-10-2011 21:29 ]

[merijnen]

Dan zit daar je probleem in: de case moet exact overeenkomen. Je kan misschien even een trim doen op je returnwaarde, of zorgen dat deze netter terug wordt gegeven dan nu het geval is

[ik222]

Die spaties zijn dan in elk geval de de reden dat het fout gaat aangezien "help " niet hetzelfde is als "help". Je kunt nu natuurlijk nasty iets van trim gaan doen waardoor de spaties weg zijn en de functie werkt. Maar beter is het natuurlijk om uit te vinden waarom die spaties erin komen.

[ Voor 43% gewijzigd door ik222 op 15-10-2011 21:37 ]

[AeroXbird]

Helaas werkt het niet, ik trim de waarde van whitespaces voordat de switch geroepen word, maar het werkt helaas niet.
En met spaties bedoel ik het volgende:

Op het console scherm links zie je wel wat ik bedoel.

[Mini-me]

AeroXbird schreef op zaterdag 15 oktober 2011 @ 21:40:
...
En met spaties bedoel ik het volgende:
[afbeelding]

Newline characters dus? \r\n ?

[AeroXbird]

Mini-me schreef op zaterdag 15 oktober 2011 @ 21:43:
[...]

Newline characters dus? \r\n ?

Het lijkt daar op, al voeg ik deze zelf niet toe in de code, ze komen gewoon poef uit het niets het consolescherm op zodra ik wat probeer te outputten.
Het zou kunnen komen omdat ik de tekst van bytes naar ASCII omzet, maar dat zou geen problemen moeten opleveren.

[Snake]

C#:

switch(dataFromClient.Trim()) 

[AeroXbird]

Snake schreef op zaterdag 15 oktober 2011 @ 21:45:

C#:

switch(dataFromClient.Trim()) 

Helaas, hij schiet weer terug naar default, toch bedankt.

[Snake]

Ok, om te testen:

C#:

foreach(char c in dataFromClient)
{
   Console.WriteLine((int)c);
}

Zie je direct wat er eigelijk mee komt.

[AeroXbird]

Snake schreef op zaterdag 15 oktober 2011 @ 21:48:
Ok, om te testen:

C#:

foreach(char c in dataFromClient)
{
   Console.WriteLine((int)c);
}

Zie je direct wat er eigelijk mee komt.

Hmm interessant, hij returned met een lijn zo lang dat het niet eens op het consolescherm past, hij begint met een boel nullen, dan komen er een paar niet-nul (de 4 letters dus) en dan weer een hele meuk nullen, ik neem aan dat die nullen spaties voorstellen.

[huarru]

AeroXbird schreef op zaterdag 15 oktober 2011 @ 21:47:
[...]

Helaas, hij schiet weer terug naar default, toch bedankt.

Probeer eens dataFromClient.replace("\r\n","").trim(). Weet je zeker dat er geen newLines meekomen.

[AeroXbird]

huarru schreef op zaterdag 15 oktober 2011 @ 21:52:
[...]


Probeer eens dataFromClient.replace("\r\n","").trim(). Weet je zeker dat er geen newLines meekomen.

Nope, werkt ook niet.
Zoals ik hiervoor al zei, hij voegt wel bij de karakters die spaties toe, wat raar is.

[SKiLLa]

Ik gebruik meestal iets als ("###{0}###", debugString) zodat je NewLines of whitespaces/tabs meteen ziet ...
Verder zie ik dat je ook geen: using (NetworkStream networkStream) gebruikt, leer je die best-practise meteen aan zou ik zeggen ...

Maar on-topic, code:

code:

networkStream.Read(bytesFrom, 0, (int)clientSocket.ReceiveBufferSize);
dataFromClient = System.Text.Encoding.ASCII.GetString(bytesFrom);

moet zijn:

code:

int bytesRead = networkStream.Read(bytesFrom, 0, bytesFrom.Length);
dataFromClient = System.Text.Encoding.ASCII.GetString(bytesFrom, 0, bytesRead);

Nu converteer je altijd het hele buffer naar string, niet alleen het 'gevulde' gedeelte ...

[AeroXbird]

SKiLLa schreef op zaterdag 15 oktober 2011 @ 21:58:
Ik gebruik meestal iets als ("###{0}###", debugString) zodat je NewLines of whitespaces/tabs meteen ziet ...
Verder zie ik dat je ook geen: using (NetworkStream networkStream) gebruikt, leer je die best-practise meteen aan zou ik zeggen ...

Maar on-topic, code:

code:

networkStream.Read(bytesFrom, 0, (int)clientSocket.ReceiveBufferSize);
dataFromClient = System.Text.Encoding.ASCII.GetString(bytesFrom);

moet zijn:

code:

int bytesRead = networkStream.Read(bytesFrom, 0, bytesFrom.Length);
dataFromClient = System.Text.Encoding.ASCII.GetString(bytesFrom, 0, bytesRead);

Nu converteer je altijd het hele buffer naar string, niet alleen het 'gevulde' gedeelte ...

Hij werkt helemaal top!
Wat jij zei klopte, ik converteerde de hele meuk, inplaats van het stukje wat tekst was.
Heel erg bedankt
zou je ook even in kunnen gaan over dat using (NetworkStream networkStream) kunnen ingaan?
Ik snap niet helemaal hoe ik dat moet toepassen, en wat de voordelen daarvan zijn.

[SKiLLa]

Dat gaat over IDisposable implementeren. Sommige objecten (zoals files/streams/db-connecties/etc.) vereisen dat je na gebruik expliciet het object 'opruimt'. Using is 'syntax sugar'; de compiler vertaalt het eigenlijk naar:

code:

DisposableObject disposableObject;
try
{
    disposableObject = ...
    // blaat blaat code
}
finally
{
    if (disposableObject) disposableObect.Dispose();
}

en garandeerd dus dat je IDisposable object altijd netjes opgeruimd wordt, zelfs als er een Exception optreedt.

code:

using (DisposableObject disposableObject = ...)
{
    // blaat blaat code
}

is toch stukken duidelijker (je ziet namelijk direct aan de code al dat het object IDisposable is) ...

EDIT: Dispose() niet aanroepen kan tot locking en resource-/memory-leaks leiden, vandaar dat het cruciaal is dat je er heel 'zorgvuldig' mee omgaat.

[ Voor 8% gewijzigd door SKiLLa op 15-10-2011 22:16 ]

[AeroXbird]

SKiLLa schreef op zaterdag 15 oktober 2011 @ 22:13:
Dat gaat over IDisposable implementeren. Sommige objecten (zoals files/streams/db-connecties/etc.) vereisen dat je na gebruik expliciet het object 'opruimt'. Using is 'syntax sugar'; de compiler vertaalt het eigenlijk naar:

code:

DisposableObject disposableObject;
try
{
    disposableObject = ...
    // blaat blaat code
}
finally
{
    if (disposableObject) disposableObect.Dispose();
}

en garandeerd dus dat je IDisposable object altijd netjes opgeruimd wordt, zelfs als er een Exception optreedt.

code:

using (DisposableObject disposableObject = ...)
{
    // blaat blaat code
}

is toch stukken duidelijker (je ziet namelijk direct aan de code al dat het object IDisposable is) ...

EDIT: Dispose() niet aanroepen kan tot locking en resource-/memory-leaks leiden, vandaar dat het cruciaal is dat je er heel 'zorgvuldig' mee omgaat.

Sinds de hele meuk in een loop staat, kan ik het niet gebruiken, maar ik ga het zeker ergens anders toepassen, bedankt!

[Styxxy]

Gezien jouw code in de OP, zou ik toch suggereren om ook jouw code wat op te delen in methodes etc.

[SKiLLa]

PS: Loop of niet; toch altijd using gebruiken, anders is er iets mis met je ontwerp