Is something wrong with my code here(doing java.io.File writing in a coroutine)

class AsyncLogger : AbstractLogger {

    constructor(filepath: String,name: String) : super() {
        var file = File(filepath)
        file.mkdirs()
        filename = "$filepath/$name"
        file = File(filename)
    }

    suspend fun log(toLog: Boolean,msg: String) = coroutineScope {
        GlobalScope.launch {
            if (toLog)
                writeToFile(msg)
        }
    }

    suspend fun log(toLog: Boolean,toFile: Boolean,msg: String) = coroutineScope {
        GlobalScope.launch {
            if(toLog)
                when(toFile) {
                    true -> writeToFile(msg)
                    false -> writeToFile(msg)
                }
        }
    }

    override suspend fun writeToSTDOUT(msg: String) =
            println(message = "[${Thread.currentThread().name}]: $msg")

    override suspend fun writeToFile(msg: String): Job = coroutineScope {
        GlobalScope.launch {
            val stream = RandomAccessFile(filename, "rw")
            val channel = stream.channel
            var lock: FileLock? = null
            try {
                lock = channel.tryLock()
            } catch (e: OverlappingFileLockException) {
                stream.close()
                channel.close()
            }
            stream.writeBytes(msg + '\n')
            lock?.release()
            stream.close()
            channel.close()
        }
    }

}

m guess is that you are still doing work on the stream, but since you just launch and do not wait it closes it while still working on it.. where that code is.. i do no know on first glance

so should i do a runBlocking call ?

i have honestly no cle if that would be better

hmm thnx anyways

with

use {}

you don't have to bother about closing the streams yourself ...

hmm thnx

Think locking a file on every write operation is a bad idea. Such a writer can be implemented using an actor:

override suspend fun writeToFile(msg: String) = scope.actor<String> {
    for (msg in this) {
        stream.writeBytes("$msg\n")
    }
}