Zadatak u C-u

Molim vas, možete li mi pomoći, dobio sam ovaj zadatak za sutra a početnik sam u programiranju pa ako bi mi tko mogao pomoći?

zadatak je:

Napisati program u kojem korisnik unosi vrijednosti prosječnih dnevnih temperatura zraka za određeni vremenski period. Ako se unese vrijednost manja od -50 ili veća od +50, korisnik dobiva mogućnost da upiše 0, 1 ili 2, gdje će 0 značiti da želi prekinuti unos (break), 1 da želi stvarno iskoristiti tu temperaturu, a 2 da je krivo upisao temperaturu te da želi nastaviti s unosom (continue). Potrebno je ispisati ukupan broj dana u razdoblju te najveću, najmanju i prosječnu vrijednost temperature u danom periodu.
Za zadani program nacrtati dijagram toka.

Zadatak sam samo copy-paste.

Ne znam kako da napravim dio zadatka gdje dobivam mogućnosti odabira 1,2,3 tj. ako upotrijebim naredbu break preskače mi daljnje izvođenje.

Ne treba mi glazbena a ni dijagram toka, to je lako napraviti, vec sam napravio.

Kako si postavio uvjete?? Kad zelis nesto izjednaciti koristis "==" mozda si to zaboravio... Samo daj kod koji sada imas pa cemo smislit nesto :)

ja bih ipak maknuo "volatile" jer to samo može izazvati grešku. i mislim da je greška što mu se ponudilo gotovo rješenje a da nije sa svoje strane pokazao nikakav rad i trud.

Mi smo na faksu radili u dev c++. Link-- http://www.bloodshed.net/devcpp.html

Uopće se ne slažem.

1) volatile nema ulogu prometnog znaka i nikoga na ništa ne upozorava niti je to njegova primarna namjena. Ako se na nešto želiš upozoriti napišeš si komentar sa strane, jer niti si išta posebno upozoren na one varijable koje nemaju volatile.

2) volatile nema veze s "nestabilnošću objekta" jer takvo što kao pojam u programiranju ne postoji.

3) prekidna rutina nije naredba if, niti se volatile koristi kod prekidnih rutina.

volatile predstavlja podatak ili objekt koji se može promijeniti od strane operacijskog sustava, sklopovlja ili na bilo koji drugi način nepoznat prevoditelju.

Koristi se npr. tamo gdje se radi sa dijeljenom memorijom kojoj može pristupiti više procesa. Prilikom pristupa takvom podatku prevoditelj neće raditi pretpostavku o njegovom sadržaju jer je možda već u međuvremenu izmjenjen, već će nanovo zatražiti provjeru vrijednosti. Ovo će jako loše utjecati na moguće optimizacije i zato se u praksi izbjegava.

Moram se ispričati za pomutnju koja je stvorena. Onaj volatile gore je njemu posve nepotreban (neće stvarati probleme). Stavio sam ga mehanički jer sam imao nešto skroz deseto na umu s ovom petljom, a kod sam pisao u par minuta u notepadu i slagao ga navrat na nos.

Isprika još jednom.

Problem je što si funkciji Pucaj vrijednosti predao preko call by value tj. sama funkcija kreira novi objekt tipa Tracer pa u njega kopira one stare podatke i još pri tome nije dinamički alociran metak (za kopiju unutar funkcije). Pokušaj ovo:

void Pucaj(Tracer& kalasnjikov)
{
    cout<<kalasnjikov.spremnik<<endl
    <<*kalasnjikov.metak<<endl;
}

Jedno od rješenja koje sam pokušao je ovo:

#include <iostream>
using namespace std;

class Tracer
{
public:
    Tracer(){spremnik = 0;}
    virtual ~Tracer(){delete metak;}
    int spremnik;
    static int* metak;
};
int* Tracer::metak = NULL;

void Pucaj(Tracer kalasnjikov)
{
    cout<<kalasnjikov.spremnik<<endl<<*kalasnjikov.metak<<endl;
}

void main()
{
    Tracer kalasnjikov;
    kalasnjikov.spremnik = 1;
    kalasnjikov.metak = new int(500);
    cout<<*kalasnjikov.metak<<endl;     //500
    Pucaj(kalasnjikov);                 // 1 500
    cout<<*kalasnjikov.metak<<endl;     // promjena vrijednosti!
}

..iako mi se čini da bi trebalo raditi.. ne radi :D nakon poziva funkcije vrijednost se neočekivano promjenila.

i zato jedino rješenje koje trenutno vidim je preko call by reference. Ili ono gore što sam napravio ili mala modifikacija ovog gornjeg code-a:

#include <iostream>
using namespace std;

class Tracer
{
public:
    Tracer(){spremnik = 0;}
    virtual ~Tracer(){delete metak;}
    int spremnik;
    static int* metak;
};
int* Tracer::metak = NULL;

void Pucaj(Tracer* kalasnjikov)
{
    cout<<kalasnjikov->spremnik<<endl<<*kalasnjikov->metak<<endl;
}

void main()
{
    Tracer kalasnjikov;
    kalasnjikov.spremnik = 1;
    kalasnjikov.metak = new int(500);
    cout<<*kalasnjikov.metak<<endl;     //500
    Pucaj(&kalasnjikov);                 // 1 500
    cout<<*kalasnjikov.metak<<endl;     // 500
}

Znam da si nešto imao na umu s volatile ali ne pada mi ništa trenutno na pamet...

Svaka čast ;) Rješenja radi. Naveo si me na krivi trag jer si spominjao volatile pa o ovome nisam ni razmišljao. Iako ako pogledaš ono moje prvo tj. :

void Pucaj(Tracer& kalasnjikov){
    cout<<kalasnjikov.spremnik<<endl<<*kalasnjikov.metak<<endl;
}

to je vrlo slična stvar. Čak ovdje niti nemaš potrebu za dodatnom alokacijom memorije.

EDIT:

Skužio sam problematiku...

Gdje je sad bio problem? Pa piše ti gore u čemu je štos.

Taj destruktor je radio problem od starta. Uništi zajednički pointer. :)