Timkor

reviewTriangles

Aug 6th, 2020 (edited)
325
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
  1. //ВАЖНО помнить, что ревью - это про код, а не про автора.
  2. // Ревью - это обмен опытом и мнением, аргументация и контраргументация, задача ревью - думать, узнавать, рассуждать и
  3. // становиться опытнее и грамотнее, прокачка одним словом.)
  4. // все могут ошибаться, и ревьювер в том числе.
  5.  
  6. // объединив три треугольника в одной программе вы сделали не только
  7. // лучше, но и хуже, т.к. вам пришлось использовать копипасту, т.е.
  8. // некоторые почти идентичные блоки кода - на подготовительном курсе
  9. // не стоит усложнять себе задачи, такая возможность - делать задачи
  10. // "со звездочкой" у вас еще будет.) А после прохождения "функций" вы сможете
  11. // делать это эффективнее.
  12.  
  13. // точку с запятой , вернее ее отсутствие, не все языки прощают,
  14. // лучше использовать. В js также есть ситуации , когда без нее
  15. // нельзя обойтись.
  16.  
  17. //именования: ask1 = triangleType; ask = triangleSize;
  18. let triangleType = parseInt(prompt('Какой тип треугольника тебе нужен 1, 2 или 3 ?'))
  19. let abzac //row (рядок, а есть еще col - колонка)
  20. let zvezd //asterisk or starSymb, а по коду col, positionInRow, starNum, starsCount
  21. let probel //spaceSymb, see above
  22. // transliteraciya budet viglyadet abrakodabroy dlya ne znayushih
  23. // russkiy yazik programmistov chitayushih vash kod
  24. // именования переменных очень важны, т.к. они или помогают понимать , писать , читать код, или...
  25. // путают и мешают.)
  26.  
  27.  
  28. if(triangleType == 1){
  29. let triangleSize = parseInt(prompt('Какого размера ты хочешь треугольник ?'))
  30. for(abzac = 0; abzac <= triangleSize; abzac = abzac + 1){ // abzac = abzac + 1 можно так ++abzac
  31.     console.log("") //  я бы переводил строку после вложенного цикла
  32.     for(zvezd = abzac; zvezd !== 0; zvezd = zvezd - 1) {
  33.         console.put("*")
  34.     } // не упускайте всегда использовать фигурные скобки, сэкономите время
  35.     // console.log() // я здесь бы переводил строку
  36.   }
  37. }
  38.  
  39.  
  40.  
  41. else if(triangleType == 2){
  42. let triangleSize = parseInt(prompt('Какого размера ты хочешь треугольник ?'))
  43. for(abzac = triangleSize; abzac > 0; abzac = abzac - 1){
  44.     console.log("") // console.log()
  45.     for(zvezd = 0; zvezd < abzac; zvezd = zvezd + 1){
  46.         console.put("*")
  47.     }
  48.   }
  49. }
  50.  
  51.  
  52.  
  53. else if(triangleType == 3){
  54. let triangleSize = parseInt(prompt('Какого размера ты хочешь треугольник ?'))
  55. //for(abzac = 0; abzac !== triangleSize; abzac = abzac + 1){
  56. for(abzac = 0; abzac < triangleSize; abzac = abzac + 1){    
  57.     console.log('')
  58.     /*for(probel = 0; probel < triangleSize; probel = probel + 1){
  59.         if(probel > abzac - 1) break  
  60.         console.put(" ")        
  61.     }*/
  62.     for(probel = 0; probel < abzac; probel = probel + 1){
  63.         //if(probel > abzac - 1) break  используйте для крайних случаев
  64.         console.put(" ")        
  65.     }
  66.  
  67.     for(zvezd = abzac; zvezd < triangleSize ; zvezd = zvezd + 1){
  68.         console.put("*")
  69.     }
  70.   }
  71. }
  72.  
  73.  
  74.  
  75. else{
  76.     alert("Такого треугольника нету")
  77. }
  78. // ваше решение вполне нормальное, не усложняйте себе задачи (на этом этапе), действуйте! Хорошего кода!
Add Comment
Please, Sign In to add comment