shrink array size in the stack while we remove elements












0












$begingroup$


I am working on stack implementation where I need to shrink array size as objects are removed from the stack. Just like I increase the array size if it reaches it's default limit.



I need to implement two methods for this:





  • isBig: The first new method checks whether we should reduce the size of the array. This method returns true if the number of entries in the stack is less than half the size of the array and the size of the array is greater than 20.


  • reduce: This method creates a new array that is three quarters the size of the current array and then copies the objects in the bag to the new array.


And after implementing above two methods, I need to use them in the pop() method.



So I implemented isBig and reduce method basis on the above spec and using them in pop method.



Below is my code:



public class StackMethods {
private static final int LIMIT = 10;
private Integer stack;
private int index;

public StackMethods() {
this(LIMIT);
}

public StackMethods(int cap) {
Integer holder = (Integer) new Object[cap];
stack = holder;
index = -1;
}

@Override
public void push(Integer newEntry) {
checkCapacityLimit();
index++;
stack[index] = newEntry;
}

private void checkCapacityLimit() {
if (index == stack.length - 1)
stack = Arrays.copyOf(stack, 2 * stack.length);
}

@Override
public Integer pop() {
Integer top = null;
// this is how it will be used?
if (isBig()) {
reduce();
}
if (!isEmpty()) {
top = stack[index];
stack[index] = null;
index--;
}
return top;
}

// does this look right?
private boolean isBig() {
return ((index < stack.length / 2) && (stack.length > 20));
}

// does this look right as well?
private void reduce() {
stack = Arrays.copyOf(stack, 3 / 4 * stack.length);
}
}


Any better way to write isBig or reduce method here?









share









$endgroup$

















    0












    $begingroup$


    I am working on stack implementation where I need to shrink array size as objects are removed from the stack. Just like I increase the array size if it reaches it's default limit.



    I need to implement two methods for this:





    • isBig: The first new method checks whether we should reduce the size of the array. This method returns true if the number of entries in the stack is less than half the size of the array and the size of the array is greater than 20.


    • reduce: This method creates a new array that is three quarters the size of the current array and then copies the objects in the bag to the new array.


    And after implementing above two methods, I need to use them in the pop() method.



    So I implemented isBig and reduce method basis on the above spec and using them in pop method.



    Below is my code:



    public class StackMethods {
    private static final int LIMIT = 10;
    private Integer stack;
    private int index;

    public StackMethods() {
    this(LIMIT);
    }

    public StackMethods(int cap) {
    Integer holder = (Integer) new Object[cap];
    stack = holder;
    index = -1;
    }

    @Override
    public void push(Integer newEntry) {
    checkCapacityLimit();
    index++;
    stack[index] = newEntry;
    }

    private void checkCapacityLimit() {
    if (index == stack.length - 1)
    stack = Arrays.copyOf(stack, 2 * stack.length);
    }

    @Override
    public Integer pop() {
    Integer top = null;
    // this is how it will be used?
    if (isBig()) {
    reduce();
    }
    if (!isEmpty()) {
    top = stack[index];
    stack[index] = null;
    index--;
    }
    return top;
    }

    // does this look right?
    private boolean isBig() {
    return ((index < stack.length / 2) && (stack.length > 20));
    }

    // does this look right as well?
    private void reduce() {
    stack = Arrays.copyOf(stack, 3 / 4 * stack.length);
    }
    }


    Any better way to write isBig or reduce method here?









    share









    $endgroup$















      0












      0








      0





      $begingroup$


      I am working on stack implementation where I need to shrink array size as objects are removed from the stack. Just like I increase the array size if it reaches it's default limit.



      I need to implement two methods for this:





      • isBig: The first new method checks whether we should reduce the size of the array. This method returns true if the number of entries in the stack is less than half the size of the array and the size of the array is greater than 20.


      • reduce: This method creates a new array that is three quarters the size of the current array and then copies the objects in the bag to the new array.


      And after implementing above two methods, I need to use them in the pop() method.



      So I implemented isBig and reduce method basis on the above spec and using them in pop method.



      Below is my code:



      public class StackMethods {
      private static final int LIMIT = 10;
      private Integer stack;
      private int index;

      public StackMethods() {
      this(LIMIT);
      }

      public StackMethods(int cap) {
      Integer holder = (Integer) new Object[cap];
      stack = holder;
      index = -1;
      }

      @Override
      public void push(Integer newEntry) {
      checkCapacityLimit();
      index++;
      stack[index] = newEntry;
      }

      private void checkCapacityLimit() {
      if (index == stack.length - 1)
      stack = Arrays.copyOf(stack, 2 * stack.length);
      }

      @Override
      public Integer pop() {
      Integer top = null;
      // this is how it will be used?
      if (isBig()) {
      reduce();
      }
      if (!isEmpty()) {
      top = stack[index];
      stack[index] = null;
      index--;
      }
      return top;
      }

      // does this look right?
      private boolean isBig() {
      return ((index < stack.length / 2) && (stack.length > 20));
      }

      // does this look right as well?
      private void reduce() {
      stack = Arrays.copyOf(stack, 3 / 4 * stack.length);
      }
      }


      Any better way to write isBig or reduce method here?









      share









      $endgroup$




      I am working on stack implementation where I need to shrink array size as objects are removed from the stack. Just like I increase the array size if it reaches it's default limit.



      I need to implement two methods for this:





      • isBig: The first new method checks whether we should reduce the size of the array. This method returns true if the number of entries in the stack is less than half the size of the array and the size of the array is greater than 20.


      • reduce: This method creates a new array that is three quarters the size of the current array and then copies the objects in the bag to the new array.


      And after implementing above two methods, I need to use them in the pop() method.



      So I implemented isBig and reduce method basis on the above spec and using them in pop method.



      Below is my code:



      public class StackMethods {
      private static final int LIMIT = 10;
      private Integer stack;
      private int index;

      public StackMethods() {
      this(LIMIT);
      }

      public StackMethods(int cap) {
      Integer holder = (Integer) new Object[cap];
      stack = holder;
      index = -1;
      }

      @Override
      public void push(Integer newEntry) {
      checkCapacityLimit();
      index++;
      stack[index] = newEntry;
      }

      private void checkCapacityLimit() {
      if (index == stack.length - 1)
      stack = Arrays.copyOf(stack, 2 * stack.length);
      }

      @Override
      public Integer pop() {
      Integer top = null;
      // this is how it will be used?
      if (isBig()) {
      reduce();
      }
      if (!isEmpty()) {
      top = stack[index];
      stack[index] = null;
      index--;
      }
      return top;
      }

      // does this look right?
      private boolean isBig() {
      return ((index < stack.length / 2) && (stack.length > 20));
      }

      // does this look right as well?
      private void reduce() {
      stack = Arrays.copyOf(stack, 3 / 4 * stack.length);
      }
      }


      Any better way to write isBig or reduce method here?







      java algorithm





      share












      share










      share



      share










      asked 7 mins ago









      daviddavid

      46252138




      46252138






















          0






          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function () {
          return StackExchange.using("mathjaxEditing", function () {
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          });
          });
          }, "mathjax-editing");

          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "196"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f213078%2fshrink-array-size-in-the-stack-while-we-remove-elements%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes
















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f213078%2fshrink-array-size-in-the-stack-while-we-remove-elements%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          404 Error Contact Form 7 ajax form submitting

          How to know if a Active Directory user can login interactively

          Refactoring coordinates for Minecraft Pi buildings written in Python