C++ - błąd związany z transform(...)

Question

Witam bajecznie,

otóż nie potrafię już sobię poradzić samemu, głowię się nad pewnym błędem, a dokładniej polega on na tym, że używając linii kodu 

transform(odp.begin() odp.end() odp.begin(), ::toupper);

kompilator wywala mi błąd "no matching function for call to 'transform(std::__cxx11::basic_string<char>::i..."

jakiś spec może wie o co chodzi i poratowałby odpowiedzią? Byłbym niezmiernie wdzięczny, bo tracę już zmysły...

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <algorithm>


using namespace std;

string odp;

int ile;

int main()
{
    setlocale( LC_ALL, "" );

    fstream plik;
    plik.open("odpowiedzi.txt", ios::out);
    if(plik.good()==false) cout << "Nie można odtworzyć pliku.";

    cout << "Ile odpowiedzi masz zamiar udzielić?";
    cin >> ile;

    int* tablica;
    tablica= new int [ile];

    for (int i=0; i<ile; i++)
    {
        cout << i+1 << ". ";


        ponownaodpowiedz:
        cin >> odp;

        if(odp!="a" && odp!="b" && odp!="c" && odp!="A" && odp!="B" && odp!="C") {cout << "Nie ma takiej odpowiedzi, wybierz ją ponownie: "; goto ponownaodpowiedz;}

        transform(odp.begin() odp.end() odp.begin(), ::tolower);

        plik << i+1 << ". " << odp << endl;
    }
    plik.close();

    return 0;
}

? Ja już tracę zmysły...

Comments

A przecinki to zjadles z glodu? :P

https://en.cppreference.com/w/cpp/algorithm/transform

nie chcesz uzyc moze

http://www.cplusplus.com/reference/locale/toupper/

Zeby zmienic wszystkie na wielkie litery?

I co to za goto?

---

@Nabuchadonozor,

Ja już tracę zmysły...

Chyba tak - używasz goto tam, gdzie spokojnie można użyć pętli do...while, wywołujesz std::transform w sumie nie wiadomo po co.

---

@DragonCoder, dzięki wielkie za pomoc! Od wczoraj staram się zrobić to z tym co mówisz, ale nie mogę ogarnąć tego, ciężko zrozumieć mi coś z tej biblioteki <locale>. Znajdę jakieś poradniki i ogarnę to, bo widzę kilka innych ciekawych funkcji w niej :D A teraz się zabiore za zamienienie goto na po prostu pętle do ... while zamiast for. Miłego!

---

@j23, funkcja transform() jest po to, by zamieniła mi litery nie ważne jakie będą wpisane -> na duże. Zlecenie przez nauczyciela takie było, bo on potrzebuje tego, żeby wpisując na szybko, nie trzeba było odpalać capsa czy trzymać shifta, program przeciez to może sam zrobić. A jeszcze nie napisałem, duże litery są czytelniejsze dla oka, jeśli ich jest 40. Podczas robienia tego w pętli do ... while, nie będę miał możliwości tak prostego skodzenia tego programu. Czysty juz zrobiony kodzik taki jaki powinien być dołączam na wszelki.

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <algorithm>
#include <conio.h>

using namespace std;

string odp;
int ile;

int main()
{
    setlocale( LC_ALL, "" );

    fstream plik;
    plik.open("odpowiedzi.txt", ios::out);
    if(plik.good()==false) cout << "Nie mo¿na odtworzyæ pliku.";

    cout << "Ile odpowiedzi masz zamiar udzieliæ?: ";
    cin >> ile;

    int* tablica;
    tablica= new int [ile];

    for (int i=0; i<ile; i++)
    {
        cout << i+1 << ". ";


        ponownaodpowiedz:
        cin >> odp;

        if(odp!="a" && odp!="b" && odp!="c" && odp!="A" && odp!="B" && odp!="C") {cout << "Nie ma takiej odpowiedzi, wybierz j¹ ponownie: "; goto ponownaodpowiedz;}

        transform(odp.begin(), odp.end(), odp.begin(), ::toupper);

        plik << i+1 << ". " << odp << endl;
    }
    plik.close();

    cout << "Czas programu dobieg³ koñca.." << endl << endl << "KLIKNIJ DOWOLNY KLAWISZ, ABY WY£¥CZYÆ PROGRAM";


    _getch();
    return 0;
}

 

---

Nie musisz mi tłumaczyć, co robi std::transform, tylko ja nie widzę sensu wywołania tej funkcji w twoim kodzie. Jakbyś zrobił tak:

while (std::cin >> odp) {
    std::transform(odp.begin(), odp.end(), odp.begin(), ::toupper);
    if (odp == "A" || odp == "B" || odp == "C") break;
    std::cout << "Nie ma takiej odpowiedzi, wybierz ja ponownie: ";
}

to co innego.

---

@j23, aaa o to Ci chodziło, a szczerze Ci powiem, że nie pomyślałem o takim użyciu jego. Dzięki za pomoc!