Programiranje u C++-u - pitanja i odgovori

Evo jedan program koj sam napravio al ima error tj. nezeli inicijalizirati varjablu.

#include <iostream>

using namespace std;

class ZbrajanjeOduzimanje
{
    public : double c;

    public : double Zbroji(float a, float b)
             {
                 c = a + b;
                 return c;
             }

             float Oduzmi(float a, float b)
             {
                 c = a - b;
                 return c;
             }
};

class Identifikacija
{
    private : string username;

    username = "David";

    private : string password;

    password = "mypwdavid123";

    public : string un;
    public : string pw;

    public : bool IsDavid(string un, string pw)
             {
                 if(un == username && pw == password)
                 {
                     return true;
                 }
                 return false;
             }

};

int main()
{
    string operacija;
    double unos1, unos2;

    ZbrajanjeOduzimanje obj1;
    Identifikacija id;


    cout << "Korisnicko ime:";
    cin >> id.un;
    cout << "Lozinka:";
    cin >> id.pw;

    if(id.IsDavid(id.un, id.pw) == true)
    {
    cout << "Unesi recunsku operaciju koju zelis izvesti (+,-) :";
    cin >> operacija;

    if(operacija == "+")
    {
        cout << "Odabrali se racunsku operaciju zbrajanje" << endl;
        cout << "Unesite brojeve koje zelite zbrojiti" << endl;
        cin >> unos1 >> unos2;
        cout << "Rezultat je:" << obj1.Zbroji(unos1, unos2) << endl;
    }
    if(operacija == "-")
    {
        cout << "Odabrali se racunsku operaciju oduzimanje" << endl;
        cout << "Unesite brojeve koje zelite oduzeti" << endl;
        cin >> unos1 >> unos2;
        cout << "Rezultat je:" << obj1.Oduzmi(unos1, unos2) << endl;
    }
    }
    return 0;
}

Errori:

||=== OOP, Debug ===|
C:\Users\David\Dropbox\C++\OOP\main.cpp|26|error: 'username' does not name a type|
C:\Users\David\Dropbox\C++\OOP\main.cpp|30|error: 'password' does not name a type|
||=== Build finished: 2 errors, 0 warnings ===|

Napravio sam ovako(u klasi Identifikacija):

void SetID(string a, string b)
             {
                 username = a;
                 password = b;
             }

i u int main()

id.SetID(David, mypwdavid123)

i dobijem:

C:\Users\David\Dropbox\C++\OOP\main.cpp||In function 'int main()':|
C:\Users\David\Dropbox\C++\OOP\main.cpp|59|error: 'David' was not declared in this scope|
C:\Users\David\Dropbox\C++\OOP\main.cpp|59|error: 'mypwdavid123' was not declared in this scope|
||=== Build finished: 2 errors, 0 warnings ===|

Napravio si grešku u tome što ne možeš inicijalizirati podatkovne članove klase na način kako si to učinio.

Najbolje je to učiniti preko konstruktora.

Obrati pažnju kako sam podatkovne članove klase stavio pod private, a metode pod public, to sam učinio zato jer mi nema što vanjski kod brljati po podatkovnim članovima klase.

A zaboravio si i u headeru uključiti string, neophodno za uspoređivanje stringova.

Zašto kao argumente funkcija Zbroji i Oduzmi koristiš float tip za argumente, a klasa ima double podatkovni član - nepotreban casting koji može dovesti do gubitka podataka.

Klasu ZbrajanjeOduzimanje bi trebalo drukčije napisati, odnosno da funkcija na osnovu argumenta + ili - kao trećeg računa rezultat, a ne u vanjskom kodu raditi komplikacije, ali to ostavljam tebi.

#include <iostream>
#include<string>
using namespace std;

class ZbrajanjeOduzimanje
{
    private :
       double c;

    public :
       double Zbroji(double a, double b)
             {
                 c = a + b;
                 return c;
             }

             float Oduzmi(double a, double b)
             {
                 c = a - b;
                 return c;
             }
};

class Identifikacija
{
    private :
      string username;
      string password;

    public :
       Identifikacija()
       {
          username = "David";
          password = "mypwdavid";
       }
       bool IsDavid(string un, string pw)
             {
             if((un == username) && (pw == password))
                 {
                     return true;
                 }
                 return false;
             }

};

