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

Added Consumer Java 8 feature #39

Merged
merged 1 commit into from
Aug 1, 2018
Merged

Conversation

fmarazita
Copy link

Hi Ozler,

thanks a lot for your email. I have submitted the modifications, please contact me if you have any question.

Thanks a lot
Fabrizio

@coveralls
Copy link

coveralls commented Jul 25, 2018

Coverage Status

Coverage increased (+0.2%) to 87.037% when pulling 4bb1d32 on fmarazita:master into d5dd112 on ozlerhakan:master.

@ozlerhakan
Copy link
Owner

Hi @fmarazita ,

I'll look at your changes, thank you.

@fmarazita
Copy link
Author

Hi Ozler,

I forgot to change
.travis.yml -> deleting the Java 7

Alle the best
Fabrizio

@ozlerhakan
Copy link
Owner

Hi @fmarazita ,

Could you give me an example usage for that case, so that I can mention it on README?

@fmarazita
Copy link
Author

fmarazita commented Aug 1, 2018

I think you can explain that this feature is very usefull for:

  • Huge excel file ( without you have all in memory)
  • Run time processing/ filtering data
  • DB batch insertion

Calculation class

  • @ExcelCell(0) String Name
  • @ExcelCell(1) int a
  • @ExcelCell(2) int b

In Main:
File fileCalculation = new File(example.xlsx);
PoijiOptions options = PoijiOptionsBuilder.settings(1).sheetIndex(1).build();
Poiji.fromExcel(fileCalculation, Calculation.class, options, this::dbInsertion);

Function dbInsertion
private void dbInsertion(Calculation siCalculation) {
int value= siCalculation.getA() + siCalculation.getB();
String name = siCalculation.getName();
return insertDB(name , value);
}

Thanks a lot for your time!!

@fmarazita fmarazita closed this Aug 1, 2018
@fmarazita fmarazita reopened this Aug 1, 2018
@fmarazita
Copy link
Author

fmarazita commented Aug 1, 2018

Sorry @ozlerhakan I clicked on the Close and comment instead of Comment, now is again opened.

@ozlerhakan
Copy link
Owner

Thank you for the detailed examples @fmarazita

@ozlerhakan ozlerhakan merged commit 4bb1d32 into ozlerhakan:master Aug 1, 2018
@ozlerhakan
Copy link
Owner

Thanks @fmarazita merged!

@fmarazita
Copy link
Author

@ozlerhakan thanks a lot for your collaboration!

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.

None yet

3 participants