search

nurlandadasev / mobile-banking-app

mobile-banking-app
0 stars 0 forks source link

Checking the account number against the base does not work #2

Open aandrey1993 opened 3 years ago

aandrey1993 commented 3 years ago

Добрый день. Меня зовут Андрей и мы вместе с Роман делаем ревью Вашего тестового задания. Возникшие вопросы к функционалу откроем отдельными тикетами. Отпишите по ним, пожалуйста, ответы в комментариях или сделайте соответствующие фиксы в коде.

Описание

При добавлении нового счета в базе, номер счета генерируется автоматически. В методе createAndSaveAccountNumber делается проверка если сгенерированный номер уже есть в базе.

изображение

При работе с рекурсией, допущена ошибка, из-за которой метод createAndSaveAccountNumber может все-таки вернуть номер существующий в базе.

Задача

Если есть какие-то вопросы, можно задать их в комментарии.

nurlandadasev commented 3 years ago

Спасибо за предупреждение. Я увидел свою ошибку и её исправил.

28 мая 2021 г., в 17:08, aandrey1993 @.***> написал(а):

Добрый день. Меня зовут Андрей и мы вместе с Роман делаем ревью Вашего тестового задания. Возникшие вопросы к функционалу откроем отдельными тикетами. Отпишите по ним, пожалуйста, ответы в комментариях или сделайте соответствующие фиксы в коде.

Описание

При добавлении нового счета в базе, номер счета генерируется автоматически. В методе createAndSaveAccountNumber делается проверка если сгенерированный номер уже есть в базе.

https://user-images.githubusercontent.com/49781313/119995905-c55c5000-bfd6-11eb-883e-9eeb4338b8fc.png При работе с рекурсией, допущена ошибка, из-за которой метод createAndSaveAccountNumber может все-таки вернуть номер существующий в базе.

Задача

Найти допущенный просчет при работе с рекурсией Описать как можно его исправить Если есть какие-то вопросы, можно задать их в комментарии.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nurlandadasev/mobile-banking-app/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASDU4ELBSCQWUO3R3CBB6XTTP6PWDANCNFSM45WXA3VQ.

aandrey1993 commented 3 years ago

Спасибо за исправление. Вариант с циклом - рабочий. Для исправление предыдущего варианта с рекурсией, можно было добавить return перед повторным вызовом метода, чтобы успешный результат генерирования счета, был возвращен по всей цепочке вызовов:

изображение

nurlandadasev commented 3 years ago

Да всё верно после того как написал вариант с while протестировал для себя варанты с рекурсией. Спасибо!

пн, 31 мая 2021 г., 08:19 aandrey1993 @.***>:

Спасибо за исправление. Вариант с циклом - рабочий. Для исправление предыдущего варианта с рекурсией, можно было добавить return перед повторным вызовом метода, чтобы успешный результат генерирования счета, был возвращен по всей цепочке вызовов:

[image: изображение] https://user-images.githubusercontent.com/49781313/120143471-d67cab00-c1e8-11eb-80e1-c5a2df3124d2.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nurlandadasev/mobile-banking-app/issues/2#issuecomment-851182606, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASDU4EMWTH7V3CBU2DC2X7DTQML45ANCNFSM45WXA3VQ .