I'd prefer a function than a method to get the positions from a log file. I find it not natural to have self being a string for a non static method.