Skip to content

FinalExercise#1

Open
DLeibner wants to merge 1 commit intoavl-cpp-training:masterfrom
DLeibner:master
Open

FinalExercise#1
DLeibner wants to merge 1 commit intoavl-cpp-training:masterfrom
DLeibner:master

Conversation

@DLeibner
Copy link
Copy Markdown

Nisam napravio unit testove za sequence i učitavanje sekvence iz file-a pri pokretanju, to ću kad uhvatim vremena :)

@ncaklovic
Copy link
Copy Markdown
Contributor

ncaklovic commented Nov 19, 2018

Nije li druga polovica cijeli executeSequenceFromFile obični std::transform (output je ostream_iterator)?
A executeSequence zvuči kao std::accumulate...

Zar ti je stvarno čitkiji kod sa ok varijablom od nešto tipa

bool Sequence::deleteBlock(int index)
{
  if (index > blocks.size() || index <= 0)
    return false;
  blocks.erase(blocks.begin() + index - 1);
  return true;
}

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