C# Discussion Boards - CodeProject

26-Aug-12 12:06

I am trying to pass a variable from one function into another one. However, seeing as I am still in college learning C# I'm not that good yet. While this IS for an assignment it is only a small part of the assignment but a critical one I need in order to finish my program. Below is the code I have.

What I want to do is pass the variable str to a different function which will store that amount for use in another class.

public void RollStr()
{
    attributes BaseClass = new attributes();
    int Str;

    Str = dice.Roll5D6();
    Console.Write("Str: {0} \n", Str);
    //Reroll the dice
    char MyChoice;
    string MyInput;
    char ReRoll = Convert.ToChar("y");

    Console.Write("Would you like to reroll this score?");
    MyInput = Console.ReadLine();
    MyChoice = Convert.ToChar(MyInput);

    if (MyInput == Convert.ToString(ReRoll))
    {
        BaseClass.RollStr();
    }

}

This is the other function I want to store it in. Seeing as I have no ideal how to store it I haven't gotten anything in it yet.

public void ShowAbilities()
{

}

IF anyone can help me figure out how to pass the str variable into ShowAbilities I would be extremly happy. I have been messing with this program for a week now and no closer to figuring this part out and my tutors are gone until Monday, which is past the deadline for this assignment. IF I can't get help tonight I will be late with this assignment.

Wes Aday26-Aug-12 12:50

26-Aug-12 12:50

drgsldr69 wrote:
public void ShowAbilities(string str)
{

}

drgsldr6926-Aug-12 14:46

26-Aug-12 14:46

Can you do multiple variables like that? All together there is 6 functions just like the str one that i need to do lol.

Wes Aday26-Aug-12 14:50

26-Aug-12 14:50

drgsldr69 wrote:
multiple variables

Yes you can,

public void ShowAbilities(string one, int two, double three, object etc)
{
 
}

drgsldr6926-Aug-12 16:02

26-Aug-12 16:02

Ok looks like I have a problem.

error

No overload for method'ShowAbilities' takes 0 arguments

I'm getting that in the function I am trying to call the ShowAbilities

public void Character()
 {
    string N;
    N = base.Name();
    Console.Write("\n");
    Console.Write("Your Name is: {0} \n", N);
    abilities.ShowAbilities();
 }

Does that mean it wants me to put all the int's into it like I did for the actual function?

Which is

public void ShowAbilities(int Str, int Dex, int Con, int Int, int Wis, int Cha)
{
    Console.WriteLine("Strength:     {0}\n", Str);
    Console.WriteLine("Dexterity:    {0}\n", Dex);
    Console.WriteLine("Constitution: {0}\n", Con);
    Console.WriteLine("Intelligence: {0}\n", Int);
    Console.WriteLine("Wisdom:       {0}\n", Wis);
    Console.WriteLine("Charisma:     {0}\n", Cha);
}

Wes Aday26-Aug-12 16:06

26-Aug-12 16:06

Yes you have to pass all of the variables.

drgsldr6926-Aug-12 16:27

26-Aug-12 16:27

Ok I must be passing them wrong or something.
the error Im getting is showing them as int, int, int, int instead of Str, Dex, etc etc.
I tried them as int Str and ref Str neither worked. Either I am missing something or did something wrong here lol.

lukeer27-Aug-12 0:13

lukeer

27-Aug-12 0:13

Just pass them as Str, no int, no ref. That's the default. There are situations when you need to pass them as ref Str or out Str, but not here.

Ciao,

luker

Wes Aday27-Aug-12 0:26

27-Aug-12 0:26

You would call your function by

abilities.ShowAbilities(Str, Int, Wis, Dex, etc....);

drgsldr6927-Aug-12 1:55

27-Aug-12 1:55

I tried that but it says "it doesn't exist in the current context"

public class Fighter : attributes
 {
    Abilities abilities = new Abilities();
      public void Character()
       {
          string N;
          N = base.Name();
          Console.Write("\n");
          Console.Write("Your Name is: {0} \n", N);
          abilities.ShowAbilities(Str, Dex, Con, Int, Wis, Cha);
       }

So am I missing something?

public void ShowAbilities(int Str, int Dex, int Con, int Int, int Wis, int Cha)
{
    Console.WriteLine("Strength:     {0}\n", Str);
    Console.WriteLine("Dexterity:    {0}\n", Dex);
    Console.WriteLine("Constitution: {0}\n", Con);
    Console.WriteLine("Intelligence: {0}\n", Int);
    Console.WriteLine("Wisdom:       {0}\n", Wis);
    Console.WriteLine("Charisma:     {0}\n", Cha);
}

This block below is repeated 5 more times once for each ability.

public void RollStr()
{
    Abilities BaseClass = new Abilities();
    int Str;

    Str = dice.Roll5D6();
    Console.Write("Str: {0} \n", Str);
    //Reroll the dice
    char MyChoice;
    string MyInput;
    char ReRoll = Convert.ToChar("y");

    Console.Write("Would you like to reroll this score?");
    MyInput = Console.ReadLine();
    MyChoice = Convert.ToChar(MyInput);

    if (MyInput == Convert.ToString(ReRoll))
    {
        BaseClass.RollStr();
    }

}

They are in 2 different classes if that makes a difference.

lukeer27-Aug-12 2:04

lukeer

27-Aug-12 2:04

So, if they don't exist int that context, do they exist at all?
Means: Do you have a int Dex = 42; anywhere in your code?

And if so, where in your code does that appear?

Edit

There is a int Str; in method RollStr().
Do you also have a mehtod RollDex() that includes int Dex;?

Ciao,

luker

drgsldr6927-Aug-12 2:24

27-Aug-12 2:24

yes I have one for each int

lukeer27-Aug-12 2:48

lukeer

27-Aug-12 2:48

In that case, they shouldn't be declared as local variables within their respective methods but as private variables of the containing class Fighter.
Then just don't declare them again in each method, use the class variable instead and every method of the class can access the variables (even without passing them explicitly).

Ciao,

luker

Wes Aday27-Aug-12 2:19

27-Aug-12 2:19

drgsldr69 wrote:
abilities.ShowAbilities(Str, Dex, Con, Int, Wis, Cha);

This is not going to do anything since you have not defined what Str, Dex, Con and the rest are.

drgsldr6927-Aug-12 2:26