[Nefarious]

In this tutorial I will be teaching you how to make a modification tool similar to the editors in Modio and Horizon.

You will need to download PackageIO.dll there will be a link at the bottom of the thread and it will be attached.

So first we want to create a new project, choose windows application form.

Spoiler:

Now we will design the form, for this it will be simple, add two buttons two textbox's. (The top textbox is the current value)
It should look something like this.

Spoiler:

Add PackageIO as a reference and import it.

In Public Class Form1 enter this code.

Spoiler:

Now under that add this code

Spoiler:

And under that add this code.

Spoiler:

Now we will add this code to the open button.

Spoiler:

Now for the mod button.

Spoiler:

Making this tutorial I assumed You'd already have the offsets, if you don't you should download a hex editor I use HxD and find them.

PackageIO

http://www.sendspace.com/file/5ae2vu

If you want the project file post here and I might send it to you.

[cxS]

You need to be calling the Close() method here to dispose of the underlying streams.

Same thing here:




ProjectData is not meant to be used directly in the code, so this isn't even your own code, this was a disassembled bit of code that you took from someplace else evidently. Then declaring exception As an Exception which points to the already referenced Exception in the heap by the identifier ex...

Yes, I have .NET Reflector too, and this is added in by that program, it's not part of the original code. So if you're going to use someone else's code, perhaps next time you should be giving credits?

Furthermore:


X will be null, so if Button2 was pressed before Button1 which sets X to a non-null value (because string is a reference type, not a value type)... You will get an exception here, that you handle with a Try Catch, and that's not good at all.

Better way, if you're going to do it this way at all would be to check the reference to Nothing.

Code:

If X IsNot Nothing Then...

[cxS]

I would personally have a function that acts as a wrapper for OpenFileDialog, and internally handles the exceptions otherwise, and returns a null string if no file was selected. Then check the function return value against null to know whether the action was successful or not... OTHERWISE, you can adopt the TryParse() paradigm, and do things that way, which could be clean...

Nice share otherwise..

[Nefarious]

You need to be calling the Close() method here to dispose of the underlying streams.

Same thing here:




ProjectData is not meant to be used directly in the code, so this isn't even your own code, this was a disassembled bit of code that you took from someplace else evidently. Then declaring exception As an Exception which points to the already referenced Exception in the heap by the identifier ex...

Yes, I have .NET Reflector too, and this is added in by that program, it's not part of the original code. So if you're going to use someone else's code, perhaps next time you should be giving credits?

Furthermore:


X will be null, so if Button2 was pressed before Button1 which sets X to a non-null value (because string is a reference type, not a value type)... You will get an exception here, that you handle with a Try Catch, and that's not good at all.

Better way, if you're going to do it this way at all would be to check the reference to Nothing.

Code:

If X IsNot Nothing Then...

I did not use .net reflector I don't even know how. I learned how to do this from other tutorials when I started modding XBOX 360 games.

[cxS]

Well lol, you got it from somebody who used .NET reflector from someone else's code and they didn't realize that they left behind some evidence of a skiddish copy and paste.

[Nefarious]

Well lol, you got it from somebody who used .NET reflector from someone else's code and they didn't realize that they left behind some evidence of a skiddish copy and paste.

I thought it was crucial to the application since it was in both tutorials I had seen. :-/

[cxS]

Well lol, you got it from somebody who used .NET reflector from someone else's code and they didn't realize that they left behind some evidence of a skiddish copy and paste.

I thought it was crucial to the application since it was in both tutorials I had seen. :-/

It is a bad use of the Try Catch block anyways. Anywhere there is a Catch, this is the way you will see it in the new .NET reflector. So essentially you got this code from someone who took the code from someone else's decompiled application, that wrote bad code to begin with lol. Kind of a long chain of unfortunate events. It's typical that this happens on youtube though, things just circulate, good or bad code, regardless.

[xVenom]

Your writing code won't you. You can't call ReadFile() then WriteFile()

Call it like

WriteFile()
ReadFile()