[C#] Potion User

[Haard]

Item gmp = new Item(client, Tibia.Constants.Items.Potion.GreatManaPotion.Id);
            Item smp = new Item(client, Tibia.Constants.Items.Potion.StrongManaPotion.Id);
            Item mp = new Item(client, Tibia.Constants.Items.Potion.ManaPotion.Id);
            String potiontype = (textBox11.Text);
            if ((double)client.Player.Mana / client.Player.ManaMax * int.Parse(textBox9.Text) > int.Parse(textBox10.Text)) 
            {
                (potiontype).useonself();
            } 

I came down to this when i were fixing some for my bot.
Got total brainfreeze haha

[Blaster_89]

Use a switch-case for this, like so:
[code=c#]
switch (textBox11.Text)
{
case "MP":
mp.useonself();
break;
// fill in SMP and GMP
}
[/code]

I'm assuming this will send a hotkey packet.

[Haard]

Cheers dude![hr]
Error 2

Only assignment, call, increment, decrement, and new object expressions can be used as a statement	C:\Users\comp0wn\documents\visual studio 2010\Projects\WindowsFormsApplication1\WindowsFormsApplication1\Form2.cs	116	25	WindowsFormsApplication1

hmm...

[Blaster_89]

Cheers dude![hr]
Error 2

Only assignment, call, increment, decrement, and new object expressions can be used as a statement	C:\Users\comp0wn\documents\visual studio 2010\Projects\WindowsFormsApplication1\WindowsFormsApplication1\Form2.cs	116	25	WindowsFormsApplication1

hmm...

Code?

[Haard]

Item gmp = new Item(client, Tibia.Constants.Items.Potion.GreatManaPotion.Id);
            Item smp = new Item(client, Tibia.Constants.Items.Potion.StrongManaPotion.Id);
            Item mp = new Item(client, Tibia.Constants.Items.Potion.ManaPotion.Id);
            if ((double)client.Player.Mana / client.Player.ManaMax * int.Parse(textBox9.Text) > int.Parse(textBox10.Text)) 
            {

                switch (textBox11.Text)
                {
                    case "GMP":
                        gmp.UseOnSelf;
                        break;
                    case "SMP":
                        smp.UseOnSelf;
                        break;
                    case "MP":
                        mp.UseOnSelf;
                        break;
                    // fill in SMP and GMP
                }
            } 

My apologies!

[Blaster_89]

UseOnSelf is a method, not a property.

[Haard]

edit - Fixed!

Thanks blaster!
Just a little question do you think i should use any threading or something or do you think it will work out without?

[Blaster_89]

Considering that you are a beginner, timers will do fine for now.

[Haard]

Okey thanks dude.

[Haard]

private void button6_Click(object sender, EventArgs e)
        {
            Player player = client.GetPlayer();
            player.Light = Tibia.Constants.LightSize.Full;
            player.LightColor = Tibia.Constants.LightColor.White;
        } 

causes

>

utsuts.exe!WindowsFormsApplication1.Form3.button6_Click(object sender, System.EventArgs e) Line 55    C# 

Do i need to set the Tibia Clients Version or whats going on?