Fringie Posted July 14, 2014 Share Posted July 14, 2014 (edited) Yup, using visual studio(o god i love that editor) Your code is very.. interesting; you should stick to making smaller projects to help you learn rather than waste a lot of time hard-coding an application. Here are some good projects you can do to improve https://www.udemy.com/blog/c-sharp-projects/ . A few tips based on the program you made - Think about how you want the program to run before you start coding, you hard-coded a lot because you didn't build logic into your program (i.e. you could have grabbed data online and do categorizing like this - if scimitar is present in name the weapon is a melee weapon). Use a settings class for variables, especially variables you may need to change (http://www.codeproject.com/Articles/17659/How-To-Use-the-Settings-Class-in-C). Actually give butonns/labels etc names so you don't need to view the designer to understand what you're doing. Re-use code (i.e. when you hid all the buttons you could have just put it in a method and re-used it for other buttons). Finally, make methods for each thing you want done, don't just jumble it all up in a button and call it a day. Good effort though, c# master-race. Edited July 14, 2014 by Fringie Link to comment Share on other sites More sharing options...
The Hero of Time Posted July 14, 2014 Author Share Posted July 14, 2014 (edited) Your code is very.. interesting; you should stick to making smaller projects to help you learn rather than waste a lot of time hard-coding an application. Here are some good projects you can do to improve https://www.udemy.com/blog/c-sharp-projects/ . A few tips based on the program you made - Think about how you want the program to run before you start coding, you hard-coded a lot because you didn't build logic into your program (i.e. you could have grabbed data online and do categorizing like this - if scimitar is present in name the weapon is a melee weapon). Use a settings class for variables, especially variables you may need to change (http://www.codeproject.com/Articles/17659/How-To-Use-the-Settings-Class-in-C). Actually give butonns/labels etc names so you don't need to view the designer to understand what you're doing. Re-use code (i.e. when you hid all the buttons you could have just put it in a method and re-used it for other buttons). Finally, make methods for each thing you want done, don't just jumble it all up in a button and call it a day. Good effort though, c# master-race. Thanks for the feedback, but im a junior heh, i could do that, make a class, put it in a method, but i have bad experiences with methods, mostly they don't work or w/e, ill see how it goes when i continue school i will do all this much better later. Edited July 14, 2014 by The Hero of Time Link to comment Share on other sites More sharing options...
RSturtle Posted July 14, 2014 Share Posted July 14, 2014 This is pretty freaking awesome. Nice job mate Some of this stuff I didn't even know was in this version of the game lol Link to comment Share on other sites More sharing options...
Fringie Posted July 14, 2014 Share Posted July 14, 2014 Thanks for the feedback, but im a junior heh, i could do that, make a class, put it in a method, but i have bad experiences with methods, mostly they don't work or w/e, ill see how it goes when i continue school i will do all this much better later. No problem. You were using the buttons like they were methods so you shouldn't have problems when using methods as long as you know what you want the end result produced from the method should be. P.S. Assuming you continue with programming in c# you should use resharper (it's an add-on), it's great. If you don't want to buy a key you can probably get a crack on thepiratebay. If you're using the express version of visual studio you'll need to get a crack of the the premium or ultimate version as the express version cant use add ons. Microsoft need to do a deal with resharper, it should be included with every copy! Link to comment Share on other sites More sharing options...
The Hero of Time Posted July 14, 2014 Author Share Posted July 14, 2014 This is pretty freaking awesome. Nice job mate Some of this stuff I didn't even know was in this version of the game lol Still, with all the new armours in the game i gotta add more, and im not sure if i will do it, but if i get my main acc back from the dead(banned)(working on it) then i will definility No problem. You were using the buttons like they were methods so you shouldn't have problems when using methods as long as you know what you want the end result produced from the method should be. P.S. Assuming you continue with programming in c# you should use resharper (it's an add-on), it's great. If you don't want to buy a key you can probably get a crack on thepiratebay. If you're using the express version of visual studio you'll need to get a crack of the the premium or ultimate version as the express version cant use add ons. Microsoft need to do a deal with resharper, it should be included with every copy! i might take a look at that in the future,and yes i will continue =p Link to comment Share on other sites More sharing options...
Cerberus Posted July 28, 2014 Share Posted July 28, 2014 Holy shit its messy. Link to comment Share on other sites More sharing options...
Anaesthetic Posted July 29, 2014 Share Posted July 29, 2014 Looks very nice! Link to comment Share on other sites More sharing options...
Anaesthetic Posted August 4, 2014 Share Posted August 4, 2014 Welcome! Link to comment Share on other sites More sharing options...
VinUnlededDiesel Posted August 4, 2014 Share Posted August 4, 2014 swag af mate 1 Link to comment Share on other sites More sharing options...
The Hero of Time Posted August 5, 2014 Author Share Posted August 5, 2014 Looks very nice! ty Welcome! ? Link to comment Share on other sites More sharing options...
TheRealSwaps Posted August 9, 2014 Share Posted August 9, 2014 clean it up lol Link to comment Share on other sites More sharing options...
The Hero of Time Posted August 9, 2014 Author Share Posted August 9, 2014 clean it up lol what, the code? lol this is all about how the user sees it, there will be no bugs so i never have to edit it, and i wrote it so it's kay for me Link to comment Share on other sites More sharing options...
botme Posted August 14, 2014 Share Posted August 14, 2014 Great work Link to comment Share on other sites More sharing options...
TheRealSwaps Posted August 22, 2014 Share Posted August 22, 2014 it looks like dicks Link to comment Share on other sites More sharing options...
liverare Posted August 22, 2014 Share Posted August 22, 2014 ...291 MB! Okay, the background images aren't really doing anything for me. They're a distraction. And the music; unnecessary. Remove these elements to reduce the file size. Link to comment Share on other sites More sharing options...