LucciaChia / sudoku-with-tests

2 stars 0 forks source link

New package console #36

Closed dodokpeter closed 5 years ago

dodokpeter commented 5 years ago

This package will have all classes connected to console displaying

dodokpeter commented 5 years ago

Methods: display (message), input()

andamundale commented 5 years ago

update all tests (and check other classes) to use console package

LucciaChia commented 5 years ago

I have problems with System.console() - it returns null for me. Please run Main class and try reproduce this. According to next article it might be impossible to use console(). https://intellij-support.jetbrains.com/hc/en-us/community/posts/206852195-Unable-to-use-java-groovy-console-while-debugging

I've also found multiple suggestions on the internet recomend to avoid using console()

dodokpeter commented 5 years ago

I have problems with System.console() - it returns null for me. Please run Main class and try reproduce this. According to next article it might be impossible to use console(). https://intellij-support.jetbrains.com/hc/en-us/community/posts/206852195-Unable-to-use-java-groovy-console-while-debugging

I've also found multiple suggestions on the internet recomend to avoid using console()

First test if console != null. if it is not, just use good old method from System.out.

LucciaChia commented 5 years ago

Incorrectly pushed also Main class here. It should've been pushed into #38

andamundale commented 5 years ago

https://github.com/LucciaChia/sudoku-with-tests/commit/2d903221fe606c0314926f14e713abf584f63675#r33599645