Go Premium for a chance to win a PS4. Enter to Win

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

alternate strings challenge

Hi,

I was working on below challenge

http://codingbat.com/prob/p165666

I wrote as below

public String stringBits(String str) {
  for(int i=0;i<str.length()-1;i=i++){
  String s="";
  return (s+str.substring(i,i+2));  
  }
  return null;
}

Open in new window

I got output as below

Expected      Run            
stringBits("Hello") → "Hlo"      "He"      X         
stringBits("Hi") → "H"      "Hi"      X         
stringBits("Heeololeo") → "Hello"      "He"      X         
stringBits("HiHiHi") → "HHH"      "Hi"      X         
stringBits("") → ""      "null"      X         
stringBits("Greetings") → "Getns"      "Gr"      X         
stringBits("Chocoate") → "Coot"      "Ch"      X         
stringBits("pi") → "p"      "pi"      X         
stringBits("Hello Kitten") → "HloKte"      "He"      X         
stringBits("hxaxpxpxy") → "happy"      "hx"      X         



How to improve it and fix it. Please advise
0
gudii9
Asked:
gudii9
  • 8
  • 8
  • 3
  • +1
3 Solutions
 
krakatoaCommented:
Try this:

public String stringBits(String str) {
  StringBuilder sb = new StringBuilder();
  for (int y=0;y<(str.length());y=y+2){
  sb.append(str.charAt(y));
  }
  return sb.toString();
}

Open in new window

0
 
n2fcCommented:
public String stringBits(String str) {
  String s="";
  for(int i=0;i<str.length();i+=2)
      s=s+str.substring(i,i+1);  
  
return s;
}

Open in new window

0
 
n2fcCommented:
As an alternative:

public String stringBits(String str) {
  String s="";
  for(int i=0; i<str.length(); ++i)
       s+=str.substring(i, ++i);  
  
  return s;
}

Open in new window


See if you can tell why they BOTH work!!
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
gudii9Author Commented:
public String stringBits(String str) {
  for(int i=0;i<str.length()-1;i=i++){
  String s="";
  return (s+str.substring(i,i+2));  
  }
  return null;
}







public String stringBits(String str) {
  String s="";
  for(int i=0;i<str.length();i+=2)
      s=s+str.substring(i,i+1);  
  
return s;
}

Open in new window



comparing my solution with your solution as above both looks almost similar except in the increment i wrote i=i+1 and inside for loop i=i+2 (where as yours i=i+2 and i=i+1) . I wonder what is the difference.


I also made my usual mistake
i<str.length()-1

I though i represent like index so it is 1 less than original length.

Please advise
0
 
