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

Compiler class refactoring: interface extraction #16

Open
ghost opened this issue Jun 17, 2015 · 3 comments
Open

Compiler class refactoring: interface extraction #16

ghost opened this issue Jun 17, 2015 · 3 comments
Milestone

Comments

@ghost
Copy link

ghost commented Jun 17, 2015

Wouldn't it be interesting, for future use, to have the Compiler class implement an interface with its essential methods defined in the interface ? That would

  1. make future implementations of compilers for other target languages easier, as developers would have a guide
  2. enforce a general "same look-and-feel" between future target languages compilers
  3. be simple good practice ( "program for interfaces instead of for concrete classes" )
@ghost
Copy link
Author

ghost commented Jun 17, 2015

As there is only one public method in Compiler, alternatively we could extract a superclass.

@nobeh
Copy link
Member

nobeh commented Jun 17, 2015

Hi Jan, I'd suggest having a look at this repo to get started with the tooling: https://github.com/CrispOSS/jabsc-maven-plugin-example

I am not sure if direct jabsc API usage would be of the most convenient way to start coding with it. Thanks.

@ghost
Copy link
Author

ghost commented Jun 17, 2015

Thanks for the suggestion, was not or hardly aware of this.

*Fortuna audaces adiuvat - hos solos ? *

On 17 June 2015 at 14:27, Behrooz Nobakht [email protected] wrote:

Hi Jan, I'd suggest having a look at this repo to get started with the
tooling: https://github.com/CrispOSS/jabsc-maven-plugin-example

I am not sure if direct jabsc API usage would be of the most convenient
way to start coding with it. Thanks.


Reply to this email directly or view it on GitHub
#16 (comment).

@nobeh nobeh added this to the 1.0.0 milestone Jun 21, 2015
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

1 participant