Answer 1

Try to revisit your code there most be a mistake that is causing it not to run properly.

Answer 2

But you know that the index starts at 0 and the last index is length() - 1? ;)

Answer 3

Szymon

I found this about npos:

http://www.cplusplus.com/reference/string/string/npos/

I am wondering if you really know what npos is. I don't understand it and would be interested in how you get the idea to use npos.

Answer 4

Okay, now I got it. In the reference about string.find() it is mentioned that it returns npos if no matches were found.

Thanks.

Answer 5

I think your second loop is wrong.

i = 0 -> p.length() - 0 = p.length()
p[p.length()] out of bounds -> should be garbage value.
In java you would get an exception.

Answer 6

Szymon

I have my doubts about this:

p[i] = (char) 0.

The char is not visible. But did you check p.length() after the first loop? Not visible is not the same as not there.
I would say there is still a char.

How does your java code looks like? Can't you convert your java code to c++?

Answer 7

Your code is not working. It does not remove all non letters.

I don't think that npos is helpful here.

Btw:

Why don't you just print p in reversed order with the condition that p[i] is inside alpha?

pseudo code:

for i = length - 1; i >= 0; i-- :
if p[i] in alpha : print p[i]
else continue

Answer 8

Nothing's wrong with this system... just change the last code to "cout << d.length()" to find the mistake
I'll give you a hint:
Your code has made a mess of whitespace
(I don't know c++.. it's just my idea)

Answer 9

Szymon

Can you please show me your new code?

Answer 10

Szymon

And erase? Maybe you did something there. But you deleted your code. So I can't test it.

Answer 11

any one can help me?

Answer 12

Samsung 8