c++ Jak pokazac wszystkie dostepne pliki

Question

Hejo!

Mój projekt: https://github.com/jaroslawroszyk/-how-many-pages-per-day

1) Mam problem, poniewaz nie wiem jak pokazac dostepne pliki do zapisu :/

2) Co mogę poprawic w tym projekcie

Comments

2) Co mogę poprawic w tym projekcie

To:

void HowManyPages::enterdata()
{
    HowManyPages p;
    ...

    SaveToFile(p, en);
}

void HowManyPages::SaveToFile(HowManyPages &savedate, std::string &tit)
{
    ...
        save << ... << savedate.DateOfReading << ... << savedate.NumberOfPagesRead << '\n';
}

Dlaczego te metody nie działają na obiekcie this, tylko na tymczasowym/lokalnym lub podanym z zewnątrz?

tit powinien być const (jak i metoda SaveToFile).

---

referencja tam jest

---

No i?

---

ze nie dzialam na kopi obiektu tylko na orginale

---

Może źle się wyraziłem. "Tymczasowy" w rozumieniu, że tworzysz obiekt tylko po to, by zapisać jakieś dane w pliku, co jest bez sensu i przeczy idei obiektowości. Obie metody powinny odnosić się do obiektu this (po to one są metodami, a nie zwykłymi funkcjami).

---

@j23,  tit powinien być const (jak i metoda SaveToFile).

ta metoda jest statyczna

---

A po co?

---

jak uzyc tego this w tym kodzie? w tych metodach

---

save << "Date: " << this->DateOfReading << ", number of pages: " << this->NumberOfPagesRead << '\n';

 

---

Nie musisz jawnie używać this, wystarczy tak:

void HowManyPages::enterdata()
{
    std::string en = enterTitle();
    std::cout << "enter the day: ";
    std::cin >> DateOfReading;
    std::cout << "how many pages: ";
    std::cin >> NumberOfPagesRead;
    SaveToFile(en);
}

void HowManyPages::SaveToFile(const std::string &BookTitle) const // wywal static
{
    std::ofstream save(BookTitle.c_str(), std::ios::app);
    save << "Date: " << DateOfReading << ", number of pages: " << NumberOfPagesRead << '\n';
    if(save) std::cout << "Data has been saved !\n";
}

 

---

poprawione :)

---

Dobrze.

Zastanowiłbym się też nad zmianą nazwy klasy, bo HowManyPages to nie jest dobra nazwa według ogólnie przyjętych konwencji. Nazwa klasy powinna być rzeczownikiem.

Co do pierwszego pytania: pliki listujesz tak (w aktualnym katalogu):

for(auto& fn : std::filesystem::directory_iterator("."))
     if(std::filesystem::is_regular_file(fn)) {
         std::cout << fn.path() << '\n';
     }

 

---

to jaka nazwe moge dac zaproponuj mam pustke w glowie po prostu myhead = nullptr;

---

co do twojego kodu mam blad w filesysatem moglbys funkcje napisac kotra pokazuje .txt?

---

@j23

---

@j23, 

wiem ze on pokazuje co jest w aktualnym katalogu lecz mi chodzi o pliki .txt nie widze nawet tego w cppreference albo jestem slepy

---

Obawiam się, że to drugie ;)

    namespace fs = std::filesystem;
    
    for(auto& fn : fs::directory_iterator(".")) {
        if(fs::is_regular_file(fn) && fn.path().extension() == ".txt") {
            std::cout << fn.path() << '\n';
        }    
    }

Hint: std::filesystem::path, std::filesystem::directory_entry

to jaka nazwe moge dac zaproponu

Skoro klasa reprezentuje obiekt, który trzyma statystyki przeczytanych stron dla danej książki, to może BookStatistic?

---

naprawde dziekuje dzieki tobie poznalem troche c++17

Answer 1

Hej, co do 1) to rozwin prosze bo nie rozumiem.

2)

