C++ Daily Programmer #2 [difficult] - Stoper problem z zapisem danych.

Question

Witam wszystkich, zabrałem się w końcu za trudniejsze zadania i programowanie obiektowe. Piszę program z daily programmera, chciałem sam do tego dojść ale nie wymyśliłem rozwiązania. Wszystko super śmiga itd. tylko jeden kłopot. Zapisując przez fstream w pliku textowym czasy zapisuje się tylko jeden (chyba pierwszy) Chcę żeby zapisywały się wszystkie linijka po linijce (potem to zrobie do funkcji wyswietl() ) i chce żeby po zakończeniu na 4 case sie usuwało wszystko z notatnika czyli plik clear(); wystarczy co nie? (Jestem początkujący proszę o w miarę łatwe rozwiązanie :) ) Myśłałem nad ddynamicznym alokowaniem nowych czasów i stworzenie pod to klasy czas i kazdy nowy czas jako nowy objekt ale nie mam pojecia jak tworzyc przez program to bez hardcodowania jako czas c1, c2, c3... no wiecie.

main.cpp

#include <iostream>
#include <fstream>
#include "stoper.h"
#include <time.h>
#include <windows.h>

using namespace std;
char wybor;
int main()

{

    cout << "Stoper" << endl<< endl;
    cout << "-------------------" << endl;
    cout << "Menu glowne" << endl;
    cout << "-------------------" << endl;
    cout << "1.Start" << endl;
    cout << "2.Stop" << endl;
    cout << "3.Okrazenie" << endl;
    cout << "4.Zakoncz program" << endl << endl<<endl<<endl;
do
  {
cin >>wybor;

switch (wybor)
{

case '1':{
start();
break;}

case '2':{
stop();
zapis();
break;}

case '3':{
okrazenie();
zapis();
break;}

case '4':{
zakoncz();
return 0;}
}
}while (wybor !=4);
return 0;}

--------------------------

stoper.cpp

#include <iostream>
#include "stoper.h"
#include <time.h>
#include <windows.h>
#include <fstream>
using namespace std;
clock_t first, loop, last;
double czas;
fstream plik;
void start()
{
first = clock();
}
void stop()
{
last = clock();
czas= (double)(last-first) / CLOCKS_PER_SEC;
}
void okrazenie()
{
loop=clock();
czas= (double)(loop-first) / CLOCKS_PER_SEC;

}
void zapis()
{
    plik.open("Zapisy.txt", ios::out | ios::app);
    plik<<czas<<endl;
}
void wyswietlanie()
{
cout << ":) ";
}
void zakoncz()
{
plik.close() ;
plik.clear() ;
}


-----------------------------

stoper.h

#include <iostream>
#include <time.h>
#include <windows.h>
#include <fstream>
using namespace std;
void start();
void stop();
void okrazenie();
void zapis();
void wyswietlanie();
void zakoncz();

 

Comments

https://www.reddit.com/r/dailyprogrammer/comments/pjsdx/difficult_challenge_2/

tutaj zadanie z reddita :)

---

Drugie pytanie jak dynamicznie wyswietlac czas tak jak w stoperze na ekranie?

Answer 1

Na pewno nie zamykasz pliku po zapisie, co skutkuje próbą otwracia już otwartego pliku i ustawia flage błędu.

http://www.cplusplus.com/reference/fstream/fstream/open/

If the stream is already associated with a file (i.e., it is already open), calling this function fails.

If the function fails to open a file, the failbit state flag is set for the stream (which may throw ios_base::failure if that state flag was registered using member exceptions).

  i chce żeby po zakończeniu na 4 case sie usuwało wszystko z notatnika czyli plik clear(); wystarczy co nie? 

Pojęcia nie mam jaki związek ma notatnik z całą resztą, ale moge ci na pewno powiedzieć, ze źle myślisz. Druga sprawa: clear() nie czyści pliku tylko flagi błędów.

http://www.cplusplus.com/reference/ios/ios/clear/

Sets a new value for the stream's internal error state flags.

A nawet jeśli by to robił, to chyba nie w tej kolejności wołasz funkcje, hm? :P

 

plik.close() ;
plik.clear() ;

Pokazałem ci twoje błędy. Gotowego rozwiązania raczej nie dostaniesz, bo takie mamy zasady na tym forum, no trudno :P 

Comments

Dziękuję za wskazówki, będę jeszcze szukał jak to wyświetlanie zrobić na różnych dokumentacjach.

---

Jeśli masz jeszcze jakieś pytania, to postaram się odpowiedzieć, ale raczej nie odrazu bo ide AFK właśnie

---

No to zamykanie dodałem w voidzie zapis(), zostało mi wsm tylko wyświetlanie dynamicznie płynącego czasu (nie mam pomysłu żadnego na to :C ) i wyczyszczenie zawartości notatnika z czasami na co też w referencji fstreama nie znalazlem sposobu.

---

@Criss, ogółem zrobiłem coś takiego

void wyswietlanie()
{

plik.open("Zapisy.txt", ios::in);
while (getline(plik, linia))
{
    switch (nr_linii)
    {
        case 1: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 2: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 3: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 4: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 5: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 6: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
        case 7: {czas=atoi(linia.c_str()); cout << czas <<endl;} break;
    }
    nr_linii++;
}

tylko konwertuje mi to czasy na liczby calkowite a jak chce double to co mam zmienić?

---

Po co ten switch skoro wszystie case-y robią to samo? I po co w ogóle wrzucasz to w atoi? cout umie printować stringi.

PS: do konwersji stringa na double służy atof()

---

Dziękuję :3 mam już w głowie zarys nawet jak zrobić wyświetlanie czasu na bierząco, bo z Sleep nie wypalilo.