SRL Archive - MSI Suggestions

Okay, so I did a lot of work in this last night to get it up and running. It includes:

• Multiple players
• Multiple skills
• Multiple items for each skill
• Player activity
• If false, gives a reason
• Player location
• Player city
• Skill level
• Skill experience
• Item name
• Item amount

It's not perfect, there are still a couple things looks-wise I would like to change, but I want your guys' opinions first. If you don't like the look, don't worry that's the easy part. It can easily be changed. The hard part was making it work. :p

Here is what the progress report looks like:
[PHP]
{—————————————————————-}
{ MSI Extreme Player Report }
{—————————————————————-}
{ }
{ Player 0: Empty Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: -1 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 1: Troll Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 2: Cohen Gorak }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 3: Honour Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 4: Broddi Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 5: Cohen Abyss }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 6: Pirate Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{ }
{ Player 7: Run Cohen }
{ City: Varrock }
{ Location: Varrock east bank }
{ Active: True }
{ }
{ Skill: Woodcutting }
{ Time: 00s }
{ Experience: 0 }
{ Level: 0 }
{ Item: Normal logs }
{ Amount: 0 }
{ }
{—————————————————————-}
{—————————————————————-}
[/PHP]
Don't feel bad if you don't like it, like I said, changing the look is the easy part. Thoughts/comments?

Cheers,
Coh3n

Looks perfect.

~RM

looks better, I still think a report more like

[php]| name | location | skill | item | amount | time |

trol varrock wc log 200 0:8:09[/php]

kinda style. Takes up less space and looks less bulky, but I like the new one too :)

From: Rasta Magician
Looks perfect.

~RM

Thanks! :) Maybe you can commit that addition to the TUser? Preeeeeety pleeeeeease!?

From: Blumblebee
looks better, I still think a report more like

[php]| name | location | skill | item | amount | time |

trol varrock wc log 200 0:8:09[/php]

kinda style. Takes up less space and looks less bulky, but I like the new one too :)

Yeah I know, I like that too, but there were things that may look dumb doing it that way, such as the reason for being inactive. However, I just thought of a couple things that would make it shorter and still use that type of style.

I really like it, Nice job :)

Eeeeerrrrrr….. changing TUser…. humm.

I'd need to make a thread about that. Don't we have a different way of doing it?

We can just set up a new array. I really don't like the idea of having to change SRL's core for MSI. Or for any particular script in general.

var MSIPArray: Array of Array of TVariant;
SetLength(MSIPArray, HowManyPlayers)

~RM

From: Rasta Magician
Eeeeerrrrrr….. changing TUser…. humm.

I'd need to make a thread about that. Don't we have a different way of doing it?

We can just set up a new array. I really don't like the idea of having to change SRL's core for MSI. Or for any particular script in general.

var MSIPArray: Array of Array of TVariant;
SetLength(MSIPArray, HowManyPlayers)

~RM

I didn't think it would be that big of deal. I figured it'd just sit there like half the other attributes that don't get used. You wouldn't even have to call it MSI_Arrays, you could make it generic, like 2DArrays, but I understand what you're saying.

If we created out own array, we may as well create our own array for the skills as well, and make them integer arrays. That way we won't have to have the extra function that checks whether the variant is an integer. I didn't think about making our own to be honest, but having it all as one record would be more convenient, don't you think?

EDIT: Revision 30 works without having the alter SRL. :)