Insert Related Database


Re: Insert Related Database
#19
I'm not sure if names are necessary or not.

Having said that, one thing I didn't notice when I initially looked at your file is that I would argue that it goes against best practices for meta-commands. If you feel it is necessary to introduce new meta-commands to make this work, ideally you should only introduce a single one, and have it have one ENUM argument and then optional info after that. The LDraw spec states that you should use the program name of your program, but since you're trying to make this useful for everyone, that doesn't seem appropriate. However, I still feel that it's better to have related functionality be grouped under a single meta-command, instead of two like you have. It makes it obvious that the statements are related.

For example, you could do the following:

Code:
0 !RELATION PARENT
1 4    0 0 0    1 0 0 0 1 0 0 0 1 122c01.dat
1 4    0 0 0    1 0 0 0 1 0 0 0 1 122c02.dat
0 !RELATION CHILD Left Tire
1 1    -31.000000 6.000000 0.000000    0.000000 0.000000 1.000000  0.000000 1.000000 0.000000  -1.000000 0.000000 0.000000  3641.dat
1 1    -31.000000 6.000000 0.000000    -0.000000 0.000000 1.000000  -0.000000 1.000000 -0.000000  -1.000000 -0.000000 -0.000000  4084.dat
0 !RELATION PARENT
1 4    100 0 0    1 0 0 0 1 0 0 0 1 122c01.dat
1 4    100 0 0    1 0 0 0 1 0 0 0 1 122c02.dat
0 !RELATION CHILD Right Tire
1 1    131.000000 6.000000 0.000000    -0.000000 0.000000 -1.000000  -0.000000 1.000000 -0.000000  1.000000 0.000000 -0.000000  3641.dat
1 1    131.000000 6.000000 0.000000    -0.000000 0.000000 -1.000000  -0.000000 1.000000 -0.000000  1.000000 0.000000 -0.000000  4084.dat

In the above, you would be introducing the !RELATION meta-command. PARENT and CHILD are required as the first argument, and every other argument after CHILD is the child's name. At least two current official meta-commands behave like this (BFC and !TEXMAP).
Reply
« Next Oldest | Next Newest »



Messages In This Thread
Insert Related Database - by Ben Supnik - 2013-03-31, 20:35
Re: Insert Related Database - by Ben Supnik - 2013-04-02, 15:10
Re: Insert Related Database - by Steffen - 2013-04-14, 12:50
Re: Insert Related Database - by Ben Supnik - 2013-04-14, 14:38
Re: Insert Related Database - by Steffen - 2013-04-14, 16:23
Re: Insert Related Database - by Ben Supnik - 2013-04-15, 3:47
Re: Insert Related Database - by Ben Supnik - 2013-04-16, 2:39
Re: Insert Related Database - by Allen Smith - 2013-04-15, 16:26
Re: Insert Related Database - by Sergio Reano - 2013-04-17, 12:09
Re: Insert Related Database - by Ben Supnik - 2013-04-17, 14:00
Re: Insert Related Database - by Sergio Reano - 2013-04-17, 14:20
Re: Insert Related Database - by Steffen - 2013-04-17, 23:26
Re: Insert Related Database - by Allen Smith - 2013-04-18, 15:43
Re: Insert Related Database - by Ben Supnik - 2013-04-19, 0:21
Re: Insert Related Database - by Travis Cobbs - 2013-04-19, 4:51
Re: Insert Related Database - by Ben Supnik - 2013-04-19, 18:14
Re: Insert Related Database - by Travis Cobbs - 2013-04-19, 18:32
Re: Insert Related Database - by Tim Gould - 2013-04-19, 21:29
Re: Insert Related Database - by Ben Supnik - 2013-04-20, 2:04
Re: Insert Related Database - by Travis Cobbs - 2013-04-19, 18:27
Re: Insert Related Database - by Steffen - 2013-12-12, 5:22

Forum Jump:


Users browsing this thread: 1 Guest(s)