Problem z kodem C++

Question

Chciałem stworzyć kalkulator używając języka obiektowego C++. Napisałem kod:

main.cpp

#include <iostream>
#include "obliczenia.h"

using namespace std;

int main()
{
    Obliczenia d1;

    d1.wczytaj();
    d1.oblicz();

    return 0;
}

obliczenia.h

#include <iostream>

using namespace std;

class Obliczenia
{
    public:
    int skladnik1, skladnik2, odjemna, odjemnik, czynnik1, czynnik2, dzielna, dzielnik, wybor;


    void wczytaj();
    void oblicz();
};

obliczenia.cpp
 

#include <iostream>
#include "obliczenia.h"

using namespace std;

void Obliczenia::wczytaj()
{
    cout<<"1 - Dodawanie";
    cout<<endl<<"2 - Odejmowanie";
    cout<<endl<<"3 - Mnozenie";
    cout<<endl<<"4 - Dzielenie";
    cout<<endl<<"Wybierz: ";
    cin>>wybor;
}

void Obliczenia::oblicz()
{
    if (wybor=="1")
    {
        cout<<endl<<"Podaj pierwszy skladnik: ";
        cin>>skladnik1;
        cout<<endl<<"Podaj drugi skladnik: ";
        cin>>skladnik2;
        cout<<endl<<"Wynik: "<<skladnik1+skladnik2;
    }
}

I wyskakuje mi:

||=== Build: Debug in ćwiczenia (compiler: GNU GCC Compiler) ===|
H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\Obliczenia.cpp||In member function 'void Obliczenia::oblicz()':|
H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\Obliczenia.cpp|18|warning: comparison with string literal results in unspecified behaviour [-Waddress]|
H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\Obliczenia.cpp|18|error: ISO C++ forbids comparison between pointer and integer [-fpermissive]|
||=== Build failed: 1 error(s), 1 warning(s) (0 minute(s), 3 second(s)) ===|

Proszę, pomóżcie. Próbowałem już, ale kompletnie nie wiem o co chodzi.

 

 

 

Comments

Wklej kod w odpowiednią ramkę : http://forum.pasja-informatyki.pl/faq#faq1

Answer 1

Wybor to zmienna int, a ty porównujesz ją ze stringem.

Comments

Dzięki :) Zmieniłem wybor na zmienną string i problem zniknął. Ale pojawił się nowy. Teraz po skompilowaniu wyskakuje komunikat:

||=== Build: Debug in ćwiczenia (compiler: GNU GCC Compiler) ===|
obj\Debug\Obliczenia.o||In function `ZSteqIcSt11char_traitsIcESaIcEEbRKSbIT_T0_T1_EPKS3_':|
H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\obliczenia.cpp|7|multiple definition of `Obliczenia::wczytaj()'|
obj\Debug\Obliczenia.o:H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\obliczenia.cpp|7|first defined here|
obj\Debug\Obliczenia.o||In function `ZN10Obliczenia6obliczEv':|
H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\obliczenia.cpp|17|multiple definition of `Obliczenia::oblicz()'|
obj\Debug\Obliczenia.o:H:\Documents and Settings\komp\Moje dokumenty\ćwiczenia\obliczenia.cpp|17|first defined here|
||error: ld returned 1 exit status|
||=== Build failed: 5 error(s), 0 warning(s) (0 minute(s), 3 second(s)) ===|
 

Coś z nawiasami jest nie tak, ale nie wiem co. Fajnie by było, gdybyś znów pomógł.

---

wybór jest zmienną typu int, więc nie możesz go porównywać w ten sposób:

if (wybor=="1")

//sorry odpowiedziałem nie na to pytanie

 

---

A nie piszesz przypadkiem 2 razy ciała funkcji? Nie zmieniałeś nic w kodzie od wysłania oprócz tego typu na string?

---

Oprócz tego nic.

---

Spróbuj użyć strażniku nagłówka czyli na początku pliku .h dodaj:

#ifndef OBLICZENIA_H
#define OBLICZENIA_H

a na końcu

#endif

EDIT: Jak nie zadziała to powiedz jakie środowisko.

---

Nie działa. A środowisko to Code::Blocks.

---

Też mam codeblocksa i mi działa. Skopiuj kod który wkleiłeś w temacie i zmień warunek na if(wybor == 1).

---

Jak masz jakieś pliki *.o w folderze z projektem to je usuń.

---

Cudownie, działa! Nareszcie udało mi się rozwiązać problem z którym borykałem się kilka godzin :)