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

Ghost getters/setters logic #5

Open
rjacas opened this issue May 6, 2013 · 4 comments
Open

Ghost getters/setters logic #5

rjacas opened this issue May 6, 2013 · 4 comments

Comments

@rjacas
Copy link
Collaborator

rjacas commented May 6, 2013

When calling a getter of a Ghost, thus creating such method, the Ghost Perspective ignore that the method is a getter, that is, it does not creates a corresponding variable. Same goes for setters.

Is such behavior desirable?

@etanter
Copy link
Member

etanter commented May 7, 2013

How would you be sure it is a getter?

@rjacas
Copy link
Collaborator Author

rjacas commented May 15, 2013

Apologies, due to a filter i just noticed this thread.

I'm guessing that there is no correct answer here. I would call a method
getter/setter if it complies with the getName()/setName() structure. Since
the Ghost is meant to be used, later on, for testing, there is no damage in
creating unnecesary members during testing, the user could perfectly erase
them later. I am inclined to believe that there are far more hits than false
positives with this aproach.

2013/5/6 Éric Tanter [email protected]

How would you be sure it is a getter?


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-17520106
.

lets reign all together

@etanter
Copy link
Member

etanter commented May 15, 2013

Ok - this could be a configurable feature ("introduce field with getter/setter ghost method").

@rjacas
Copy link
Collaborator Author

rjacas commented May 15, 2013

I agree, a configurable feature sounds good.

2013/5/15 Éric Tanter [email protected]

Ok - this could be a configurable feature ("introduce field with
getter/setter ghost method").


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-17960566
.

lets reign all together

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants