Skip to content

Семантическая неточность#864

Merged
iliakan merged 1 commit intojavascript-tutorial:masterfrom
ArMANIAK:patch-2
Mar 1, 2020
Merged

Семантическая неточность#864
iliakan merged 1 commit intojavascript-tutorial:masterfrom
ArMANIAK:patch-2

Conversation

@ArMANIAK
Copy link
Contributor

Думаю, инкрементировать скорость в этом методе не совсем корректно, т.к. если заяц, например, побежал со скоростью 5:
rabbit.run(5);
а потом он устал и побежал со скоростью 3:
rabbit.run(3);
то в итоге мы получим скорость 8.
Конечно, это никак не влияет на суть урока, но как неточность ее можно поправить.
В любом случае спасибо за отличный учебник.

Думаю, инкрементировать скорость в этом методе не совсем корректно, т.к. если заяц, например, побежал со скоростью 5:
rabbit.run(5);
а потом он устал и побежал со скоростью 3:
rabbit.run(3);
то в итоге мы получим скорость 8.
Конечно, это никак не влияет на суть урока, но как неточность ее можно поправить. 
В любом случае спасибо за отличный учебник.
@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Feb 22, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team February 22, 2020 20:35
@lex111 lex111 requested a review from iliakan February 28, 2020 22:31
@iliakan
Copy link
Member

iliakan commented Feb 28, 2020

Сейчас посмотрел и тоже не понимаю, зачем там +=, почему не просто =

Есть идеи? @lex111 ?

@lex111
Copy link
Member

lex111 commented Feb 28, 2020

@iliakan нет, я поэтому и запросил ревью :)

@iliakan
Copy link
Member

iliakan commented Feb 29, 2020

Глянь плиз англ вариант, мб почему-то там надо += ?

@lex111
Copy link
Member

lex111 commented Feb 29, 2020

@iliakan вроде нет, мне кажется, при написании статьи, было предположение, что животное может только ускорятся, прибавлять свою скорость.

@iliakan iliakan merged commit 2e9c307 into javascript-tutorial:master Mar 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants