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

fix any to unknown #120

Closed
wants to merge 1 commit into from
Closed

Conversation

realing29
Copy link

Лучше использовать тип unknown вместо any, когда вы хотите указать, что тип аргумента неизвестен.

Тип any означает, что аргумент может быть любого типа, в то время как unknown означает, что тип аргумента неизвестен, но нужно указать, что с ним нужно делать.

Использование unknown вместо any позволяет избежать неожиданного поведения и ошибок при работе с неизвестными типами.

Поэтому в данном случае правильно будет написать function concat(a: unknown, b: unknown).

@bondiano
Copy link
Collaborator

bondiano commented Jan 8, 2023

К сожелению, функционал typescript для вывода типов не работает с перегрузками, вот issue про это microsoft/TypeScript#22609. Давай оставим такое решение

@bondiano
Copy link
Collaborator

bondiano commented Jan 8, 2023

поправили немного по-другому

@bondiano bondiano closed this Jan 8, 2023
@realing29 realing29 deleted the fixLesson19 branch January 9, 2023 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants