Mój pierwszy program błąd w kodowaniu

Question

Strzała! piszę iż mam problem dokładniej z moim kodem . Dopiero zaczynam się w to bawić . Nie wiem jak zmodyfikować mój kod aby działał poprawnie.

#include <iostream>

using namespace std;
string date;
int main()
{
    cout << "W ktorym miesiacu sie urodziles :";
    cin>>date;

        if(date=="styczen");
    {
        cout<<"koziorozec";
    }

        if(date=="luty");
    {
        cout<<"wodnik";
    }

        if(date=="marzec");
    {
        cout<<"ryby";
    }

        if(date=="kwiecien");
    {
        cout<<"baran";
    }

        if(date=="maj");
    {
        cout<<"byk";
    }

        if(date=="czerwiec");
    {
        cout<<"bliznieta";
    }

        if(date=="lipiec");
    {
        cout<<"rak";
    }

        if(date=="sierpien");
    {
        cout<<"lew";
    }

        if(date=="wrzesien");
    {
        cout<<"panna";
    }

        if(date=="pazdziernik");
    {
        cout<<"waga";
    }

        if(date=="lisopad");
    {
        cout<<"skorpion";
    }
       if(date=="grudzien");
    {
        cout<<"strzelec";
    }
    return 0;
}

Ogólnie chodzi mi o to że po wpisaniu jakiegokolwiek miesiąca w odpowiedzi wyskakują wszystkie znaki zodiaku naraz. Z GÓRY DZIĘKI ZA POMOC !

Answer 1

po każdym ifie masz średnik, usuń go bo nie wykonują się instrukcje pod nim

 

if(date=="styczen")
    {
        cout<<"koziorozec";
    }

tak to powinno wyglądać. I używaj za miast if if if to lepiej coś takiego

if(date=="styczen")
    {
        cout<<"koziorozec";
    }
else if(date=="costam")
{
cout<<"costam";
}

przyśpiesza to program bo nie sprawdza on warunków, które i tak się nie sprawdzą (no chyba, że twój wybór padnie na ostatni warunek)

Answer 2

Zapomniałeś o jednej rzeczy:

#include "string"

 

Comments

Do używania dynamicznie alokowanej tablicy charów nie jest potrzebny nagłówek string

---

Visual Studio twierdzi jednak inaczej:

Error	C2679	binary '>>': no operator found which takes a right-hand operand of type 'std::string' (or there is no acceptable conversion)

Linia 9.

---

Bez tego działa poprawnie .

---

Jest to po prostu zależne od implementacji. Mimo wszystko warto zaincludować, gdyż umożliwiamy w ten sposób przenośność programu.