int main()
{
    string operacija, username, password;
    double unos1, unos2;

    ZbrajanjeOduzimanje obj1;
    Identifikacija id;


    cout << "Korisnicko ime:";
    cin >>  username;
    cout << "Lozinka:";
    cin >> password;
    if(id.IsDavid(username, password))
    {
    cout << "Unesi recunsku operaciju koju zelis izvesti (+,-) :";
    cin >> operacija;

    if(operacija == "+")
    {
        cout << "Odabrali se racunsku operaciju zbrajanje" << endl;
        cout << "Unesite brojeve koje zelite zbrojiti" << endl;
        cin >> unos1 >> unos2;
        cout << "Rezultat je:" << obj1.Zbroji(unos1, unos2) << endl;
    }
    if(operacija == "-")
    {
        cout << "Odabrali se racunsku operaciju oduzimanje" << endl;
        cout << "Unesite brojeve koje zelite oduzeti" << endl;
        cin >> unos1 >> unos2;
        cout << "Rezultat je:" << obj1.Oduzmi(unos1, unos2) << endl;
    }
    }
    return 0;
}

Vidi stvarno tu sam fulao,ma navika iz onog fcking PAWN-a.

Na što misliš?

Nije tu bila greška, nego ne može izvršiti inicijalizaciju podatkovnog člana na taj način.

Ovo je inače dozvoljena operacija:

string username;

username = "fgdt";

Deklaracija, pa inicijalizacija.

Druga stvar je što se ne može vršiti inicijalizacija podatkovnog člana klase na ovaj način.

Aha, misliš na onaj SetID funkciju, je, to je greška, u pravu si.

Da, ali ni sintaksa nije bila tocna. Valjda je na to mislio.

Aha, sorry, to nisam ni gledao.

Te dvije funkcije su inače nepotrebne, učini inicijalizaciju kroz konstruktor, ovako samo uvodiš dvije suvišne varijable u igru.

A program ima i logičku grešku, korisnik namješta username i password preko metoda, a onda ih unosi za provjeru.

Nista mi nije jasno, kuzim da koristis standardni DevCpp predlozak za WinAPI aplikaciju, ali sto zelis napraviti? Sto ti nije jasno? Ne znas napisati funkciju koja vraca string? Ne znas ispisati taj string u prozor? Daj malo detaljnije, po mogucnosti i kôd.

U meduvremenu pogledaj si ovamo kako bi pohvatao konce sa nekim osnovama.

A ako si ozbiljan s time i ne kanis prijeci na nesto drugo (modernije, jednostavnije), onda si obavezno nabavi i ovu knjigu.

@191x7

Program je bezveze kao program jer ispisivanje ne mozes raditi paraleleno (dobiti ces isprepleteni ispis mozda), a poanta threaded programiranja je da radis paraleleno stvari,

I ne moras koristiti nestandardne metode, a cak ne moras ni threads koristiti, iako su u C++11 standardu

Dakle

#include <future>
#include <vector>
#include <iostream>
using namespace std;

vector<int> prostbr(int  x)
{
   bool prost = true;         
   vector<int> primes;
   for (int i = 1; i <= x; i++){
    prost = true;
     for ( int j = 2; j*j <= x; j++){
       if ( i!=j && i % j == 0 ){   
         prost = false;
         break;
       }
     }
     if (prost){
       primes.push_back(i);
     }
   } 
   return primes;
}
int main()
{
    int x=10,y=20,z=40;
    vector<future<vector<int>>> futures;
    futures.push_back(std::move(async (prostbr,x)));
    futures.push_back(std::move(async (prostbr,y)));
    futures.push_back(std::move(async (prostbr,z)));
    for(auto& ftr: futures)
    {
       const auto primes=ftr.get();
       for(auto& el : primes)
       {
          std::cout << el <<endl;
       }
       cout << endl;
    }
}

Da to bi tako nekako islo, medutim za ovo sto dalje zelis (varijabilan broj niti) ti je puno bolji izbor polje, nego hardkodiran broj struktura.

Kao sto vec rekoh, polja i petlje.

#include <iostream>
#include <vector>
#include <process.h>
#include <cmath>
#include <windows.h>

using namespace std;

void prostbr(void*);

typedef struct
{
    int interval;
    int prosti;
    unsigned long vrijeme;
    vector<int> prosti_brojevi;
} THREAD_DATA, *PTHREAD_DATA;

