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

Замечание по тестам для программы "Калькулятор" #92

Open
alexey-malov opened this issue Nov 25, 2024 · 3 comments

Comments

@alexey-malov
Copy link
Owner

В условии фигурируют только следующие ошибки:

  • Если введена неизвестная команда: "Unknown command"
  • Если синтаксис введенной команды не удовлетворяет правилам (в том числе недопустимый идентификатор): "Invalid usage"
  • Если переменная с таким именем уже была объявлена: "Name already exists"
  • Если в команде используется необъявленная переменная: “Name does not exist”

Однако тесты проверяют и другие. Тесты не должны предъявлять требования, не описанные в задании, иначе их ни одна программа не пройдёт.

@alexey-malov
Copy link
Owner Author

alexey-malov commented Nov 25, 2024

Пройдитесь по тестам и проверьте, что они проверяют то, что описано в условии задачи

@zxzmlk
Copy link

zxzmlk commented Nov 30, 2024

В условии фигурируют только следующие ошибки:

* Если введена неизвестная команда: "Unknown command"

* Если синтаксис введенной команды не удовлетворяет правилам (в том числе недопустимый идентификатор): "Invalid usage"

* Если переменная с таким именем уже была объявлена: "Name already exists"

* Если в команде используется необъявленная переменная: “Name does not exist”

Однако тесты проверяют и другие. Тесты не должны предъявлять требования, не описанные в задании, иначе их ни одна программа не пройдёт.

Перепроверил тесты и не совсем понял, какие именно вы имеете в виду. Если вы про тест 14 с делением на 0, то для этого случая в описании команд print и printfns есть уточнение: "Если при вычислении функции произошло деление на ноль, её значение считается неопределенным (nan), а сообщение об ошибке не выводится."

Тесты на идентификатор проверяют только требования, описанные в условии. То, что в таком случае выводится "Invalid usage", я указал. В целом все тесты, связанные с ошибкой нотации ( "Invalid usage"), следуют требованиям задания.

Тест 20, где в функции используется три переменных - проверяет правило нотации, описанное для команды fn.

В остальном все тесты соответствуют логике условия задачи и не требуют дополнительных уточнений (Исходя из моего понимания) . В блоке с ошибками я указал только те ситуации, которые не были описаны в блоке с описанием команд и которые требуют специального вывода

@zxzmlk
Copy link

zxzmlk commented Nov 30, 2024

Вопрос 1: в Pull-реквесте по данной задаче я также вносил изменения в условия бонусов. Считаются ли измененные условия бонусов принятыми и могу ли я переносить их в Яндекс. Контест в соответствующие задачи?

Вопрос 2: в задачах с бонусами Calculator (+speedup) и Calculator (+speedup, +deep-recursion) должны присутствовать все предыдущие тесты из задачи Calculator или только тесты, связанные с бонусом? Тот же вопрос к условиям заданий - в условии указывать только то, что относится к бонусу или все задание целиком?

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

No branches or pull requests

2 participants