Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- // код на першому рев'ю:
- let c = 1;
- let num = +prompt("загадайте число");
- while (c <= num) {
- if (c < num) {
- console.put(c++ + ",");
- } else if (c = num) { // будьте уважні: присвоєння теж повертає значення, доречі яке? А яке значення поверне порівняння?
- console.put(c++ + ".");
- } else {
- console.put("чи така ситуація трапляється, залогуємо її"); // згідно умови вайлу - не трапляється.
- }
- }
- // код на друге рев'ю
- let c = 1;
- let num = +prompt("загадайте число"); // краще "введіть число"
- while (c <= num) {
- if (c < num) {
- console.put(c++ + ",");
- } else { // прибрали зайву умову - добре, чи розібралися що вона робила, коли відбувалася?)
- console.put(c++ + "."); // доречі крапку виводити не потрібно, краще просто не виводити кому
- }
- /* прибрали зайвий елс - добре*/
- }
- /* Ще на першому рев'ю були такі пропозиції:
- - наявні дві команди виводу числа, чи можна зменшити до однієї? - я мав на увазі, що є така задача з числами:
- "На весь код має бути лише одна команда виводу числа." - тобто корисно зробити так, щоб команда console.log(...) була одна.
- - для читабельності на навчальному рівні варто винести інкремент змінної (c++) до окремого рядку коду.
- Ще було запитання:
- будьте уважні: присвоєння теж повертає значення, доречі яке? А яке значення поверне порівняння?
- Звісно вам не обов'язково давати мені відповідь ,але важливо відповісти для себе. Хоча мені було б цікаво (та приємно) почути відповідь, а можливо і корисно.)
- Мені довелося порівняти ваш старий та новий код, щоб зрозуміти яки саме зміни ви зробили, яку взагалі задачу вирішували я зрозумів з коду, але було б дуже зручно для рев'ювера , якщоб б ви додали цю інформацію - тоді можна було б зосередитися на більш цінних порадах, не втрачаючи час. Доречі, рев'ювером вам теж доведеться бути, тоді краще мене зрозумієте */
Advertisement
Add Comment
Please, Sign In to add comment