String loop issue

Question

below is my code that checks the incoming model and modifies the source accordingly, checking if its ALLCAPS or Firstcap. The problem I am having is when the model contains a symbol e.g. matchCase("I'm","apple"). This would return apple, when it's supposed to return Apple. On the other hand, If I use "Im", it modifies it correctly to "Apple". Is there a way i can modify it that would work. I tried to run a few methods but I keep getting stuck

public static String matchCase(String model, String source){
    boolean o = true;
    if(model.toUpperCase().equals(model)){
      source = source.toUpperCase();
    }
    if(Character.isUpperCase(model.charAt(0))){
      for(int i=1;i<model.length();i++){
         if(Character.isLowerCase(model.charAt(i)) == false){
           o = false;
         }
      }
     // if(o == model.length()-1){
      if(o == true){
        String can = "";
        for(int j=0;j<source.length();j++){
          if(j==0){
            can += Character.toUpperCase(source.charAt(j)); }
          else{
            can += source.charAt(j);
          }
        }
        source = can;
//    Character.toUpperCase(source.charAt(0));
       
  }
    }
    
    return source;
  }
}

Can you please explain the logic in more clear way? What are you exactly trying to achieve? — Anand Undavia
– Anand Undavia, Commented Mar 2, 2017 at 18:22
This would return apple, when it's supposed to return Apple. On the other hand, If I use "Im", it modifies it correctly to "Apple". It all sounded good. What's the problem? — Naman
– Naman, Commented Mar 2, 2017 at 18:24
It takes in a String as a model and one is the source. Lets say the model is "CHECK" and the source is "hello". It will check the model and modify the source to HELLO as well since the model is all caps. Similarly if its model is "Check" and source is "hello", it will modify the source to "Hello". — Basil Sindhu
– Basil Sindhu, Commented Mar 2, 2017 at 18:26
What should source be converted to if model is cHecK and source is hELLO? — ahoxha
– ahoxha, Commented Mar 2, 2017 at 18:28

Clementine C. · Accepted Answer · 2017-03-02 18:24:46Z

1

I think your problem comes from the fact that

Character.isLowerCase('\'') // is false

You should change this test

if(Character.isLowerCase(model.charAt(i)) == false)

By

if(Character.isUpperCase(model.charAt(i)))

answered Mar 2, 2017 at 18:24

Clementine C.

664 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

DVT Over a year ago

For illustration, check this out: tutorialspoint.com/java/character_islowercase.htm

DVT Over a year ago

@BasilSindhu Please accept the answer if you think it is correct.

Eddie Martinez · Accepted Answer · 2017-03-02 19:38:03Z

0

If you know your model is always going to be either uppercase or firstcap can't you do something like this:

  public static String matchCase(String model, String source){ 
    if(model.toUpperCase() == model)
      return source.toUpperCase();
      // capitalize the first letter of source and send back 
   return Character.toUpperCase(source.charAt(0)) + source.substring(1);   
  }

edited Mar 2, 2017 at 19:38

answered Mar 2, 2017 at 19:32

Eddie Martinez

13.9k15 gold badges86 silver badges109 bronze badges

Collectives™ on Stack Overflow

String loop issue

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related