Beginner Loop Question

[katiebear128]

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.

[mwr1976]

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.............

[katiebear128]

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

[RichersooN]

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

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:

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

now it should print all lowercase

[katiebear128]

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

[RichersooN]

did we help you? Please press the add the reputation

[JosAH]

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