Infos about the environment needed to work

Developer
Dec 21, 2010 at 3:54 PM
Edited Dec 21, 2010 at 8:38 PM

Ok so we have the code :)

Now you should make a TO DO list ^^

by the way....

 

I have copy paste all the packet command and input them into two enum, you will found it here : http://paste.pocoo.org/show/308100/

About your code, why are you using some char[] variables where you could use string ?

 

exemple : public char[] szAccount = new char[64];

string are easier to manipulate...

Another thing, when you're reading "option" file i suggest you to replace : string[] lineData = line.Split(' '); by

string[] lineData = line.Split(new char[]{' '},StringSplitOptions.RemoveEmptyEntries);

 

Like this if someone add by mistake an ' ' before any option this will avoid you to loose a line while reading ^^

Developer
Dec 21, 2010 at 7:17 PM
Edited Dec 21, 2010 at 8:35 PM

I also suggest you to replace : switch (lineData[0])

by : switch (lineData[0].ToLower())

 

You have forget to close you StreamReader sReader = new StreamReader(filename); after loading the option. I suggest to put the while loop into a :

 

try { loop} catch {write log and keed default value } finally { close the streamreader } like this : 

 

K i have rewrite all the option.cs with the comment i gave you and i have add the save function...

 

you have it here : http://paste.pocoo.org/show/308095/

Developer
Dec 22, 2010 at 5:43 AM

 a little fix.

 

Option class should be public and static i think

Coordinator
Dec 22, 2010 at 6:50 PM

I have a small to do list at the forum.