Pliki tektstowe, ciągle zapisuję sie 0.

Question

To znowu Ja, zmodyfikowałem lekko kod jednak nadal nie jestem w stanie sobie poradzić z tym uporczywym 0 w pliku txt.

#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <fstream>
#include <conio.h>
using namespace std;
int minuty;
int wybor;
int i; //to wywala w notatnik

int main()
{


cout<<"WITAJ W PROGRAMIE, WYBIERZ JEDNA Z OPCJI"<<endl;
cout<<"----------------------------------------"<<endl;
cout<<"Nowa Sesja"<<endl;
cout<<"Pokaz czas wczesniejszczej sesji"<<endl;
cin>>wybor;
switch(wybor)
case 1:
{

    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();
break;
}
    switch(wybor)
    case 2:
    {
    fstream t;
    t.open("Program_Minutowy.txt", ios::in);
     cout<<i;
    if(t.good()==false)
        {

                cout<<"Nie mozna otworzyc pliku!";
        }
    break;
    }

    return 0;

}

 

Answer 1

To już drugi temat z tym programem. Mogłeś kontynuować poprzedni wątek. Jak już Ci pisałem, w pętli for definiujesz nową zmienną o nazwie i, chociaż planujesz działać na globalnej. Btw. - zmienne globalne są nieeleganckie.

W Twoim kodzie jest dwukrotnie użyta konstrukcja switch, jest to bardzo nieładne i nawet nie wiem, czy działa. Poprawiłem Ci i tę zmienną z pętli, i switch. Przeanalizuj.

#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <fstream>
#include <conio.h>

using namespace std;
int minuty;
int wybor;
int i; //to wywala w notatnik

int main()
{


    cout<<"WITAJ W PROGRAMIE, WYBIERZ JEDNA Z OPCJI"<<endl;
    cout<<"----------------------------------------"<<endl;
    cout<<"Nowa Sesja"<<endl;
    cout<<"Pokaz czas wczesniejszczej sesji"<<endl;
    cin>>wybor;
    switch(wybor)
    {
    case 1:
        for (i=1; i<=36000; i++) // <- tutaj nie definiuj nowej zmiennej (int i = 1), jedynie przypisz wartość (i = 1)
        {

            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();
        break;

    case 2:
        fstream t;
        t.open("Program_Minutowy.txt", ios::in);
        cout<<i;
        if(t.good()==false)
        {

            cout<<"Nie mozna otworzyc pliku!";
        }
        break;
    }

    return 0;

}

 

Comments

Ja bym zamiast dać

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

dałbym

for (;;)

---

Już Mu o tym wspominałem. Nie poprawiałem logiki programu, niech się autor trochę pogimnastykuje. ;)

---

Na początku też zamierzałem użyć pętli nieskończonej, jednak nie widziałem co napisać w samej pętli.  Gdy program będzie działać to go zrefaktoryzuje :)

---

Kimi, już rozumiesz, o co chodziło z tymi zmiennymi i?

---

Tak już rozkminiłem :) Siadłem przy kodzie na nowo drugiego dnia i wszystko stało się takie przejrzyste.

edit. Dziękuje za pomoc bo już dostrzegłem mój prosty i bardzo głupi błąd.

---

To super. :) Jakbyś miał jeszcze jakieś pytania do tego kodu, to wal śmiało - jak będę, to podpowiem.

---

Teraz muszę ogarnąc aby tekst w notatniku miał postać taką jak w konsoli czyli minuty i godziny ale mam nadzieję że dam rady sam :)

Answer 2

Wywal int z pętli for, masz deklaracje na szczycie, wystarczy. W pętli zostaw tylko " i=0 "

Pozdro ;]]

Comments

Nie zbyt rozumiem. Pętla ma wyglądać tak:

for(int i=0)

Może wydawać się to banalne, ale dopiero zaczynam ;(

Answer 3

To jest problem redefinicji. Nie wyrzucało Ci żadnego błędu?

Comments

Nie ma problemu redefinicji - w C++ istnieje takie zjawisko, jak przesłanianie zmiennych.

---

Dobrze, że są mądrzejsze głowy ode mnie ;)