Skad pomysl na pliki cpp w katalogu src i headery w katalogu include? To ma byc biblioteka? Pytam, bo taki podzial na katalogi src, lib, include widywalem dotychczas w bibliotekach i ma to sluzyc temu zeby latwo bylo zidentyfikowac gdzie sa pliki dll albo lib ze zbudowanym kodem, gdzie zrodla a gdzie headery. Czy po prostu chcesz miec headery w jednym miejscu? Tak z ciekawosci pytam :)

fajnie, ze dodales jak zbudowac program, w execute to raczej nie 'cd /bin' a 'cd bin' albo 'cd ./bin'

Mozesz tez pokrotce opisac co robi program w README.

Zmienilbym nazwe enuma 'Pages' na cos co bardziej opisuje co to jest, moze ProgramMode?

Co znaczy prefix p_ w nazwach zmiennych?

Mozesz pooznaczac niektore parametry const jak np. savedate w tej metodzie mogloby byc const

HowManyPages::SaveToFile(HowManyPages &savedate, std::string &tit)

Mozesz tez popracowac nad msg commitow, widac, ze sie starasz ale czasami tez jest cos jak commit nr ~60 z message "INITIAL", wiec historia commitow wyglada srednio

Ogolnie spoko konsolowy programik

pozdro

Comments

1) spojrz teraz na kod na githubie prosze, chodzilo mi o to ze uzytkwonik po stworzeniu kilku plikow np, cleancode,c++guide itp... bedzie mogl widziec liste tych plików w programiem, wiem ze w c++ 17 jest std::filesystem ale nie wiem jak tego uzyc  :/

---

Co do katalogów chce miec headery w jednym miejscu i plii cpp w drugim :)

juz poprawilem w README cd /bin -> cd bin

"Mozesz tez pokrotce opisac co robi program w README." 

to juz na koniec projektu :)

"Zmienilbym nazwe enuma 'Pages' na cos co bardziej opisuje co to jest, moze ProgramMode?" 

slusznie dizekuje :)

 

Co znaczy prefix p_ w nazwach zmiennych?

pierwsza litera nazwy enuma, tak o w sumie :/

Mozesz tez popracowac nad msg commitow, widac, ze sie starasz ale czasami tez jest cos jak commit nr ~60 z message "INITIAL", wiec historia commitow wyglada srednio

Posiadasz moze jakas stronke z praktycznymi uwagami/radami co do msg commitow?

Ogolnie spoko konsolowy programik

Dziękuje

---

@dziablo,

Skad pomysl na pliki cpp w katalogu src i headery w katalogu include?

Przecież to dość powszechny podział plików w projektach C/C++ (bez znaczenia czy to aplikacja, czy biblioteka).

---

Posiadasz moze jakas stronke z praktycznymi uwagami/radami co do msg commitow?

Nie znam zadnych materialow, ale generalnie moim zdaniem idealne commit history to takie, ktore mozesz czytac i mniej wiecej wiesz co sie dzieje w projekcie.

Na przykladzie Twoich ost commitow co moznaby poprawic:

Rename enum class -> mozna dodac co z czego zmieniles 'Rename enum Pages to ProgramMode'

Adding static void function, which writes data to a file,  -> tu wydaje mi sie najbardziej informacji jakiej brakuje to jakie dane sa zapisywane do pliku, informacje o typie mozna pominac.

Generalnie starac sie dodac jak najwiecej istotnego info, bez rozpisywania sie, ale jesli robisz jakas kontrowersyjna zmiane warto sie rozpisac i dokladnie napisac w commicie dlaczego te zmiane robisz.

I roznie ludzie pisza commity zmienienie nazwy pliku moze byc:
Renamed file x to y

Rename file x to y

Renaming file x to y

Wazne zebys wybral w jakiej formie chcesz pisac msg i sie tej jednej formy trzymal, bo widzialem w historii, ze troche to sie mieszalo.

---

bede sie starac