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

Finished but with a fair bit of questions #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Finished but with a fair bit of questions #15

wants to merge 1 commit into from

Conversation

mikeadeleke
Copy link

All done but I have some questions:

  1. class variables were a struggle
  2. trouble with making a count
  3. TDD

@jwo
Copy link
Member

jwo commented Jan 13, 2014

Hi mike!

Sorry I somehow missed this one from you :( Will get right on it.


class Automobile < ActiveRecord::Base
def initialize
@color = color
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only reason this may appear to work is that ActiveRecord takes control of the initialize method.

So, when you use ActiveRecord, you can't use initialize -- you have to use other weird methods.

tl;dr -- ActiveRecord will handle the assignments when you do automobile = {color: "Black", make: "SUV", model: "Range Rover", year: 2013}

@jwo
Copy link
Member

jwo commented Jan 13, 2014

OK, read over and let me know --

Also, the TDD will make more sense once you do the first 2 episodes in the TDD course.

@mikeadeleke
Copy link
Author

Thanks! Sorry for my lack of involvement the last couple of weeks. I am
taking two courses on Thinkful.com, my startup is growing quickly, and I
have a part-time job. I decided to come back to Ruby off Rails when things
die down.

Take care,

Mike Adeleke
Co-founder at Domi
219-472-9882
www.domiapp.co

You've been dying to know how Domi is doing.
Now you can find out
https://medium.com/i-woke-up-in-a-new-lean-startup/66ac21d26843.

On Mon, Jan 13, 2014 at 9:19 AM, Jesse Wolgamott
[email protected]:

OK, read over and let me know --

Also, the TDD will make more sense once you do the first 2 episodes in the
TDD course.


Reply to this email directly or view it on GitHubhttps://github.com//pull/15#issuecomment-32178013
.

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