C++ Pytanko, co robie źle ?

Question

11.

Napisz program, który zlicza ilość wystąpień par sąsiadujących ze sobą liter ’o’ w

podanym przez użytkownika ciągu. Program nie rozróżnia małych i dużych liter. Np.

dla podanego ciągu: Ooo ale fajny piesioo! bedzie wynosilo 3 pary.

#include <iostream>

using namespace std;

void liczenieO(char x[]);

int main()
{
    char zdanie[1024];
    cout << "Podaj dowolne zdanie:";
    cin.get(zdanie,1024);
    liczenieO(zdanie);
}
void liczenieO(char x[])
{
    int suma=0;
    for(int i=0; x[i]!='\0'; i++)
    {
        if((x[i]=='o'||'O')&&(x[i+1]=='o'||'O'))
        {
            suma++;
        }
    }
    cout<< suma;
}

Napisalem cos takiego, co jest zle? Wgl program inkrementuje mi sume jak byle co wpisze, czemu tak sie dzieje ?

Comments

Dlaczego masz funkcje bez ciała nad main a później już z ciałem pod?

---

Tak się robi w C++. W czym problem?

---

Nie wyglądało to przypadkiem tak że każda funkcja w c++ musi zostać zadeklarowana. Umieszczenie jej nad funkcją main powoduje jej zadeklarowanie automatycznie a gdy chcemy umieścić ją pod main musimy właśnie ja zadeklarować najpierw?

---

@Hipcio, Problemu nie mam. Jak nie wiem to pytam, tak to śmiesznie u mnie działa.

Answer 1

1. Takie coś nie ma sensu:

   x[i]=='o'||'O'
   // Powinno być:
   x[i]=='o' || x[i]=='O'
   

2. Jeżeli para złożona z dużej i małej litery się nie liczy trzeba zmienić warunek.

3. Wychodzisz poza zakres tablicy. Popraw warunek pętli na x[i + 1] != '\0'

Poprawny kod:

for (int i = 0; x[i + 1] != '\0'; i++)
	{
		if ((x[i] == 'o' && x[i + 1] == 'o') || (x[i] == 'O' && x[i + 1] == 'O'))
		{
			suma++;
		}
	}

Comments

Dzieki za pomoc, ale chyba Twoj warunek nie bedzie poprawny ponieważ nie dupuszcza pary Oo ani oO, powinno być

if ((x[i] == 'o'||x[i] == 'O') && (x[i+1] == 'o' || x[i+1] == 'O'))

Jezeli sie myle, popraw mnie.

---

wtedy wynik powinien wynosić 3, czyli tak jak ty napisales + ten moj warunek bedzie poprawnie?

#include <iostream>

using namespace std;

void liczenieO(char x[]);

int main()
{
    char zdanie[1024];
    cout << "Podaj dowolne zdanie:";
    cin.get(zdanie,1024);
    liczenieO(zdanie);
}
void liczenieO(char x[])
{
    int suma=0;
    for(int i=0; x[i+1]!='\0'; i++)
    {
        if ((x[i] == 'o'||x[i] == 'O') && (x[i+1] == 'o' || x[i+1] == 'O'))
        {
            suma++;
        }
    }
    cout<< suma;
}

 

 

---

A okej w takim wypadku wystarczy zinkrementować i po znalezieniu pary:

	suma++;
	i++;