Thread: Need help with declaring static method for code.

Hi Marcusjamaal
Please see the announcements page for use of code tags, and properly format your code in the post above.

Originally Posted by jps

Hi Marcusjamaal
Please see the announcements page for use of code tags, and properly format your code in the post above.

Thank you.

You are welcome. Please note that the closing tag should not be the same as the opening tag. This is the reason the tags are visible in the post and the code is not treated as code. The correct closing tag is listed on the announcements page linked to above. Please edit the post and correct the code to be highlighted and properly formatted.

I will also mention that your question could use some improvement.

I can't figure out how to do that

How to do what exactly?

it says it doesn't work.

What says what does not work?

Can you tell me how to fix it?

Surely you get the idea.
If there is an error message, paste it with your post. Try to form your question to be more specific as to what is not working the way you expected, or what part you are having problems with.

Sorry Guys here's what's specifically wrong with it


1. Illegal start of expression , cannot find symbol, symbol class : class first number, location: class Greatest common division finder.

2. Illegal start of expression, cannot find symbol, symbol class : class second number, location : class Greatest common division finder.

3. int Greatestcommondivisionfinder = 0; < The assigned value is never used

4. public static int Greatestcommondivisionfinder(firstNumber, secondNumber){ <identifier expected>

Z

heres an updated code...I can say I'm understanding you, but it's alot different then actually doing so...

public static void main(String[] args) {
 
        // welcome the user to the program
        System.out.println("Greatest Common Division Finder");
        System.out.println();  // print a blank line
 
        //create a Scanner object and start a while loop
        Scanner sc = new Scanner(System.in);
        String choice = "y";
        while (choice.equalsIgnoreCase("y"))
        {
            //get input from user 
 
            System.out.print("Enter first number:        ");
            int firstNumber = sc.nextInt();
 
            System.out.println("Enter second number:     ");
            int secondNumber = sc.nextInt();
 
            int j = firstNumber;
            int i = secondNumber;
        }
 
    }
 
 
 
            //creat static method int j = firstNumber, int i = secondNumber;
 
 
      static int Greatestcommondivisionfinder (int j, int i)
      {
 
 
 
 
 
 
 
 
 
 
            while (j > i)
            {
 
                 i -= j;
 
                                j--;
            }
 
            else if ( int j <= int i) 
            {
                   j -= i ;
                                  i--;
 
        }
            while ( i >= 1)
 
        {
                    j -= i ;
                                    i--;
 
                }
 
              if (  i == 0);
            { int Greatestcommondivisionfinder = (j); 
 
           }
 
 
 
 
               //Display GreatestCommondivisionfinder
                String message = 
                        "Greatest Common Division Finder:" + Greatestcommondivisionfinder + "\n";
                System.out.println(message);
 
        }

I'm having these problems now with it.

1. theres a problem with "else without if"

2. while ( i >= 1) , cannot find symbol <identifier expected>

3. Alot of number 2 for mutiple functions

First of all: It's a really, really (really) Good Thing to define a function (method) like your GCD finder in such a way as it does one thing: Calculates the GCD and returns that value. Then the main() function calls that method and prints the result that is returned. (You don't have to do it this way, but that's the way that I will illustrate.)

So the guts of your main() look like the following. I used a shorter name for the function. You can use anything you please. I, personally can't see the point of using "CalculateGCD" or ObtainGCDByEuclidsAlgorithm or stuff like that, but maybe you are supposed to use a particular name that was specified in the assignment.

Anyhow, here's the part of main() that gets user values, calls the function and prints the value returned by the function.

            System.out.print("Enter first  number:        ");
            int firstNumber = sc.nextInt();
 
            System.out.print("Enter second number:     ");
            int secondNumber = sc.nextInt();
 
            System.out.println("Will find gcd of " + firstNumber +
                               " and " + secondNumber);
 
            int result = Gcd(firstNumber, secondNumber);
            System.out.println("Result = " + result);

In other words, there is no reason to have to make up new names for variables to use to call the function. You can use the names you already have.


Secondly: Get into the habit of using indentation that is consistent, and it's a lot easier to match braces and see the program flow. (For example: to see whether an 'else' appears naked (without a corresponding 'if').


Thirdly: I don't particularly like the wording of the algorithm in your first post, and I can see why people might have a problem following it.

I'll give a description that makes more sense to me. This is pseudo-code not Java, but it leads to something that you can easily implement in any of a number of procedural languages. So you can compare notes with that C++ programmer down the hall.

Function Gcd(a, b)
 
    IF a IS EQUAL TO 0
    THEN
        RETURN  b
    END IF
 
    // a was not equal to 0.  Must do a little work
    WHILE  b IS NOT EQUAL to 0
    LOOP
        IF a IS GREATER THAN b
        THEN
            SET a EQUAL TO a − b
        ELSE
            SET b EQUAL TO b − a
        END IF
    END LOOP // End of the while loop
    RETURN a

So, here is the skeleton of the Gcd function in Java:

    //
    // A Java function starts like this: A return type, the name of the function,
    // and parameters in parentheses.
    // Each parameter has its own data type declaration.
    // If the function doesn't have any parameters, the parentheses are empty.
    //
    // I'll use a and b as the parameters, since that's what was used in the 
    // pseduo-code
    //
    static int Gcd(int a, int b)
    {
        if (a == 0) { // I"ll give you the first one. For everything else, fill in the "somethings"
            //Do something
        }
        // use a and b to calculate their gcd and return the value of the gcd
        while (something) {
            if (something) {
                //something
            }
            else {
                //something
            }
        }
        //something;
    } // End of Gcd()

Now, the point is that the function works on its local copies of the values of parameters a and b. The values of a and b inside the function correspond to the arguments inside the parentheses the statement in main() that calls this function. I used the names of the arguments in main() of variables that you had already defined and given values to.

Each argument should be the same type of variable as the corresponding parameter, but their names are arbitrary. I made the names inside the function a and b so that I could fill in the "somethings" directly from the pseudo-code.

By the way, this method is called the "Euclidean algorithm" and the pseudo-code that I used is a slightly re-styled version of the example at Euclidean algorithm - wikipedia In spite of what some people seem to believe, I don't just make these things up and I don't pretend that I did. Really.

If you examine the pseduo-code that I show above and compare with the description in your first post, I think you may conclude that it is implementing the GCD algorithm the way you are supposed to be doing. (At least that's what I conclude.) There are other ways of finding the GCD of two numbers.


Cheers!

Z

Thanks Z! Wow, this is gonna help me a ton!