Reversing a string input

[csharp]

Hello,

I am trying to have an input statement inside a loop
to input a string and when it sees a period in the end it breaks
and display the string in reversed order.

  

#include<iostream>
#include<string>

using namespace std;

class Stack
{
public:
	static const unsigned MAX_SIZE = 8;
	Stack();
	unsigned size()const;
	void push(const string & item);
	string pop();
	string top();
private:
	string data[MAX_SIZE];
	unsigned elments;

};

bool die(const string & msg);

int main()
{
	Stack s;
	string x;
	//getline(cin, x);
	//s.push(x);
	//cout << s.pop() << endl;
	
	while (x != ".")
	{
		
		cin >> x;
		s.push(x);
		if (s.top() == ".")
		break;
	}
	cout << reverse(x,x-1) << endl;
	/*
		//for (; x != ".";)
		//{
			//cin >> x;
			//s.push(x);
			//break;
		//}
	//cout << s.pop() << endl;
	*/
}//main

Stack::Stack()
{
	elments = 0;
}

unsigned Stack::size() const
{
	return elments;
}

void Stack::push(const string & item)
{
	if (elments == MAX_SIZE) die("Overflow");
	data[elments++] = item;
}

string Stack::pop()
{
	if (elments == 0) die("Underflow");
	return data[--elments];
}

string Stack::top()
{
	if (elments == 0) die("Underflow");
	return data[elments - 1];
}

bool die(const string & msg)
{
	cerr << "Fatal error: " << msg << endl;
	exit(EXIT_FAILURE);
}

[keskiverto]

I am trying to

... and?

Can you describe the problem/question that you have?

[csharp]

this is my question " input statement inside a loop
to input a string and when it sees a period in the end it breaks
and display the string in reversed order. "

[keskiverto]

That looks more like the objective than a question.

You already have some code. Does it have problems?

What happens on line 41?

How will your program react to these three inputs:

Hello world .

Hello world.

Hello world

[csharp]

I am having difficulty in writing a loop that can input bunch strings and when it sees a period it exits the loop and display the whole strings backwards. Is the question clear now?

the examples you suggested won't work since my code is not correct!

[keskiverto]

while ( cin >> x && /*something*/ ) {
  s.push( x );
}

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

[csharp]

not quite , I still can't figure it out is my cout statement right ?

while (cin >> x && x != ".") {
		s.push(x);
	}
	cout << s.pop() << endl;

[keskiverto]

Printing in reverse is an entirely separate task. Think, why you do use a stack for storing the words.


I now see that my code cannot properly do the necessary logic. What I had in mind would have discarded the last word. Again:

while ( cin >> word ) {
  // there are now four possibilities:
  // * word is empty -- skip it
  // * word is just a word -- store it
  // * word ends with a dot -- store and break
  // * word is a dot -- (store? and) break
}

I did give you the link to std::string::back() for a purpose.