string basic

[ssandha]

Hi everyone I am new to c++ so I am having a problem with my string question.
i have to determine if the string is Palindrome.Palindrome is a word, phrase, or sequence that reads the same backward as forward ex level.
my code keeps giving me 0 as output.
i can use only one return statement and no recursion or break.

really appreciate your help. thanks

#include <iostream>
#include <string>
using namespace std;

bool is_Palindrome (string s)
{
	bool isPalindrome = false;
	int len = s.length();
	int backLen = len-1;

	if (s.length() > 1)
	{
		int i = 0;
		while(i > len/2)
		{
			if ( s[i] == s[backLen])
			{ 
				isPalindrome = true;
				
				i++;
				backLen--;
			}
			else 
				isPalindrome = false;
		}
	}
	else if (s.length() == 1)
	{
		isPalindrome = true;
	}
	
	return isPalindrome;
}

int main()
{
	string s = "level";
	cout << is_Palindrome(s)<< endl;
	return 0;
}  

[Peter87]

On line 11 you use the right bit shift operator >>. You probably want to use >.

On line 16 you are always comparing the first character in the string. Don't you want to use i here?

[ssandha]

hi Peter87,
i made the changes you said but i still returns 0.

[demonwolf]

Its the condition of the while loop that is the problem. You set i = 0, then the while loop is only entered if i > len/2 which it never is.

change it to

while(i < len/2)

I've tested it and your code works with that change.

[ssandha]

thank you demonwolf