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.

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

Румянцев Даниил #1

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

Conversation

RDaniil
Copy link

@RDaniil RDaniil commented Feb 22, 2017

Я добавил код, который позволяет рисовать снежинку Коха, используя библиотеку p5.js

@disturm disturm changed the title Added snowflake code Румянцев Даниил Feb 26, 2017
@disturm disturm self-requested a review February 26, 2017 14:00
Copy link

@disturm disturm left a comment

Choose a reason for hiding this comment

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

Похоже p5 сделана как раз для непрограммистов и поэтому продвигает глобальные функции и неявные зависимости. Как следствие, код с ее использованием вызывает сомнения. Но инструмент интересный: возможно где-нибудь использую 😄

А твое решение мне понравилось 👍

@@ -0,0 +1,56 @@
var canvasH = 600;
Copy link

Choose a reason for hiding this comment

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

Не экономь на спичках. canvasHeight читается лучше.

var canvasH = 600;
var canvasW = 600;
var lineSize = 400;
var strokeCol = 200;
Copy link

Choose a reason for hiding this comment

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

Что такое col сходу не понятно: то ли column, то ли color. Кстати, col - это довольно стандартное сокращение именно для column.

translate(canvasW/2 + lineSize/2, canvasH/2 + lineSize/3);
rotate(PI/2);

instucions = createP("Слайдер - количество итераций (от 0 до 5). Кнопка - рисует снежинку");
Copy link

Choose a reason for hiding this comment

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

Неявное объявление переменной.

}
}

//Рисует кривую коха с "инициатором" длины length
Copy link

Choose a reason for hiding this comment

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

ненужный комментарий

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants