Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Steve Coulson: interview question 1 #10

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Steve Coulson: interview question 1 #10

wants to merge 16 commits into from

Conversation

sc3141
Copy link

@sc3141 sc3141 commented Apr 22, 2015

I used the programming question to reacquaint myself with python after a 4 year hiatus.

I have only very rarely ever written sql statements of the simplest variety. Nevertheless, I read up on the topic and then made an attempt.

… there was a tornado warning, and then, since the storm passed directly over my subdivision, I had to got home to check on pets and damage. Pets safe. House safe. Plants and trees shredded by hail.
…se 'abstract base class' as I was using the overloaded c++ meaning. replaced use of double quotes with single quotes. rearranged the order of classes and introduced class Operation.
…oved proscriptions of use (exceptions). Added unit tests for class Num and base class Operation.
…on), exposed some bugs in specializations of Operation (i.e. +,-,*,/). How about that\?\! Fixed python novice bug as follows: made public the private function __operate.
…this was outside the core concept. also in class Node, made member 'token' private. In class ExprTreeError, made member 'msg' private'. Renamed Operation.add() to 'addOperand'.
…ferences to member' __msg' had been renamed from 'msg'. In class Operation added method, term(), which provides access to individual operands.
…rtree module. changed exception throw by instance of class Num when value is not convertible to float (new error == NumError).
…ents. modified program to allow identification of token in vicinity of error condition.
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.

1 participant