Java-to-Kotlin translation bug












0















Using the latest Kotlin 1.3.10 plugin from the latest IntelliJ IDEA 2018.3, if I translate the following Java code to Kotlin:



    BufferedReader br = Files.newBufferedReader(Paths.get("filename"));
String line;
while ((line = br.readLine()) != null)
System.out.println(line);


I get this:



    val br = Files.newBufferedReader(Paths.get("filename"))
var line: String
while ((line = br.readLine()) != null)
println(line)


The while ((line = br.nextLine()) != null) is idiomatic Java, but its automatic translation results in illegal syntax because assignments in Kotlin are not expressions.




  1. Is this a known bug in the Java-to-Kotlin translator?


  2. What's the Kotlin idiomatic syntax equivalent to the above Java code?











share|improve this question


















  • 2





    youtrack.jetbrains.com/issue/KT-25765

    – Thomas
    Nov 22 '18 at 11:09











  • I suppose that declaration at an upper level is the only way

    – Andrey Ilyunin
    Nov 22 '18 at 11:17






  • 1





    Partly a duplicate of stackoverflow.com/questions/36879236/…

    – Klitos Kyriacou
    Nov 22 '18 at 11:24






  • 1





    It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

    – Alexey Belkov
    Nov 22 '18 at 12:11
















0















Using the latest Kotlin 1.3.10 plugin from the latest IntelliJ IDEA 2018.3, if I translate the following Java code to Kotlin:



    BufferedReader br = Files.newBufferedReader(Paths.get("filename"));
String line;
while ((line = br.readLine()) != null)
System.out.println(line);


I get this:



    val br = Files.newBufferedReader(Paths.get("filename"))
var line: String
while ((line = br.readLine()) != null)
println(line)


The while ((line = br.nextLine()) != null) is idiomatic Java, but its automatic translation results in illegal syntax because assignments in Kotlin are not expressions.




  1. Is this a known bug in the Java-to-Kotlin translator?


  2. What's the Kotlin idiomatic syntax equivalent to the above Java code?











share|improve this question


















  • 2





    youtrack.jetbrains.com/issue/KT-25765

    – Thomas
    Nov 22 '18 at 11:09











  • I suppose that declaration at an upper level is the only way

    – Andrey Ilyunin
    Nov 22 '18 at 11:17






  • 1





    Partly a duplicate of stackoverflow.com/questions/36879236/…

    – Klitos Kyriacou
    Nov 22 '18 at 11:24






  • 1





    It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

    – Alexey Belkov
    Nov 22 '18 at 12:11














0












0








0








Using the latest Kotlin 1.3.10 plugin from the latest IntelliJ IDEA 2018.3, if I translate the following Java code to Kotlin:



    BufferedReader br = Files.newBufferedReader(Paths.get("filename"));
String line;
while ((line = br.readLine()) != null)
System.out.println(line);


I get this:



    val br = Files.newBufferedReader(Paths.get("filename"))
var line: String
while ((line = br.readLine()) != null)
println(line)


The while ((line = br.nextLine()) != null) is idiomatic Java, but its automatic translation results in illegal syntax because assignments in Kotlin are not expressions.




  1. Is this a known bug in the Java-to-Kotlin translator?


  2. What's the Kotlin idiomatic syntax equivalent to the above Java code?











share|improve this question














Using the latest Kotlin 1.3.10 plugin from the latest IntelliJ IDEA 2018.3, if I translate the following Java code to Kotlin:



    BufferedReader br = Files.newBufferedReader(Paths.get("filename"));
String line;
while ((line = br.readLine()) != null)
System.out.println(line);


I get this:



    val br = Files.newBufferedReader(Paths.get("filename"))
var line: String
while ((line = br.readLine()) != null)
println(line)


The while ((line = br.nextLine()) != null) is idiomatic Java, but its automatic translation results in illegal syntax because assignments in Kotlin are not expressions.




  1. Is this a known bug in the Java-to-Kotlin translator?


  2. What's the Kotlin idiomatic syntax equivalent to the above Java code?








kotlin






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 22 '18 at 11:03









DodgyCodeExceptionDodgyCodeException

3,5561424




3,5561424








  • 2





    youtrack.jetbrains.com/issue/KT-25765

    – Thomas
    Nov 22 '18 at 11:09











  • I suppose that declaration at an upper level is the only way

    – Andrey Ilyunin
    Nov 22 '18 at 11:17






  • 1





    Partly a duplicate of stackoverflow.com/questions/36879236/…

    – Klitos Kyriacou
    Nov 22 '18 at 11:24






  • 1





    It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

    – Alexey Belkov
    Nov 22 '18 at 12:11














  • 2





    youtrack.jetbrains.com/issue/KT-25765

    – Thomas
    Nov 22 '18 at 11:09











  • I suppose that declaration at an upper level is the only way

    – Andrey Ilyunin
    Nov 22 '18 at 11:17






  • 1





    Partly a duplicate of stackoverflow.com/questions/36879236/…

    – Klitos Kyriacou
    Nov 22 '18 at 11:24






  • 1





    It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

    – Alexey Belkov
    Nov 22 '18 at 12:11








2




2





youtrack.jetbrains.com/issue/KT-25765

– Thomas
Nov 22 '18 at 11:09





youtrack.jetbrains.com/issue/KT-25765

– Thomas
Nov 22 '18 at 11:09













I suppose that declaration at an upper level is the only way

