Zamiana małej litery na wielką i na odwrót

Question

Witam, próbuje rozgryźć takie zadanie:

Napisz program wczytujący dane z klawiatury aż do znaku @, a następnie pokazujący wczytane dane poza cyframi; każda wielka litera ma być zamieniona na małą i odwrotnie.

Nie wiem dlaczego w moim programie następuje tylko zamiana z wielkiej litery na małą, a odwrotnie nie działa, mimo że jest zrobione analogicznie. Nie rozumiem :/

 

Oto mój kod:

#include <iostream>
#include <cctype>

using namespace std;

int main()
{
    char ch;
    cout << "Wprowadz zdanie, @ konczy program." << endl;
    cin.get(ch);

    while (ch!='@')
    {
    if (isdigit(ch))
      ch = NULL;
    if (isupper(ch)==true)
      ch=tolower(ch);
    if (islower(ch)==true)
      ch=toupper(ch);
      
       cout << ch;
       cin.get(ch);
    }

    return 0;
}

Ma ktoś jakiś pomysł?

Answer 1

nie znam się na C++ ale łatwo zauważyć że najpierw zmniejszasz te wielkie litery na małe

if (isupper(ch)==true)
      ch=tolower(ch);

następnie wszystkie zwiększasz (te które były małe i te które stały się małe

if (islower(ch)==true)
      ch=toupper(ch);
       
       cout << ch;
       cin.get(ch);
    }

rozwiązanie mbędzie zastosowanie tutaj elseif jeżeli istnieje w C++ a ja nie mam zielonego pojęcia dla mnie wyglądałoby to podobnie do:

#include <iostream>
#include <cctype>
 
using namespace std;
 
int main()
{
    char ch;
    cout << "Wprowadz zdanie, @ konczy program." << endl;
    cin.get(ch);
 
    while (ch!='@')
    {
    if (isdigit(ch))
      ch = NULL;
    if (isupper(ch)==true){
      ch=tolower(ch);
    }
    else if (islower(ch)==true){
      ch=toupper(ch);
       }
       cout << ch;
       cin.get(ch);
    }
 
    return 0;
}

 

Answer 2

   
Spróbuj czy takie coś zadziała:
 if (isupper(ch)==true)
      ch=tolower(ch);
    else if (islower(ch)==true)
      ch=toupper(ch);

 

Answer 3

Macie racje! Brakowało else po warunku... ale ja ślepy. Chociaż po dodaniu else program nadal nie działał tak jak powinien, musiałem zamienić == true, na != false... Teraz rozkminiam dlaczego... 

Tak wygląda teraz kod w pętli:

while (ch!='@')
    {
    if (isdigit(ch) == true)
      ch = NULL;
    if (isupper(ch) == true)
      ch = tolower(ch);
    else if (islower(ch) != false)
      ch = toupper(ch);

       cout << ch;
       cin.get(ch);
    }

Wszystko działa super. Dzięki wielkie za pomoc :)