Thirteen Years of Service
Posts: 3,223
Threads: 29
RE: ChaT Tools 05-11-2013, 09:35 PM
#2
Thread Title Edited to reflect the non-use of special characters.
•
Thirteen Years of Service
Posts: 13
Threads: 5
RE: ChaT Tools 05-11-2013, 09:41 PM
#3
OK, ok, sory.I am new here.
•
Fourteen Years of Service
Posts: 1,475
Threads: 91
RE: ChaT Tools 05-12-2013, 11:01 PM
#5
It's alright.. Lots of people make general "toolbox" type programs, but the code is horrible.
A few things I have noticed:
- Unnecessary duplicated code in if statements
- Not disposing (even within timer methods)
- Using Shell() instead of the Process class
•
Fourteen Years of Service
Posts: 1,475
Threads: 91
RE: ChaT Tools 05-12-2013, 11:01 PM
#6
It's alright.. Lots of people make general "toolbox" type programs, but the code is horrible.
A few things I have noticed:
- Unnecessary duplicated code in if statements
- Not disposing (even within timer methods)
- Using Shell() instead of the Process class
•
Fourteen Years of Service
Posts: 1,475
Threads: 91
RE: ChaT Tools 05-13-2013, 12:55 AM
#9
Only trying to help, in the past I used to just assume that people did this out of a collection of source codes that they had seen others using and tried to put it all into one tool (because that does happen). So programs like this which just combine a whole bunch of different features, is very common because of that. I personally don't really get the point of bulking up a program like that though...
If I was looking at this code and doing a full review though, it looks like a mix of VB6, and bad VB.NET. I see the My namespace in there, so this was definitely written in VB.NET and not C#.
As for the GUI though, I can't say I'm a fan. The idea is nice, but the visual itself isn't very appealing.
•
Fourteen Years of Service
Posts: 1,475
Threads: 91
RE: ChaT Tools 05-13-2013, 12:55 AM
#10
Only trying to help, in the past I used to just assume that people did this out of a collection of source codes that they had seen others using and tried to put it all into one tool (because that does happen). So programs like this which just combine a whole bunch of different features, is very common because of that. I personally don't really get the point of bulking up a program like that though...
If I was looking at this code and doing a full review though, it looks like a mix of VB6, and bad VB.NET. I see the My namespace in there, so this was definitely written in VB.NET and not C#.
As for the GUI though, I can't say I'm a fan. The idea is nice, but the visual itself isn't very appealing.
•