LEGOanimal22 Posted July 3, 2013 Posted July 3, 2013 (edited) Hello all! I created this thread to let all of you know of my PKM editor. I will be posting features that I'm working on and I have finished. Feel free to post or PM features you think should be in my app! Finished: PKM Reader Gen 4 PC To Gen 5 PC Converter Pokedex Integration Wondercard to PKM Converter B/2 & W/2 PC PKM Reader Breeding interface Screenshots Others: I'm on Github! https://github.com/LEGOAnimal22/Project_Overload Thanks to: RubenPikachu (code) Bond697(research) Kaphotics(research) codemonkey85(PKMDS library) evandixon(lots of stuff) OmegaDonut(research) krskull(suggestion) Download Link: https://www.dropbox.com/s/o8kkvf7tbnnl7ig/Overload.zip Edited December 18, 2015 by LEGOanimal22 Screenshots
evandixon Posted July 3, 2013 Posted July 3, 2013 I'm sure everyone would love to see your current progress. Upload what you have so far, or a screenshot maybe?
LEGOanimal22 Posted July 3, 2013 Author Posted July 3, 2013 Having troubles with my PC, cant upload a screenshot
evandixon Posted July 3, 2013 Posted July 3, 2013 What kind of troubles? Depending on your version of Windows, you can use the Snip It tool to easily make a screenshot of your program.
evandixon Posted July 3, 2013 Posted July 3, 2013 Must be too big. Put it in a zip file and try again.
LEGOanimal22 Posted July 3, 2013 Author Posted July 3, 2013 Check Original Post the design is not much, ive mainly been working on the code
evandixon Posted July 3, 2013 Posted July 3, 2013 Looking forward to a release. Btw, you might want to put all those images in a folder. It will help you keep your sanity.
RubenPikachu Posted July 3, 2013 Posted July 3, 2013 Seems interesting... Finished: Gen 5 PKM support(pc/party) Gen 5 mystery gifts B2W2 daycare storage B2W2 battle box storage B2W2 party storage I'm wondering about mystery gifts, are you planning a wonder card editor and/or injector?
LEGOanimal22 Posted July 3, 2013 Author Posted July 3, 2013 (edited) Wonder card to PKM, wonder card editor, and wonder card injector Possibly wonder card creator Edited July 4, 2013 by LEGOanimal22
LEGOanimal22 Posted July 8, 2013 Author Posted July 8, 2013 Now on Github: https://github.com/LEGOAnimal22/Project_Overload My project contains some of codemonkey85's library
codemonkey85 Posted July 8, 2013 Posted July 8, 2013 You mean my old VB.Net library? FYI, the whole dictionary thing was done pretty badly (by me). I'm sure there's a better way to handle database info, even if you hardcode the data. And as for creating the PKM array, I would suggest a nested loop rather than a separate while loop for each box. EDIT: Also, if you're not doing Gen IV support there is probably a hefty chunk of code you can remove, particularly in the dictionaries. The stuff that converts "Pokemon text" to Unicode and back can definitely go, since Gen V uses Unicode anyway.
evandixon Posted July 8, 2013 Posted July 8, 2013 A few things to note: -In SaveFileToolStripMenuItem_Click, you're referencing DialogResult. That's the dialog result of Form1, not of the open file dialog. Use "If SavLoadDialog.ShowDialog() = Windows.Forms.DialogResult.OK Then" -Your images weren't uploaded to GitHub, so the solution can't find the images. -Why not put those images in a separate folder for easier navigation? -Don't be afraid to put different classes in different files, for easier management. -Try moving the code in the If statement in SaveFileToolStripMenuItem_Click to a separate function that returns a List(Of G5_PC_PKM). And as codemonkey85 pointed out, it will be easier on you if you use a nested loop. Right now, if you need to change the procedure, you'll have to replace it in 24 or 25 different places. If that doesn't take away your sanity, nothing will!
LEGOanimal22 Posted July 9, 2013 Author Posted July 9, 2013 (edited) The SavFileDialog Result Edit: my images are on github https://github.com/LEGOAnimal22/Project_Overload/tree/master/Overload Edited July 9, 2013 by LEGOanimal22
LEGOanimal22 Posted July 9, 2013 Author Posted July 9, 2013 And as for creating the PKM array, I would suggest a nested loop rather than a separate while loop for each box. Well, there is a spacer in each box...
evandixon Posted July 9, 2013 Posted July 9, 2013 The SavFileDialog Result SavFileDialog.DialogResult returns the result of the SavFileDialog. In order for it to do that, it has to show the dialog, which it does. Well, there is a spacer in each box... Like I said before, it is better for manageability and your sanity if you use nested loops. In case you're not that familiar with it, For x as Integer= 1 to 10 Console.Writeline(String.Format("This is line number {0}", x)) Next You can also have additional For statements within that for statement, just have a different variable (besides "x").
codemonkey85 Posted July 10, 2013 Posted July 10, 2013 Well, there is a spacer in each box... And why should that matter? SavLStream.Seek(&H400, SeekOrigin.Begin) For Box As Integer = 0 To 23 SavLStream.Seek(&H400 + (Box * &H1000)) For Slot As Integer = 0 to 29 pkm(z) = New G5_PC_PKM(SavL.ReadBytes(136)) Next Next
codemonkey85 Posted July 10, 2013 Posted July 10, 2013 FYI, in the code I suggested I didn't increment z, so you'll either have to add that yourself, or (more efficiently) modify the code to make a viable index from Box and Slot. i.e. SavLStream.Seek(&H400, SeekOrigin.Begin) For Box As Integer = 0 To 23 SavLStream.Seek(&H400 + (Box * &H1000)) For Slot As Integer = 0 to 29 [b]pkm((Box*30)+Slot) = New G5_PC_PKM(SavL.ReadBytes(136))[/b] Next Next
evandixon Posted July 10, 2013 Posted July 10, 2013 Don't forget to set pkm equal to something. You cannot set item x of an array equal to something if the array is null. Try this: Dim pkm(900 - 1) As G5_PC_PKM What this does is make pkm an array with 900 items. (The -1 is because the number in the parentheses specifies the last index of the array, not the number of items.) But that code won't run unless you replace this: If DialogResult = Windows.Forms.DialogResult.OK Then with this: If SavLoadDialog.ShowDialog = Windows.Forms.DialogResult.OK Then DialogResult is short for Form1.DialogResult. Form1 isn't being treated as a dialog, so its dialog result will never be OK. ShowDialog displays the form, waits for it to exit, then returns the dialog result.
LEGOanimal22 Posted July 11, 2013 Author Posted July 11, 2013 Will fix later, I have to work on the design which is killing me because it takes forever....
LEGOanimal22 Posted July 13, 2013 Author Posted July 13, 2013 Can now read through block A and the four moves for just the single PKM
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now