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

Incorrect "pythagorean_triple" task example #975

Closed
IgorGakhov opened this issue Aug 27, 2022 · 7 comments
Closed

Incorrect "pythagorean_triple" task example #975

IgorGakhov opened this issue Aug 27, 2022 · 7 comments

Comments

@IgorGakhov
Copy link
Contributor

Ошибка примера в задании pythagorean_triple:

true == solution([3,5,4]) некорректен для x^2 + y^2 = z^2. Четверку и пятерку нужно поменять местами.

Что странно, на каком то этапе пример уже был в нужном порядке, но был изменен.

@IgorGakhov
Copy link
Contributor Author

PR: #976

@solar05 solar05 closed this as completed Aug 27, 2022
@vtm9
Copy link
Contributor

vtm9 commented Aug 28, 2022

Так, секундочку. Чем треугольник 3,4,5 отличается от 3,5,4?! Все там хорошо в задаче. Возможно твоя программа рассчитывает, что гипотенуза придет третьей, но она не обязана приходить третьей.

@IgorGakhov
Copy link
Contributor Author

IgorGakhov commented Aug 28, 2022

Да, расчет был из прихода гипотенузы третьим элементом в списке. Если порядок неважен, то в таком случае, по моему мнению, нужно это явно указать в описании задания.

@IgorGakhov
Copy link
Contributor Author

Я даже зная, что порядок был изменен, не догадался до этого. Прошу прощения тогда.

@vtm9
Copy link
Contributor

vtm9 commented Aug 28, 2022

В этом и смысл задачки. Она очень простая, но догадаться что надо отсортировать по длине часть решения

@vtm9
Copy link
Contributor

vtm9 commented Aug 28, 2022

пример как раз косвенно на это показывал)

@solar05
Copy link
Member

solar05 commented Aug 28, 2022

Можно задачке тогда еще соответствующий тег добавить!

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

3 participants