Beginner Loop Question

The Question: Suppose s is any string. Write a code fragment, including a loop, that counts the number of r's (upper or lower case in s), and then prints out that count

My solution:
public class tester{
public static void main(String[] args){
String s="Roar";
int total=0;
s.toLowerCase();
for(int j=0; j<=s.length()-1;j++)
{if (s.charAt(j)=='r')
total++;
System.out.println(total);
}
}
}


this gives me an output of
0001.

How do I make the program print out just 1 number at the end and count the upper case R? Thanks for any help in advance.

Your println is executing each iteration of the loop so if you want to print it one time after the iterations are done you would.............

got it. I move the println out of the loop statement...& I put in another if statement for the other r. I just don't get why the s.toLowerCase still print out Roar rather than roar

you did'nt use the s.LowerCase you just use the raw string

Code:

---

public class tester{
public static void main(String[] args){
String s="Roar";
int total=0;
s.toLowerCase(); ///////////////you did not use me
for(int j=0; j<=s.length()-1;j++)
{if (s.charAt(j)=='r')
total++;
System.out.println(total);
}
}
}

---

you use it in your print statement if you want to show s in lowercase:

Code:

---

String t = s.toLowerCase();
System.out.println(t);

---

now it should print all lowercase

ok, thanks so much for pointing that out! So basically my original statement did nothing. Thank you for clearing that up

did we help you? Please press the add the reputation

Quote:

---

Originally Posted by RichersooN

did we help you? Please press the add the reputation

---

No, you didn't help the op; Strings are immutable and a simple s.toLowerCase() doesn't do anything; please check your facts before you post a (non) solution and begging for rep points is sort of childish too.

Jos