gudii9Author Commented:
for (int y=0;y<(str.length());y=y+2){
  sb.append(str.charAt(y));

Is y is not like index (index always starts with 0 so we take str.length()-1) but we are taking y<(str.length())
Please advise
0
 
ozoCommented:
Another mistake, is that i=i++ does not do what you seem to think it does
0
 
krakatoaCommented:
Is y is not like index (index  . . .

Right well consider this - the word "Hello". The index starts at zero as you say, so this is character 'H'. Then we add 2 to that, so that points at character 'l', then, because 'y' is now 2, and 2 is still less than 5 (the length of the string 'Hello'), we again add 2 to 'y', making it 4, and take the 4th index character, which is 'o', and so we get 'Hlo'. Now, if 2 is added to 'y', it will exceed the string length, and so this brings the loop to a halt.
0
 
krakatoaCommented:
gudii - do me this favour will you - answer these 5 questions, based on the string "Hello". :

1. What is the index number of the beginning of the string?

2. What is the index number of the end of the string ?

3. What is the length of the string?

4. What is the MAXIMUM value that this expression can return for 'y' ? : y<"Hello".length()  

5. What is the difference between the values of your answers to points 4 and point 2 above?
0
 
gudii9Author Commented:
1. What is the index number of the beginning of the string?
>>0

2. What is the index number of the end of the string ?
>>4

3. What is the length of the string?
>>5
4. What is the MAXIMUM value that this expression can return for 'y' ? : y<"Hello".length()  
>>5(since "Hello".length()  is 5)

5. What is the difference between the values of your answers to points 4 and point 2 above?
>>sorry i did not clearly understand this. point 4 gives answer as 5 and point 2 gives answer as 4 so both lenght and index are different and index is always one less than length
0
 
gudii9Author Commented:
Right well consider this - the word "Hello". The index starts at zero as you say, so this is character 'H'. Then we add 2 to that, so that points at character 'l', then, because 'y' is now 2, and 2 is still less than 5 (the length of the string 'Hello'), we again add 2 to 'y', making it 4, and take the 4th index character, which is 'o', and so we get 'Hlo'. Now, if 2 is added to 'y', it will exceed the string length, and so this brings the loop to a halt.

this explanation is very clear. I was jumping by mistake to next character rather than alternate character whose difference is 2 not1
0
 
n2fcCommented:
comparing my solution with your solution as above both looks almost similar except in the increment i wrote i=i+1 and inside for loop i=i+2 (where as yours i=i+2 and i=i+1) . I wonder what is the difference.


I also made my usual mistake
i<str.length()-1

I though i represent like index so it is 1 less than original length.

I think your best bet to figure this out would be to "hand test" by making  a table of every assignment change to see where your values are awry...

Think about what you are doing in the loop!

Putting your "return" INSIDE the loop makes NO SENSE!  It would STOP looping the first time it hits it!  Compare to the OTHER solutions offered that build up the proper answer and only then returns that answer AFTER the loop completes...

You also need to understand the way "++i" "i++" and "i=i++" work... go over these operators in a good C or Java language reference...
0
 
krakatoaCommented:
You got my point 4 wrong. Look at it again more carefully.
0
 
krakatoaCommented:
Because not understanding my question 4 is, I believe, the reason you are having trouble, so if you can deal with this point I think you will make progress.
0
 
gudii9Author Commented:
'y' ? : y<"Hello".length()

Is it is ternary operator(looks like that to me with ? and :

It also does not look like that since there is no two separate entities corresponding to condition true and false.

Please advise
0
 
krakatoaCommented:
y<"Hello".length()


IS THE EXPRESSION!!!!!!!!!!!!!


Im am asking you the max value of y !!!!
0
 
ozoCommented:
I think krakatoa means max value of y for which the expression is true
0
 
gudii9Author Commented:
oh that is  6
0
 
ozoCommented:
Is this true?
6<"Hello".length()
0
 
gudii9Author Commented:
oh i made a mistake. I understand the question now. So the y can go maximum upto 4. It if goes beyond 4 say 5  or 6 etc the condition returns false
0
 
krakatoaCommented:
oh i made a mistake. I understand the question now. So the y can go maximum upto 4. It if goes beyond 4 say 5  or 6 etc the condition returns false

That's it ! y's maximum value can be 4 - the highest number that is less than 5, . And 5, as you know, is the length of the string.
0
 
gudii9Author Commented:
public String stringBits(String str) {
  String s="";
  for(int i=0;i<str.length();i+=2)
      s=s+str.substring(i,i+1);  
  
return s;
}





public String stringBits(String str) {
  String s="";
  for(int i=0; i<str.length(); ++i)
       s+=str.substring(i, ++i);  
  
  return s;
}

Open in new window


Both of above scenarios working since we are incrementing first then assigning to i in the second scenario which makes it equivalent to fist scenario
0
 
krakatoaCommented:
Yes, they both work. But your two examples are making the code more difficult to read than it needs to be, because you can make it easier to understand I would say, if you use StringBuilder and follow the example I gave originally.

(It's good that you made the two pieces of code work, but as someone great once said, 'you should make things as simple as possible, and no simpler'.
0

Featured Post

Free Tool: Port Scanner

Check which ports are open to the outside world. Helps make sure that your firewall rules are working as intended.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

  • 8
  • 8
  • 3
  • +1
Tackle projects and never again get stuck behind a technical roadblock.
Join Now