Сделал работу Снежинка Коха. Макушев Артемий #3
Conversation
| //получаем html-элемент типа canvas и его характеристики | ||
| var canvas = document.getElementById("canvas1"); | ||
| /*============================================================================================*/ | ||
| function PaintSnowflake(){ |
There was a problem hiding this comment.
JS != C#. В JS всегда работает camelCase. Исключения мы разобрали - название конструкторов и именование констант.
| function example1() { | ||
| //получаем html-элемент типа canvas и его характеристики | ||
| var canvas = document.getElementById("canvas1"); | ||
| /*============================================================================================*/ |
There was a problem hiding this comment.
Такие комментарии делать не нужно. Сразу после тэга скрипт нет смысла. В остальных частях кода: либо пустые строчки, либо код в функции, либо разные файлы.
| <canvas height='200' width='200' id='canvas3'></canvas> | ||
| </body> | ||
| context.strokeStyle = "black"; | ||
| var points = [{x: 100, y: 200}, {x: 600, y: 200}, {x: 0, y: 0}]; |
There was a problem hiding this comment.
Ты чуть далее так часто обращаешься к точкам points[0] и points[1]. Что стоило бы сделать
var point1 = ...
var point2 = ...
var point3 = ...
var points = [point1, point2, point3];
Тем более, что point3 ты сейчас считаешь после создания массива
| } | ||
| /*============================================================================================*/ | ||
| //firstPoint - первая точка отрезка | ||
| //secondPoint - вторая точка отрезка |
There was a problem hiding this comment.
Google Translate подсказывает: firstPoint - Первая точка
Оставь ему эту работу :-)
Чуть глубже: старайся выразить смысл через название переменных и методов. И только в крайних случаях пиши комментарии.
Крайние случае - это:
- ты делаешь костыль и пишешь зачем
- ты делаешь комментарии и публичному API своего сервиса или библиотеки.
No description provided.