int main()
{
    unsigned int broj_niti=1;
    unsigned long pocetno_vrijeme;
    HANDLE niti[256];
    THREAD_DATA niti_podatci[256];

    do
    {
        cout<<"Unesite broj niti: ";
        cin>>broj_niti;

        if (broj_niti>256)
            cout<<"Unjeli ste prevelik broj. Maksimum je 256\n\n";

    } while (broj_niti>256);

    for (int n=0; n<broj_niti; n++)
    {
        cout<<"\nNit "<<n+1;
        cout<<"\n   Unesite broj: ";
        cin>>niti_podatci[n].interval;

        cout<<"   Unesite trajanje niti u sekundama: ";
        cin>>niti_podatci[n].vrijeme;
        niti_podatci[n].vrijeme*=1000;
    }

    for (int n=0; n<broj_niti; n++)
        niti[n]=(HANDLE)_beginthread(prostbr, 0, (void*)&niti_podatci[n]);

    pocetno_vrijeme=GetTickCount();

    cout<<"\nRacunanje sa "<<broj_niti<<" niti u tijeku...\n";
    WaitForMultipleObjects(broj_niti, niti, TRUE, INFINITE);
    cout<<"Dovrseno nakon "<<(GetTickCount()-pocetno_vrijeme)/1000.0<<"s\n";

    return 0;
}

void prostbr(void *arg)
{
    PTHREAD_DATA data=(PTHREAD_DATA)arg;
    long double x=data->interval;
    bool prost=true;
    int broj2=(int)floor(sqrt(x));

    unsigned long pocetno_vrijeme=GetTickCount();

    //ponavljamo u beskonacnost dok ne prodje odredeno vrijeme
    while (1)
    {
        for (int i=2; i<=x; i++)
        {
            for (int j=2; j<=broj2; j++)
            {
                //ako je vremenska razlika izmedu pocetnog i trenutnog vremena
                //veca od data->vrijeme tada prekidamo izvrsavanje
                if (GetTickCount()-pocetno_vrijeme>=data->vrijeme)
                    return;

                if (i!=j && i%j==0)
                {
                    prost=false;
                    break;
                }
            }
            if (prost)
            {
                //ima li smisla pohranjivati izracunate brojeve
                //ako nam ne trebaju?
                //data->prosti_brojevi.push_back(i);
                data->prosti+=1;
            }
            prost=true;
        }
    }
}

Primjeti da je hardkodirano ogranicenje broja petlji 256 sto je smijesno velik broj za takvo nesto, ali ako slucajno ti treba vise to se lako izmjeni. Uklonio sam i ispis izracunatih brojeva jer tebi ionako ne treba rezultat, takoder sam pod komentar stavio i liniju koja pohranjuje pronadene proste brojeve u vektor, jer ti izracunati brojevi uopce ne trebaju.

Vrhunski odrađeno. Baš proučavam kod. 

Moram ja takvo programiranje dobro savladat, baš mi je super što imam tako nekoga da mi pomaže, puno je lakše učiti. Hvala ti.

Sutra ću još detaljno proučiti kod pa ako mi štogod ne bude jasno pitam opet. 

Ako sam pretjerao s pitanjima ili imaš osjećaj da previše zanovijetam, slobodno mi reci. Bitno mi je ovo pa sam možda malo naporan.

Edit:

Našao sam problem - ako se kao broj niti unese 1 i unesemo broj, onda ispiše "unesite trajanje niti usekundama: " i odmah pokrene računanje bez da se unese trajanje niti.

Ne uspjevam reproducirati. Nakon sto upisem broj niti 1, i broj do kojega se racuna, mogu normalno upisati broj sekundi, ne preskace mi nista.

Nisi unio dovoljno veliki broj. Probaj npr 1234567891011 pa ćeš vidjeti.

Kad unesem npr 123456 onda radi. Ali meni treba baš za veliki broj, traženje do nekog malog broja mi ne optereti dovoljno i može se dogoditi da ne traje dovoljno.

Vrijednosti intervala bih eliminirao, da sve niti traže do broja zadanog, npr 9876543212345.

Dakle da se ne traži unos "do kojeg broja" za svaku nit, da se ne traži unos broja uopće već samo koliko niti i koliko vremena da vrti koju nit.

U biti vektor za pohranu do kojih brojeva se računa više mi nije potreban već samo jedna varijabla u koju bi bio pohranjen broj do kojeg se traži.

A joj, pa ti overflow radis.

Koliko velik broj moze drzati 32 bitni integer kada je signed, a koliko kada je unsigned?

Odgovor potrazi ovdje.

Ovaj tvoj broj je 574 puta veci od 32 bitnog signed integera. :D

Probaj kao broj upisati 2147483647, pa drugi puta 2147483648 i vidjet ces da na ovom drugom pocinje cudno ponasanje.

Cini se da iostream nakon loseg upisa ulazi u stanje greske nakon kojeg prekida sav ostali unos dodatnih podataka (to ce ti tracer ili floki znati bolje objasniti).  Za vise o tome si procitaj ovdje.

Dakle ili prestani upisivati tako velik broj, ili dodaj ovo:

cin.clear();

poslije ove linije:

cin>>niti_podatci[n].interval;

Sto se ovih ostalih promjena tice, izrezuj sto ti je visak, to bar nije problem :D