Learn how to a build a cloud-first strategyRegister Now

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 128
  • Last Modified:

copyEndy challenge

Hi,

I am working one below challenge



 copyEndy
prev  |  next  |  chance
We'll say that a positive int n is "endy" if it is in the range 0..10 or 90..100 (inclusive). Given an array of positive ints, return a new array of length "count" containing the first endy numbers from the original array. Decompose out a separate isEndy(int n) method to test if a number is endy. The original array will contain at least "count" endy numbers.

copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]

i am not clear what above description is talking about. How

copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]
please advise
0
gudii9
Asked:
gudii9
  • 10
  • 3
  • 2
2 Solutions
 
Guy Hengel [angelIII / a3]Billing EngineerCommented:
read this part clearly:
return a new array of length "count" containing the first "count" (I added this into the text) endy numbers from the original array
0
 
gudii9Author Commented:
what is endy?
is it is verb or noun or adjective ?

Expected      Run            
copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]      null      X      
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]      null      X      
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]      null      X      
copyEndy([12, 1, 1, 13, 0, 20], 3) → [1, 1, 0]      null      X      
copyEndy([0], 1) → [0]      null      X      
copyEndy([10, 11, 90], 2) → [10, 90]      null      X      
copyEndy([90, 22, 100], 2) → [90, 100]      null      X      
copyEndy([12, 11, 10, 89, 101, 4], 1) → [10]      null      X      
copyEndy([13, 2, 2, 0], 2) → [2, 2]      null      X      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      null      X      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      null      X      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]      null      X      
other tests
X      

how above outputs relates to below description

We'll say that a positive int n is "endy" if it is in the range 0..10 or 90..100 (inclusive). //so output array elements should always be between 0 till 10 or 90 till 100?but what is length of output array?how we know(looks like it is count)?
Given an array of positive ints, return a new array of length "count" containing the first endy count numbers from the original array.//first endy count numbers means what??
0
 
gudii9Author Commented:
copyEndy([13, 2, 2, 0], 2) → [2, 2]      null      X      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      null      X      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      null      X      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]

i think i got now. Eliminate all the numbers out of the range and return upto the count even there are 100 endy numbers are there.

let me think on psedo code and code
0
VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

 
rrzCommented:
Please post the link to the challenge.
0
 
gudii9Author Commented:
oops here is the link

http://codingbat.com/prob/p130124
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int[] arr=new int[count];
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>0)&&(nums[i]<10))|| (nums[i]>100)&&(nums[i]<10) ){
    arr[i]=nums[i];
  }
  }
  return arr;
}

Open in new window


i wrote asd above and failing some tests as below. please advise
Expected      Run            
copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]      Exception:java.lang.ArrayIndexOutOfBoundsException: 4 (line number:5)      X      
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]      Exception:java.lang.ArrayIndexOutOfBoundsException: 4 (line number:5)      X      
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:5)      X      
copyEndy([12, 1, 1, 13, 0, 20], 3) → [1, 1, 0]      [0, 1, 1]      X      
copyEndy([0], 1) → [0]      [0]      OK      
copyEndy([10, 11, 90], 2) → [10, 90]      [0, 0]      X      
copyEndy([90, 22, 100], 2) → [90, 100]      [0, 0]      X      
copyEndy([12, 11, 10, 89, 101, 4], 1) → [10]      Exception:java.lang.ArrayIndexOutOfBoundsException: 5 (line number:5)      X      
copyEndy([13, 2, 2, 0], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:5)      X      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      [0, 2, 2]      X      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 3 (line number:5)      X      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]      Exception:java.lang.ArrayIndexOutOfBoundsException: 3 (line number:5)      X      
other tests
X      
Your progress graph for this problem
0
 
Guy Hengel [angelIII / a3]Billing EngineerCommented:
>arr[i]=nums[i];

Open in new window


the issue is that you will copy the found items into the exact same index in the "result" array, and actually are not counting how many you really are copying.

create a variable (j) that defines where the found item should go to in the resulting array
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int[] arr=new int[count];
  for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>0)&&(nums[i]<10))|| (nums[i]>100)&&(nums[i]<10) ){
    arr[j]=nums[i];
  }
  }
  }
  return arr;
}

Open in new window

something like above?
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int j=0;
  int[] arr=new int[count];
 // for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>0)&&(nums[i]<10))|| (nums[i]>100)&&(nums[i]<10) ){
    arr[j]=nums[i];
    j++;
  }
  }
 // }
  return arr;
}

Open in new window

