shrink array size in the stack while we remove elements
$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?
java algorithm
$endgroup$
add a comment |
$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?
java algorithm
$endgroup$
add a comment |
$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?
java algorithm
$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
java algorithm
asked 7 mins ago
daviddavid
46252138
46252138
add a comment |
add a comment |
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
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
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
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.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
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
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
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