Well, lets see here. I will now answer your questions and reply to your responses:
Axiom wrote:1. Two messages telling me you can't find my NPC directory are just unecessary.
I didn't get two messages but I had a sneaking suspicion about this one. I will look into that.
Axiom wrote:2. What on earth was that godawful "let's go" noise. okay thank god you let us disable that
That is the lovely sound of Wario's voice. If you don't like it, you can disable it in the settings and your settings are saved so next time you run the program, it will remember to not play it if you turned it off. **Also, I would like to point out that there are two different sounds that play at start up and they are set at random so each time you get a different intro. This was just for kicks.
Axiom wrote:3. Label18

I can't believe something like this slipped past my eyes. Thanks for pointing it out. I fixed it.
Axiom wrote:4. What is the npc directory? Is is the directory with the npc graphics? You're better off to ask for the SMBX directory and assume the location from there.
Yes. its the Graphics directory which is what this program utilizes to view the NPC's so its easier to select which one you are editing. I have also added an auto find feature to find the directory automatically.
Axiom wrote:5. .NET has a weird default sorting function where it goes 1, 10, 100, 101, etc. I have to find the code, but somewhere I have an alternate sorter that actually does proper numeric ordering (1, 2, 3, etc). But it's written in C#, you should be able to adapt it though.
Yeah I know. I could not get the list to list properly. Believe me, sorting it like that was never my intention. If you could hook me up with the patch for that, that would be swell. Thanks!
Axiom wrote:6. When you're iterating through the npc files, just do a simple if statement to check if it has an m in the filename. If it does, don't add it. Having npc 100 and 100m in the list is unecessary.
Code: Select all
For Each fileName As String In fileList
If Not fileName.Contains("m") Then
listBox1.Add(fileName)
End If
Next
Something like that
I was looking for a way to do that and I just tried the method you provided but it doesnt seem to work. My code is a bit more complex so adding it should take some time to figure out what order to put it in. I will get around to this eventually. Thanks.
Axiom wrote:7. You really shouldn't add uncessary tags. The only reason mine has a name tag is because PGE uses it, and it's written as a key. Granted, SMBX ignores it. But you should probably remove the "From which Mario Title" option and change NPC Name to output "name=<name>" in the file/preview.
Unnecessary according to whom? You? These are indeed necessary because the program uses them to display the name of the NPC as well as the filename. This is for organization. Also, if the user wishes to create his own original NPC with graphical changes, and then he wishes to change it's properties, placing a name in the file will help identify with what NPC he is messing with. This was added for convenience for those who like to keep their files organized. Furthermore, it does not effect the NPC's behavior in SMBX so there was no problem adding it in. Everything works fine.
Axiom wrote:8. Some values are unecessarily outputted.
Axiom wrote:I had much more than that in my file, some of these could actually tamper with defaults that someone wanted to keep well, default.
I have tested the NPC text files and I know that would happen. But I found a workaround that made it fool proof. Thats why I made it omit certain attributes if they were not filled out, hence the switch sounds and question boxes you mention in the future of this post.
Axiom wrote:This is why in mine and Wohlstand's editors, we have checkboxes next to the values we actually want to write. Here's a comparison between the two. Left side is from yours, right side from mine.
[rimg]
http://i.imgur.com/ZnCxq9R.png[/rimg]
You sure like rubbing this kind of stuff in.
Axiom wrote:What I liked.
1. Scrolling UI. You got it to work right.
Thanks. If you look for a better solution to a problem, it makes everything easier in the end.
2. Animated icon, it's cute. I oughta try it.
Yeah. That is my signature. Well, one of them.
Axiom wrote:3. Aside from the wario intro sound, your sounds are very tasteful and well thought out. I liked that changing the "how many frames" made a switch toggle sound and that adjusting the score made the sounds as if you kicked the enemy.
Well, what is a good program without good aesthetics as well?
Axiom wrote:Potential Improvements.
1. I have code written somewhere that combines the mask and regular sprite and turns it into a single transparent sprite. I could adapt that into a library that would work in VB if you really wanted to.
Thats a good idea.
Axiom wrote:2. Also, you could animate simply with timers and image croppping if you wanted to.
Thats what I originally wanted to do but lacked knowledge of and I could not find a proper tutorial/documentation on it. If you wanna point me in the right direction, you are welcome to.
Axiom wrote:3. Why can't I open external files?????
Because. This way, you can work on all the NPC's within a certain level. You dont want to accidentally overwrite something and forget where it came from. You open the level folder in which you are editing and have all your NPC's for that level in there. At least this is the system in which I work with.
Axiom wrote:Conclusion
I must say, this isn't as bad as I thought it would be. It has charm, but it just doesn't make sense in a world where we have two other full functioning (and functioning well) NPC editors.
Which means you assumed it was gonna be a pile of shit, right? and I assume you thought this solely because I made something you made and you didnt like it one bit. Even though another guy made one too. Why is it ok there is 2 but not 3? Its certainly not the "there can be only one" situation so why the hatred towards me? Whatever.
Axiom wrote:Oh, and I checked out your website for shits and giggles.
[rimg]
http://i.imgur.com/xnILKTW.png[/rimg]
Your gaming and production class computers can barely be called those. I know you're not doing it right now, but AMD CPU's really aren't the greatest at least for the production class, god especially the 6 cores. You know it's not 6 true cores right? It's more like 3 physical 3 logical. Trust me here. Not to say they're bad but there's better choices.
This had nothing to do with this topic. This could have not been added. Do you put yourself on a pedastool that much to where you have to dish out your inaccurate
facts opinions on everything? Not that it matters, but you are wrong about the core thing. I have heard the same thing at one time but you should really do your homework on this before talking about something you didnt study for. Just saying.
But all in all. I didn't want to leave your questions unanswered. Thank you for your review. I will get started on those bugs you've pointed out.
Peace.
