Как улучшить этот код(в плане ооп)?

Question

[da da]

class MicroPythonBoard():

    def __init__(self, port, baudrate):
        self.serialPort=serial.Serial(port, baudrate)
    
    def putCursorOnNewLine(self):
        self.serialPort.write(b"\r")

    def writeln(self, string):
        """ Write a line to serial poart """
        self.serialPort.write( bytes(string, "ascii"))
        self.putCursorOnNewLine()
    

    def write(self, string):
        """write a string to serial port """
        self.serialPort.write( bytes(string, "ascii") )


    def writeFile(self, pathToFile, fileName):
        """
        pathToFile - path to file on your computer
        fileName - Name of file on your MicroPython board
        """
        with open(pathToFile) as file:
            
            self.serialPort.write((chr(5).encode("ascii"))) # write <CTRL+E> (PASTE MODE)

            self.serialPort.write((f'with(open("{fileName}", "w")) as f:').encode("ascii"))

            self.putCursorOnNewLine()

            self.serialPort.write(b"    f.write(r'''")

            self.putCursorOnNewLine()

            for line in file.readlines():
                self.serialPort.write(line.encode("ascii"))
            
            self.putCursorOnNewLine()
            self.serialPort.write(b"''')")

            self.serialPort.write(chr(4).encode("ascii")) # write <CTRL+D> (CLOSE PASTE MODE)

            print("DONE")
    
    def readLastLine(self):
        all_text=self.serialPort.read_all()
        last_line=all_text.split(b"\r")[-1]
        return last_line

Comments

[MinTnt]

Кстати, а почему под написанием с новой строки, вы используете \r вместе \n, если \r просто "устанавливает курсор" в начале?

[da da]

MinTnt, в MicroPython так устроено

Answer 1

[MinTnt]

Хоть и не мне это говорить. Но ООП идёт к минимализации повторяющихся строк, что выполняют одну и ту же работу.

К примеру. Здесь повторяется одна и та же строка.

self.serialPort.write( bytes(string, "ascii") )

def writeln(self, string):
        """ Write a line to serial poart """
        self.serialPort.write( bytes(string, "ascii"))
        self.putCursorOnNewLine()
    

    def write(self, string):
        """write a string to serial port """
        self.serialPort.write( bytes(string, "ascii") )

Так что всё можно сократить до одной функции, просто добавив ещё одной входной параметр

сократить до одной функции

def write(self, string, nextline=False):
        """write a string to serial port """
        self.serialPort.write( bytes(string, "ascii") )
        if nextline:
            self.putCursorOnNewLine()

#И далее вызывать, функцию write('text') как и раньше, а функцию writeln - как write('text', True)

Comments

[da da]

Логично, спасибо