something like above?above fails below tests. please advise
Expected      Run            
copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]      [9, 6]      X      
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]      [9, 6, 0]      X      
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]      [1, 1]      OK      
copyEndy([12, 1, 1, 13, 0, 20], 3) → [1, 1, 0]      [1, 1, 0]      OK      
copyEndy([0], 1) → [0]      [0]      OK      
copyEndy([10, 11, 90], 2) → [10, 90]      [0, 0]      X      
copyEndy([90, 22, 100], 2) → [90, 100]      [0, 0]      X      
copyEndy([12, 11, 10, 89, 101, 4], 1) → [10]      [4]      X      
copyEndy([13, 2, 2, 0], 2) → [2, 2]      [2, 2]      OK      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      [2, 2, 0]      OK      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      [2, 2]      OK      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]      [2, 2, 0]      OK      
other tests
X      
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int j=0;
  int[] arr=new int[count];
 // for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>=0)&&(nums[i]<=10))|| (nums[i]=>100)&&(nums[i]=<10) ){
    arr[j]=nums[i];
    j++;
  }
  }
 // }
  return arr;
}

Open in new window


why above give below error?

Compile problems:


Error:	if( ((nums[i]>=0)&&(nums[i]<=10))|| (nums[i]=>100)&&(nums[i]=<10) ){
	                                             ^
Syntax error on token ">", delete this token

Open in new window


which side i should put = towards left side of > or right side (even for less than also which side we have to place = . i confuse often with this?)
0
 
gudii9Author Commented:
looks like = always right side.

public int[] copyEndy(int[] nums, int count) {
  int j=0;
  int[] arr=new int[count];
 // for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>=0)&&(nums[i]<=10))|| (nums[i]>=100)&&(nums[i]<=10) ){
    arr[j]=nums[i];
    j++;
  }
  }
 // }
  return arr;
}

Open in new window


above fails below tests? please advise
Expected      Run            
copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]      [9, 6]      X      
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]      [9, 6, 0]      X      
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([12, 1, 1, 13, 0, 20], 3) → [1, 1, 0]      [1, 1, 0]      OK      
copyEndy([0], 1) → [0]      [0]      OK      
copyEndy([10, 11, 90], 2) → [10, 90]      [10, 0]      X      
copyEndy([90, 22, 100], 2) → [90, 100]      [0, 0]      X      
copyEndy([12, 11, 10, 89, 101, 4], 1) → [10]      Exception:java.lang.ArrayIndexOutOfBoundsException: 1 (line number:7)      X      
copyEndy([13, 2, 2, 0], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      [2, 2, 0]      OK      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]      [2, 2, 0]      OK      
other tests
X      
Your progress graph for this problem
0
 
rrzCommented:
(nums[i]>=100)&&(nums[i]<=10)

Open in new window

That will never be true.  You should be finding 90..100 (inclusive).  
Also, you searching all through the nums array. You need to stop when the arr array is full.
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int j=0;
  int[] arr=new int[count];
 // for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>=0)&&(nums[i]<=10))|| (nums[i]>=90)&&(nums[i]<=100) ){
    arr[j]=nums[i];
    j++;
  }
  }
 // }
  return arr;
}

Open in new window


above passign more tests.
Expected      Run            
copyEndy([9, 11, 90, 22, 6], 2) → [9, 90]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([9, 11, 90, 22, 6], 3) → [9, 90, 6]      [9, 90, 6]      OK      
copyEndy([12, 1, 1, 13, 0, 20], 2) → [1, 1]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([12, 1, 1, 13, 0, 20], 3) → [1, 1, 0]      [1, 1, 0]      OK      
copyEndy([0], 1) → [0]      [0]      OK      
copyEndy([10, 11, 90], 2) → [10, 90]      [10, 90]      OK      
copyEndy([90, 22, 100], 2) → [90, 100]      [90, 100]      OK      
copyEndy([12, 11, 10, 89, 101, 4], 1) → [10]      Exception:java.lang.ArrayIndexOutOfBoundsException: 1 (line number:7)      X      
copyEndy([13, 2, 2, 0], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([13, 2, 2, 0], 3) → [2, 2, 0]      [2, 2, 0]      OK      
copyEndy([13, 2, 13, 2, 0, 30], 2) → [2, 2]      Exception:java.lang.ArrayIndexOutOfBoundsException: 2 (line number:7)      X      
copyEndy([13, 2, 13, 2, 0, 30], 3) → [2, 2, 0]      [2, 2, 0]      OK      
other tests
X      
Your progress graph for this problem

need to figure out how to stop at count. I am not able to give additional condition like j<count in for loop though?
0
 
gudii9Author Commented:
public int[] copyEndy(int[] nums, int count) {
  int j=0;
  int[] arr=new int[count];
 // for(int j=0;j<nums.length;j++){
  for(int i=0;i<nums.length;i++){
  if( ((nums[i]>=0)&&(nums[i]<=10))|| (nums[i]>=90)&&(nums[i]<=100) ){
    arr[j]=nums[i];
    j++;
    if(j==count) break;
  }
  }
 // }
  return arr;
}

Open in new window


above passed all tests. any improvements, alternate approaches?
0
 
rrzCommented:
Looks good.
0

Featured Post

Keep up with what's happening at Experts Exchange!

Sign up to receive Decoded, a new monthly digest with product updates, feature release info, continuing education opportunities, and more.

  • 10
  • 3
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now