Problem z plikami tekstowymi

Question

Witam, zaczynam swoją nauke C++. Postanowiłem napisać program który pokazuje nam ile działa nasz komputer. Program działa, jednak chciałem zrobić tak aby wynik był zapisywany do pliku tekstowego aby mieć tygodniowy pogląd. Jedyny problem jest taki że gdy napisałem cina który ma być informacją dla kodu który zapisuje dane do pliku program ciągle pokazuje 0 zamiast działac poprawnie, oto kod:

#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <fstream>
using namespace std;
int minuty;
int ciin;
int main()

{



    for (int i=0; i<=36000; i++)
    {
        Sleep (60);
        system("cls");
        cout<<"Twoj komputer dziala"<<endl;
        cout<<i<<"Minut"<<endl;
        if(i>60)
        {
            cout<<"twoj komputer dziala"<<endl;
            cout<<i/60<<"godzin";
        }
        cin>>ciin;

    }

fstream t;
t.open("minuty.txt",ios::out|ios::app);
t<<ciin<<endl;
t.close();
    return 0;

}

 

Answer 1

Witaj.

program ciągle pokazuje 0

To znaczy...? Gdzie pokazuje Ci to zero? I do czego konkretnie potrzebna Ci zmienna ciin, bo nie mogę zrozumieć?

A tak poza tym, do funkcji Sleep() podajemy ilość milisekund, a nie minut ;)

Comments

Wiem że miliseknund :) Jednak nie będe czekał minuty dla testu. 

Gdy odpalam program bez cin>>ciin; działa tak jak powinien czyli normalnie pokazuje czas jednak z nią stoi na 0. 

Może że >>ciin jest nie potrzebna, jednak wydaje mi się zę jest ona końieczna do zapisu do pliku tekstowego a dokładniej tutaj:

t<<ciin<<endl;

Jednak mogę się mylić bo jak mówiłem jestem świeżakiem

---

Stoi na zero, ponieważ zapisałeś cin>>ciin;, co oznacza, że program czeka aż wprowadzisz z klawiatury liczbę. ;)

Co chcesz mieć zapisane w tym pliku tekstowym?

---

Wartość którą otrzyma program po wyłączeniu go czyli czas ile program był włączony.

---

To chcesz wpisać tam wartość i, a nie ciin. Wyjmij deklarację zmiennej i z pętli, żeby mieć do niej dostęp również poza pętlą i najzwyczajniej w świecie wpisz do pliku i.

    int i;
    for (i = 0; i<=36000; i++)
    {
        Sleep (60);
        system("cls");
        cout<<"Twoj komputer dziala"<<endl;
        cout<<i<<"Minut"<<endl;
        if(i>60)
        {
            cout<<"twoj komputer dziala"<<endl;
            cout<<i/60<<"godzin";
        }
        cin>>ciin;
 
    }
 
fstream t;
t.open("minuty.txt",ios::out|ios::app);
t<< i <<endl;
t.close();

 

---

Ok. Wszystko działa! Dzięki wielkie, taki prosty i głupi błąd a jednak ;)

---

A, zapomniałem wyrzucić z kodu cin>>ciin - ale na pewno sam to zrobiłeś. ;)

---

Widzę że jesteś bardziej doświadczony ode mnie. Zmodyfikowałem lekko kod. Zapisuje on końcowy wynik znaczy wtedy kiedy zakończe pętle jedyny problem to że zawsze zapisuje 0.

#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <fstream>
#include <conio.h>
using namespace std;
int minuty;
int i;

int main()

{

    for (int i=1; i<=36000; i++)
    {

        Sleep (600);
        system("cls");
        cout<<"Twoj komputer dziala"<<endl;
        cout<<i<<"Minut"<<endl;
          if( kbhit() )
             if( getch() == 'q' ) break;
        if(i>60)
        {
            cout<<"Twoj komputer dziala"<<endl;
            cout<<i/60<<"godzin"<<endl;
        }
cout<<"Jesli chcesz zakonczyc program i rownoczesnie zapisac swoj wynik nacisnij Q";

    }

fstream t;
t.open("Program_Minutowy.txt",ios::out|ios::app);
t<<i;
t.close();
    return 0;

}

 

---

W swoim programie masz dwie zmienne "i" - jedna globalna, która domyślnie przyjmuje wartość 0 oraz druga, lokalna w pętli, którą inicjalizujesz zerem.

W pętli zmienna lokalna i przesłania zmienną globalną. Kiedy wychodzisz z pętli, zmienna lokalna jest niszczona. Tak więc globalną zmienną pozostawiasz nietkniętą podczas działania pętli, po czym - po zakończeniu pętli - zapisujesz ją do pliku.

Spójrz na mój kod wyżej, na konstrukcję pętli for - nie definiuję tam nowej zmiennej, tylko używam zdefiniowanej wcześniej. Specjalnie wyrzuciłem definicję przed pętlę.

Inna sprawa, to taka że po zakończeniu pętli, Twoje i będzie zawsze miało wartość 36000. I zawsze taką wartość wpiszesz do pliku.

Jeśli chcesz, żeby program wykonywał jakąś akcję (np. zapis do pliku) w momencie wyłączenia, masz do dyspozycji dwa sposoby.

Pierwszy, "w stylu C", za pomocą funkcji atexit(), przyjmującej wskaźnik na funkcję, którą zamierzasz wykonać.

Drugi - konstruując obiekt, który będzie żył aż do zakończenia programu, a niezbędne akcje wykona w destruktorze.

Answer 2

Zmotywowałeś mnie do zrobienia tego: http://speedy.sh/W3KPP/Wyswietlacz.exe​

PS. Pomyśl, jak to zrobiłem i na pewno ci to pomoże przy robieniu własnego programu.

Comments

Całkiem w porządku. :)