Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Сделал работу Снежинка Коха. Макушев Артемий #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Сделал работу Снежинка Коха. Макушев Артемий #3

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
151 changes: 49 additions & 102 deletions index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,110 +1,57 @@
<!doctype html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="Cache-Control" content="no-cache"/>

<body>
<script>
//пример рисования произвольного множества точек
function example1() {
//получаем html-элемент типа canvas и его характеристики
var canvas = document.getElementById("canvas1");
/*============================================================================================*/
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такие комментарии делать не нужно. Сразу после тэга скрипт нет смысла. В остальных частях кода: либо пустые строчки, либо код в функции, либо разные файлы.

function PaintSnowflake(){
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JS != C#. В JS всегда работает camelCase. Исключения мы разобрали - название конструкторов и именование констант.

var iteration = document.getElementById('iteration');
var n = iteration.value;
var canvas = document.getElementById("canvas");
var canvasHeight = parseInt(canvas.getAttribute("height"));
var canvasWidth = parseInt(canvas.getAttribute("width"));

//создаем 2d context
var context = canvas.getContext('2d');

//создаем "буфер" imageData, в который будем класть новую информацию о цветах
var imageData = context.createImageData(canvasWidth, canvasHeight);
for (var i = 0; i < canvasWidth; i++) {
for (var j = 0; j < canvasHeight; j++) {
var red = 0;
var green = 0;
var blue = (i + j) % 255;
if (i % 20 == 0)
red = 150;
if (j % 20 == 0)
green = 150;
var opacity = 255;

imageData.data[4*(i + canvasWidth*j) + 0] = red;
imageData.data[4*(i + canvasWidth*j) + 1] = green;
imageData.data[4*(i + canvasWidth*j) + 2] = blue;
imageData.data[4*(i + canvasWidth*j) + 3] = opacity;
}
}
//заполненный "буфер" imageData передаем в context для вывода на экран
context.putImageData(imageData, 0, 0);
}

//пример рисования отдельных линий
function example2() {
//получаем html-элемент типа canvas и его характеристики
var canvas = document.getElementById("canvas2");
var canvasHeight = parseInt(canvas.getAttribute("height"));
var canvasWidth = parseInt(canvas.getAttribute("width"));

//создаем 2d context
var context = canvas.getContext('2d');

//задаем стиль линий
context.lineWidth = "5";
context.strokeStyle = "green";

//начинаем создание нового пути
context.beginPath();
//задаем начальную точку линии
context.moveTo(0, 75);
//задаем конечную точку линии и добавляем линию в путь
context.lineTo(150, 100);
//выводим путь на экран
context.stroke();
}

//пример рисования полилиний (сначала вычисляем точки, затем рисуем)
function example3() {
//получаем html-элемент типа canvas и его характеристики
var canvas = document.getElementById("canvas3");
var canvasHeight = parseInt(canvas.getAttribute("height"));
var canvasWidth = parseInt(canvas.getAttribute("width"));

//создаем 2d context
var context = canvas.getContext('2d');

//задаем стиль линий
context.clearRect(0, 0, canvas.width, canvas.height);
context.lineWidth = "2";
context.strokeStyle = "#FF0000";

//создаем массив точек
//в качестве точек используются анонимные объекты со свойствами x и y
var points = [{x: 20, y: 50}, {x: 120, y: 50}, {x: 120, y: 100}, {x: 50, y: 20}];

//начинаем создание нового пути
context.beginPath();
//добавляем в путь все точки из массива
for (var i = 0; i < points.length; i++) {
var point = points[i];
if (i == 0)
context.moveTo(point.x, point.y);
else
context.lineTo(point.x, point.y);
}
//выводим путь на экран
context.stroke();
}

//вызывается после загрузки body
function run () {
example1();
example2();
example3();
}
</script>

<body onload="run()">
<canvas height='200' width='200' id='canvas1'></canvas>
<canvas height='200' width='200' id='canvas2'></canvas>
<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}];
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты чуть далее так часто обращаешься к точкам points[0] и points[1]. Что стоило бы сделать

var point1 = ...
var point2 = ...
var point3 = ...
var points = [point1, point2, point3];

Тем более, что point3 ты сейчас считаешь после создания массива

//вычисление координат третьей точки равностороннего трекгольника
points[2].x = (points[1].x-points[0].x)*Math.cos(Math.PI/3) - (points[1].y-points[0].y)*Math.sin(Math.PI/3) + points[0].x;
points[2].y = (points[1].x-points[0].x)*Math.sin(Math.PI/3) + (points[1].y-points[0].y)*Math.cos(Math.PI/3) + points[0].y;
DrawCochLine(context, points[1], points[0], Math.PI, n);
DrawCochLine(context, points[2], points[1], -Math.PI/3, n);
DrawCochLine(context, points[0], points[2], Math.PI/3, n);
}
/*============================================================================================*/
//firstPoint - первая точка отрезка
//secondPoint - вторая точка отрезка
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Google Translate подсказывает: firstPoint - Первая точка
Оставь ему эту работу :-)

Чуть глубже: старайся выразить смысл через название переменных и методов. И только в крайних случаях пиши комментарии.
Крайние случае - это:

  • ты делаешь костыль и пишешь зачем
  • ты делаешь комментарии и публичному API своего сервиса или библиотеки.

//fi - угол поворота отрезка
//n - кол-во итераций
function DrawCochLine(context, firstPoint, secondPoint, fi, n){
if (n <= 0){
context.beginPath();
context.moveTo(firstPoint.x, firstPoint.y);
context.lineTo(secondPoint.x, secondPoint.y);
context.stroke();
} else {
var length = Math.sqrt(Math.pow(secondPoint.y - firstPoint.y, 2) + Math.pow(secondPoint.x - firstPoint.x, 2));
var third = length / 3;
//находим точку, находящуюся на 1/3 длины отрезка от первой точки
var newPoint1 = {x: firstPoint.x + parseInt(Math.round(third * Math.cos(fi))), y: firstPoint.y + parseInt(Math.round(third * Math.sin(fi)))};
//находим точку, находящуюся на 2/3 длины отрезка от первой точки
var newPoint2 = {x: newPoint1.x + parseInt(Math.round(third * Math.cos(fi))), y: newPoint1.y + parseInt(Math.round(third*Math.sin(fi)))};
//находим вершину нового треугольника
var peak = {x: newPoint1.x + parseInt(Math.round(third * Math.cos(fi + Math.PI/3))), y: newPoint1.y + parseInt(Math.round(third * Math.sin(fi + Math.PI/3)))};
n--;
DrawCochLine(context, newPoint1, peak, fi + Math.PI/3, n);
DrawCochLine(context, peak, newPoint2, fi - Math.PI/3, n);
DrawCochLine(context, firstPoint, newPoint1, fi, n);
DrawCochLine(context, newPoint2, secondPoint, fi, n);
}
}
/*============================================================================================*/
</script>
<button onclick="PaintSnowflake();">Построить</button>
<input type="text" class="form-control" name="iteration" id="iteration" placeholder="Сколько итераций">
<canvas height='700' width='1000' id='canvas'></canvas>
</body>
</html>