Zamiana na duże litery (optymalizacja działającego kodu)

Question

Witam napisałem program, który działa poprawnie oraz wszystko jest sprawne. Mam pytanie tylko czy jest możliwość skrócenia tego kodu (szczególnie funkcji toupper()).

#include <iostream>
#include <string>
#include <cctype>

void zamiana(std::string & ps);

int main()
{
	std::string zdanie;
	zamiana(zdanie);
	return 0;
}

void zamiana(std::string & ps)
{
	int i =0;
	std::cout << "Podaj lancuch: ";
	std::getline(std::cin, ps);
	while(ps != "@")
	{	
		std::string kt;
		for(i=0; i < ps.length(); i++)
			kt += toupper(ps[i]);
		std::cout << kt << "\nPodaj łańcuch, @ konczy program: ";
		std::getline(std::cin, ps);
	}
}

 

Answer 1

Tak, zamiast tworzyć własną funkcję, możesz użyć std::transform(), z ostatnim parametrem ::toupper

std::string string = "Ala ma kota";
std::transform(string.begin(), string.end(), string.begin, ::toupper);

 

Comments

std::transform(ps.begin(), ps.end(), ps.begin, ::toupper;

Pokazuje mi błąd że transform nie jest obiektem std.

i pytanie, ponieważ sprawdziłem działanie funkcji transform(), oczywiście tam jest 'std' ale przypisanie toupper jest bardziej zmodyfikowane poprzez dodanie obiektu (w tam tym przypadku) string zamienionego na char.

---

1. Dodaj nagłówek <algorithm> tu masz dokumentację http://en.cppreference.com/w/cpp/algorithm/transform

2. Upewnij się że masz nowoczesny kompilator. Na całym forum widać często przykłady w standardzie C++11. Dodatkowo kompilator ma włączony ten standard.

3. Tu możesz przetestować działanie: https://ideone.com/JBbEFW

---

Dziękuję po kilku modyfikacja + dodaniu nagłówka algorithm program metoda ta zadziałała.