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 some problem #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

crousser
Copy link

@crousser crousser commented Jun 13, 2023

Потенциальные проблемы кода:

  • метод errorHandler передавался с потенциальной потерей контекста. К нему требовалось или явно применять bind или же использовать его как вызов (expresion)
  • метод getResp - имеет префикс get. Такие методы должны явно что-то возвращать. И обычно, какой-либо результат
  • если уж используется в коде fetch, то предполагается, что студенты знают что такое Promise и асинхронные вызовы. Использовать подходы в callback-style мне показалось излишним, тем более, что использование Promise избавляет от проблемы именования метода get

crousser added 4 commits June 13, 2023 20:23
fix: the name of a method that does not return a value must not start with `get`
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.

1 participant