#include <iostream>
#include <string>

using namespace std;

void zaehlziffern(string s){
	cout <<"Der String hat "<<s.size()<<" Zeichen";
};

void toupperstring(string s){
	cout <<s<<endl;
	for(unsigned int i=0;i<s.size();++i){
		s[i]=toupper(s[i]);
	cout <<s[i];
    }
};

int main()
{ 
	char frage='w';
	string s="";

	do{		
	cout <<"Bitte Text eingeben: ";
	cin.ignore();
	getline(cin,s);
	zaehlziffern(s);
	cout <<endl;
	toupperstring(s);
	cout <<endl;
	cout <<"e=ende, w=weiter ";
	cin >>frage;
	}while(frage !='e');

}

#include <iostream>
#include <string>

using namespace std;

void verdreher(string s)
{
	unsigned int x = s.size();
	for(x;x>0;--x)
	{
		cout <<s[x];
	}
}

int main()
{
	string s;
	char a;
	cout <<"Bitte ein String eingeben: ";
	getline(cin,s);
	verdreher(s);
	cin >>a;
}

Dann ist Deine Abbruch-Bedingung in der For-Schleife nicht ganz richtig.
Wenn x = 0 ist, dann muss die Schleife auch noch ausgeführt werden.
Wenn Du aber als Bedingung x >= 0 machst, bekommst Du ne endlosschleife, weil x unsigned ist.

einfacher könnte es so gehen:

#include <algorithm> // für std::reverse
...
void verdreher(string s) 
{
	std::reverse(s.begin(), s.end()); // dreht den inhalt des strings um
	cout << s;
} 
...

... um bei deinem Code zu bleiben ...

getline(cin,s);
    zaehlziffern(s);
    cout <<endl;
    toupperstring(s);
    cout <<endl;
    cout <<"e=ende, w=weiter ";

... geht besser wenn du ne std. Funktion nimmst!

std::cout << "Text: " << std::flush;
		std::string text;
		std::getline(std::cin, text);
		length_show(text, std::cout);
		std::transform(text.begin(), text.end(), text.begin(), std::toupper);
		std::cout << "\n" << text << std::endl;
		std::cout << "(E)nde - (W)eiter" << std::endl;

... viel Ordentlicher
Aja die nötigen Header:

#include <algorithm>
#include <cctype>

... und um beim Muster zu bleiben:

void length_show(const std::string& string, std::ostream& stream)
{
	stream << "Der String hat " << string.length() << " Zeichen!";
}

...

so das mal als Verbesserungsvorschlag