Timkor

commaSeparatedValuesReview2

Feb 19th, 2021
795
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
  1. // код на першому рев'ю:
  2. let c = 1;
  3. let num = +prompt("загадайте число");
  4. while (c <= num) {
  5.     if (c < num) {
  6.         console.put(c++ + ",");
  7.     } else if (c = num) { // будьте уважні: присвоєння теж повертає значення, доречі яке? А яке значення поверне порівняння?
  8.         console.put(c++ + ".");
  9.     } else {
  10.         console.put("чи така ситуація трапляється, залогуємо її"); // згідно умови вайлу - не трапляється.
  11.     }
  12. }
  13.  
  14. // код на друге рев'ю
  15. let c = 1;
  16. let num = +prompt("загадайте число"); // краще "введіть число"
  17. while (c <= num) {
  18.     if (c < num) {
  19.         console.put(c++ + ",");
  20.     } else { // прибрали зайву умову - добре, чи розібралися що вона робила, коли відбувалася?)
  21.         console.put(c++ + "."); // доречі крапку виводити не потрібно, краще просто не виводити кому
  22.     }
  23.     /* прибрали зайвий елс - добре*/
  24. }
  25.  
  26. /* Ще на першому рев'ю були такі пропозиції:
  27. - наявні дві команди виводу числа, чи можна зменшити до однієї?  - я мав на увазі, що є така задача з числами:
  28. "На весь код має бути лише одна команда виводу числа." - тобто корисно зробити так, щоб команда console.log(...) була одна.
  29.  
  30. - для читабельності на навчальному рівні варто винести інкремент змінної (c++) до окремого рядку коду.
  31.  
  32. Ще було запитання:
  33.  будьте уважні: присвоєння теж повертає значення, доречі яке? А яке значення поверне порівняння?
  34. Звісно вам не обов'язково давати мені відповідь ,але важливо відповісти для себе. Хоча мені було б цікаво (та приємно) почути відповідь, а можливо і корисно.)
  35. Мені довелося порівняти ваш старий та новий код, щоб зрозуміти яки саме зміни ви зробили, яку взагалі задачу вирішували я зрозумів з коду, але  було б дуже зручно для рев'ювера , якщоб б ви додали цю інформацію -  тоді можна було б зосередитися на більш цінних порадах, не втрачаючи час. Доречі, рев'ювером вам теж доведеться бути, тоді краще мене зрозумієте */
Advertisement
Add Comment
Please, Sign In to add comment