returnで正しいものを返したい

Question

サイズ５のArrayStackの時に、pop()メソッドを実行すると戻り値は空の時は-1、それ以外は直前に入れた要素を返したい（５を返したい）のですが、結果はどちらも戻り値は「４」で返ってきてしまいます。
これを正しく返したいときはpop()メソッドのretrun文をどのようにすればいいのでしょうか？

import java.util.List;
import java.util.ArrayList;

public class ArrayStack implements Stack{
  private List<Integer> dataStack = new ArrayList<Integer>();
  private int maxSize;//スタックに格納できる最大要素数
  
/*属性maxSizeを引数で指定したsizeで初期化する。*/
  public ArrayStack(int size){
    this.maxSize = size;
    }
  
/*引数で指定したdata（1以上の整数）をスタックの末尾に追加する。
属性maxSizeの要素数に達していない場合は追加できる。
追加できる場合はtrue、追加できない場合はfalseを返す*/
  public boolean push(int data){
     boolean stack = false;
      if(dataStack.size() < this.maxSize && data > 0)
     for(int i=0; i<this.maxSize; i++){
       dataStack.add(i);
       stack = true;
     } else{
       stack = false;
     }
    return stack;
  }
  
/*スタックの末尾から要素を取り出して返す。空の場合、-1を返す。*/
  public int pop(){
    if(dataStack.isEmpty()){
    return -1;
  }
  return dataStack.get(dataStack.size()-1);
  }
  
/*スタックの要素数を返す*/
  public int getSize(){
    return dataStack.size();
  }
}

Comments

[gentaro]

インデントがメチャクチャなので揃えた方が良いのと、このクラスを使う側のコードも提示しないと何故「4」が返されるのか意味不明な気が。（「4」を入れてるコードがない）
あとpop()で取り出した要素をdataStackから削除しないで良いんですか？
インスタンス作成てからpushを１回でも読んだら「空の時」ってのがあり得ないと思うんですが。

[momon-ga]

質問の意味があまりよくわかってないですが、少なくともpopしたら、そのオブジェクトはリムーブしないといけないのでは？

[takasima20]

push() のとこ、中カッコの対応とれてなくないスか? あと for文のとこなにがしたいのかな?

Answer 1

{}のくくりもごちゃごちゃですね。
このままソースコードを貼り付けるとコンパイルエラーになりませんか？

以下のようになっている、と想像してコメントすると、

// {}を修正
public boolean push(final int data) {
    boolean stack = false;
    if (dataStack.size() < this.maxSize && data > 0) {
        for (int i = 0; i < this.maxSize; i++) {
            dataStack.add(i);
            stack = true;
        }
    } else {
        stack = false;
    }
    return stack;
}

// 呼出し部分
public static void main(String[] args){
    ArrayStack arrayStack = new ArrayStack(5);
    arrayStack.push(2);  // ← 5より小さい数を指定した際にaddされますが、引数にあまり意味がないようなので、適当な数にしています
    System.out.println(arrayStack.pop());
    System.out.println(arrayStack.getSize());
}

出力結果は

4  // arrayStack.pop() の結果
5  // arrayStack.getSize() の結果

です。

push(int data)

pop()の戻り値が4になってしまうのは、
push()のfor文で追加しているのが i(0～4までの数値)だから、ですね。

余談ですが、この引数dataがdataStackに追加されるんではないんですね？
push()が呼ばれる度にfor文で0からmaxSizeまでadd()されるようになってますが、
dataStackのsize()、maxSize以上になりませんか？