Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task02_Lyschev #104

Open
wants to merge 2 commits into
base: Lischev
Choose a base branch
from
Open

Conversation

LyschevIvan
Copy link

No description provided.

функция optimize(oldArr : ArrayList<Operations) — оптимизация кода. Если подряд идет 2
одинаковые функции, то они переписываются в 1, повторяющуюся суммарное количество раз

функция isCorect() — функция проверяющая соответствие открытие-закрытие цикла
IV.Класс FileL позволяет читать данные из файла с помощью функции FileL (private val filename : String)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну важно отметить, что половина смысла этого класса --- это обработка исключений

одинаковые функции, то они переписываются в 1, повторяющуюся суммарное количество раз

функция isCorect() — функция проверяющая соответствие открытие-закрытие цикла
IV.Класс FileL позволяет читать данные из файла с помощью функции FileL (private val filename : String)
V.Класс Compiler — компилятор из кода Code в байт-код java


Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Еще думаю не лишним было бы написать, какая логика работы у методов класса. Там создается java class , конструктор для него, потом метод main и т.д.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Еще интересно, например, что для вывода класс использует stdlib Kotlin, а для ввода Java

VIII.Класс Petooh — аналогичный классу BrainFuck за исключением наличия функции
isSubstring(index : Int, prototype : String) которая проверяет совпадает ли строка кода, начинающая с index,
со строкой prototype
IX.Класс GeneralTests — внутренние тесты программы

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Про тесты тоже неплохо было бы написать, там, например перед всеми тестами инициируются потоки ввода вывода, а потом закрываются

со строкой prototype
IX.Класс GeneralTests — внутренние тесты программы

2)К пулреквесту практически нет комментариев и в основном они состоят из стилистических поправок кода

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А ваши комментарии к данному пул реквесту? Там в некоторых местах можно убрать методы get[что-нибудь] и сделать геттеры у полей и наверняка что-то еще можно сделать лучше.

@LyschevIvan
Copy link
Author

Добавлены более подробные описания

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants