Комментарии к проекту

[jskonst]

Хорошо бы сделать PR в пустую ветку чтобы можно было прокомментировать, но сейчас постараюсь тут описать

1. Из улучшений можно было бы использовать pipenv (не обязательно - но для питона часто используют)

    def create_table(self):
        with self.connection:
            self.cursor.execute(
                """CREATE TABLE IF NOT EXISTS ships(
                    id varchar,
                    Aircraft int[][],
                    Battleship int[][],
                    Cruiser int[][],
                    Submarine int[][],
                    Carrier int[][]);"""
            )

Я бы сказал что это все же 1 таблица - ship

1. Можно было бы посмотреть на sqlAlchemy - но пока точно не надо
2. По следующему коду:

@app.route("/fire")
def fire():
    """Makes a shot at the coordinates of the field."""
    global move
    row = request.args.get("rowIndex", default=-1, type=int)
    column = request.args.get("columnIndex", default=-1, type=int)
    result = field.fire(row, column)
    move += 1
    print(move, result)
    return redirect(url_for("index"))

1. НИКОГДА не используйте глобальную переменную :)
2. Всю логику стоит вынести из контроллера в Сервис (отдельный класс отдельный метод не в App)