Java Stack method (multipop) Beginner java

问题:

I'm trying to write a Java method to preform a "multi-pop" off a stack.

It should perform a "pop" operation on a stack object k number of times. This is what I'm thinking, but it's not quite right. Any help out there?

public void multipop(int k) {
    while (top != null) {
        for (int i = 0; i <= k; i++) {
            this.pop();
        }
    }
}

回答1:

Looks like an off-by-one error.

If k=1, you will go through the loop with i=0 and i=1. You can fix this by changing i<=k to i<k

回答2:

1. You execute the while loop until the stack is exhausted, which is probably not what you want. If you want to check whether there are elements in the stack, use an if statement.
2. In the loop, you iterate from 0 to k, inclusive. This means that if k = 3, you go through 0, 1, 2 and 3 and thus call this.pop() four times.
3. Even if you replace the while with an if, you only check that there is one element is on the stack, but you may call pop() multiple times. You should do the check inside the loop or move the check inside pop().
4. The indentation is horrible :)

回答3:

There are a few issues with this:

1. The brackets should be better formatted [the first look lead me to believe a mismatch]
2. Your check for the null case should be in the middle of the for loop: for (... ; i<=k && stack.canPop(); ...
3. You need a method to check to make sure that there is an item that you can pop.
4. As the other answer states, there is an off by one error, if you wish to pop up to K items, then the condiction should be i < k.

This should run into an exception or an infinite loop because the first loop makes sure that there is still a "top" variable that isn't null, and then it directs it to the second loop that goes from 0:k.

回答4:

You're off by one. You want

 for(int i =0; i < k; i++)

If there's more problems, you need to provide more code and questions.

回答5:

1st, it loops (k+1) times, from 0 to k. 2nd, after several pops, it is possible that top is null. So it needs check top all the time.

It could be modified as below:

public void multipop(int k) {

    for (int i = 0; top != null && i < k; i++) {
        this.pop();
    }

}