These functions to find the value of the portfolio should definitely be in classes directly. That way, they can operate on every instance of the portfolio without having to rewrite code.