Skip to content

Final Exercise#4

Open
Memfisto wants to merge 5 commits intoavl-cpp-training:masterfrom
Memfisto:master
Open

Final Exercise#4
Memfisto wants to merge 5 commits intoavl-cpp-training:masterfrom
Memfisto:master

Conversation

@Memfisto
Copy link
Copy Markdown

Ima jos dosta mjesta za improvement (exception handling za pocetak), ali evo neko osnovno rjesenje posto nisam siguran hocu li stici popraviti neke stvari u neko skorije vrijeme.

@ncaklovic
Copy link
Copy Markdown
Contributor

Samo par pitanja:

  • čini mi se da "move block" ne radi uopće
  • constants_amount = 1 u limit_block teško da je OK (sruši se program)
  • nema evaluate sa više vrijednosti (zadnji dio zadatka)?

@Memfisto
Copy link
Copy Markdown
Author

  • Nesto sam gadno propustio. Imam osjecaj da je radilo, ali gledajuci dokumentaciju za std::move ne vidim kako bi to bilo moguce. Isto tako sam zaboravio napisati test da provjeri radi li dobro.

  • Slazem se. Nisam stigao rucno testirati (a testovi na zalost nisu uhvatili taj dio)

  • Krivo sam shvatio zadnji dio zadatka. Cijelo vrijeme sam zamisljao to kao da korisnik bira datoteku u koju zeli spremiti sekvencu i da su "rezultati" samo blokovi. Sad vidim da moje tumacenje apsolutno nema nikakvog smisla.

Hvala na brzom feedbacku.

Imam jedno pitanjce, moze li se ikako spremiti kao member varijabla nesto oblika "std::function<double(double ...)>"? Moj pristup na pocetku je bio napraviti nekakav genericki blok koji ima svoju zadanu funkciju spremljenu kao std::function member (razlicitih signature-a da moze primiti dodatne parametre) umjesto da imam odvojene klase s funkcijom koja ima isti signature za svaki blok (na taj nacin ne bih morao dodatne parametre dodavati preko konstruktora), ali nikako nisam uspio pronaci nacin za to napraviti.

@ncaklovic
Copy link
Copy Markdown
Contributor

Imam jedno pitanjce, moze li se ikako spremiti kao member varijabla nesto oblika "std::function<double(double ...)>"?

Ne direktno koristeći std::function. Morao bi napisati svoj variadic template, Gordan piše takvo rješenje, ali još nije push-ao kod.

…r, added sequence evaluation with multiple values from a text file (rough version)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants