cyberpirate92/random-access-file-reader

Java library for reading/searching huge text files.

random-access-file-reader's Issues

cycleBackwards() appending 2 lines into a single one

cycleBackwards() method seems to be reading 2 lines instead of reading a single line

`readPreviousLine()` missing first character in first line

For some reason, readPreviousLine() is unable to obtain the first character in the first line, except for which everything seems to be working fine.

RandomAccessFileReader.readPreviousLine() adds unwanted newline character

readNextLine removes any trailing newline characters, make readPreviousLine also remove any trailing/preceding newline and/or carriage return characters, so that both are consistent.

use stringObj.replace("\n", "").replace("\r", "") on the returned string

Logic Bug

In the latest commit, cycleForward works, but cycleBackward is broken.

Current implementation uses a cursor position variable long bufferStartPointer which contains the cursor location for the first line in buffer,

initially, bufferStartPointer is set as 0

when a cycleForward operation is performed, bufferStartPointer cannot be updated with the new cursor position since the only way to obtain it is to store cursor position for every line that is read.