– Andrey Ilyunin
Nov 22 '18 at 11:17





I suppose that declaration at an upper level is the only way

– Andrey Ilyunin
Nov 22 '18 at 11:17




1




1





Partly a duplicate of stackoverflow.com/questions/36879236/…

– Klitos Kyriacou
Nov 22 '18 at 11:24





Partly a duplicate of stackoverflow.com/questions/36879236/…

– Klitos Kyriacou
Nov 22 '18 at 11:24




1




1





It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

– Alexey Belkov
Nov 22 '18 at 12:11





It is a known bug, please vote for youtrack.jetbrains.com/issue/KT-6294

– Alexey Belkov
Nov 22 '18 at 12:11












1 Answer
1






active

oldest

votes


















2














I think the comments have answered both of my questions, so let me compile them into a single answer.




  1. Yes, it's a known bug, reported at youtrack.jetbrains.com/issue/KT-6294. You can vote for it (if you have a Jetbrains account) to show the developers that you would like it fixed. Thank you Alexey Belkov for bringing this to my attention.



  2. A common way to directly translate it would be:



    val br = Files.newBufferedReader(Paths.get("filename"))
    var line: String
    while (br.readLine().also { line = it} != null)
    println(line)


    However, a better way in this particular case is:



    val br = Files.newBufferedReader(Paths.get("filename"))
    br.forEachLine {
    println(it)
    }


    Or just



    File("filename").forEachLine {
    println(it)
    }


    (which uses a convenience method, equivalent to Java's convenience method Files.lines(Path)).








share|improve this answer

























    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53429526%2fjava-to-kotlin-translation-bug%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    1 Answer
    1






    active

    oldest

    votes








    1 Answer
    1






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    2














    I think the comments have answered both of my questions, so let me compile them into a single answer.




    1. Yes, it's a known bug, reported at youtrack.jetbrains.com/issue/KT-6294. You can vote for it (if you have a Jetbrains account) to show the developers that you would like it fixed. Thank you Alexey Belkov for bringing this to my attention.



    2. A common way to directly translate it would be:



      val br = Files.newBufferedReader(Paths.get("filename"))
      var line: String
      while (br.readLine().also { line = it} != null)
      println(line)


      However, a better way in this particular case is:



      val br = Files.newBufferedReader(Paths.get("filename"))
      br.forEachLine {
      println(it)
      }


      Or just



      File("filename").forEachLine {
      println(it)
      }


      (which uses a convenience method, equivalent to Java's convenience method Files.lines(Path)).








    share|improve this answer






























      2














      I think the comments have answered both of my questions, so let me compile them into a single answer.




      1. Yes, it's a known bug, reported at youtrack.jetbrains.com/issue/KT-6294. You can vote for it (if you have a Jetbrains account) to show the developers that you would like it fixed. Thank you Alexey Belkov for bringing this to my attention.



      2. A common way to directly translate it would be:



        val br = Files.newBufferedReader(Paths.get("filename"))
        var line: String
        while (br.readLine().also { line = it} != null)
        println(line)


        However, a better way in this particular case is:



        val br = Files.newBufferedReader(Paths.get("filename"))
        br.forEachLine {
        println(it)
        }


        Or just



        File("filename").forEachLine {
        println(it)
        }


        (which uses a convenience method, equivalent to Java's convenience method Files.lines(Path)).








      share|improve this answer




























        2












        2








        2







        I think the comments have answered both of my questions, so let me compile them into a single answer.




        1. Yes, it's a known bug, reported at youtrack.jetbrains.com/issue/KT-6294. You can vote for it (if you have a Jetbrains account) to show the developers that you would like it fixed. Thank you Alexey Belkov for bringing this to my attention.



        2. A common way to directly translate it would be:



          val br = Files.newBufferedReader(Paths.get("filename"))
          var line: String
          while (br.readLine().also { line = it} != null)
          println(line)


          However, a better way in this particular case is:



          val br = Files.newBufferedReader(Paths.get("filename"))
          br.forEachLine {
          println(it)
          }


          Or just



          File("filename").forEachLine {
          println(it)
          }


          (which uses a convenience method, equivalent to Java's convenience method Files.lines(Path)).








        share|improve this answer















        I think the comments have answered both of my questions, so let me compile them into a single answer.




        1. Yes, it's a known bug, reported at youtrack.jetbrains.com/issue/KT-6294. You can vote for it (if you have a Jetbrains account) to show the developers that you would like it fixed. Thank you Alexey Belkov for bringing this to my attention.



        2. A common way to directly translate it would be:



          val br = Files.newBufferedReader(Paths.get("filename"))
          var line: String
          while (br.readLine().also { line = it} != null)
          println(line)


          However, a better way in this particular case is:



          val br = Files.newBufferedReader(Paths.get("filename"))
          br.forEachLine {
          println(it)
          }


          Or just



          File("filename").forEachLine {
          println(it)
          }


          (which uses a convenience method, equivalent to Java's convenience method Files.lines(Path)).









        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 22 '18 at 13:19

























        answered Nov 22 '18 at 13:12









        DodgyCodeExceptionDodgyCodeException

        3,5561424




        3,5561424
































            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53429526%2fjava-to-kotlin-translation-bug%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            MongoDB - Not Authorized To Execute Command

            in spring boot 2.1 many test slices are not allowed anymore due to multiple @BootstrapWith

            Npm cannot find a required file even through it is in the searched directory