Write a function that takes a string as input and reverse only the vowels of a string.
Example 1:
Given s = "hello", return "holle".
Example 2:
Given s = "leetcode", return "leotcede".
Note:
The vowels does not include the letter "y".
描述:把字符串中的元音反转,input "hello",我的代码返回的还是hello,找不到问题在哪里,求解答!!谢谢!
class Solution {
public:
string reverseVowels(string s) {
if(s.size() == 0)
return s;
int i = 0, j = s.size() - 1;
while(i < j){
if((toupper(s[i])==('A'||'E'||'I'||'O'||'U')) && (toupper(s[j])==('A'||'E'||'I'||'O'||'U'))){
if(s[i]!=s[j]){
swap(s[i],s[j]);
}
i++,j--;//都是元音,不管是否交换,处理过后都要i++,j--
}
if(i>=j)
break;
if(toupper(s[i])!=('A'||'E'||'I'||'O'||'U'))
i++;
if(toupper(s[j])!=('A'||'E'||'I'||'O'||'U'))
j--;
if(i>=j)
break;
}
return s;
}
};
('A'||'E'||'I'||'O'||'U') 这个会单独进行计算,计算的结果是'a',所以已经失去了你所希望的或运算的要求,要判断可以这样['a','e','i','o','u'].indexOf(s[i])!= -1;
使用数组的indexOf,不然就一个一个或比较,没有简写
'A'||'E'||'I'||'O'||'U'
if((toupper(s[i])==('A'||'E'||'I'||'O'||'U')) && (toupper(s[j])==('A'||'E'||'I'||'O'||'U')))
这段代码有问题,楼主请思考一下这个判断条件对不对?
你将后面用括号括起来,实际上判断为toupper(s[i])==(true or false)