hirdbluebird / qualium-systems

0 stars 0 forks source link

мой падован #1

Open atherdon opened 6 years ago

atherdon commented 6 years ago

врятли залезут в ишью но я всетаки надеюсь что hr украины начнут чекать гитхабы. оле и диме привет! я друг чумаченко кости.

let rectangles = new Array(); - хуйня это. реально 0 живых проектов let rectangles = [] or let rectangles и все

start coordinate используется в одном месте можно ее и опустить, ноя понял что ты типо покрасоте хотел

добавь бля комментов перед каждой функцийе чтобы было хоть понятно че оно делает и виден твой скилл английского

отступы стермные - сам посмотри в гитхабе

если уже ты делаешь конструктор то бля уже юзани объект var figura = { init: function(){ //constructor here placed in first time }, constructor: func(), x:0, y:0 }

еще я бы хотел както намекнуть что ты в теме ноды и es6 есть let есть fat arrow может чтото еще?

этот код вообще работает? ну тоесть его где запускать и как надо? на ноде? просто это желатнльно тоже указать а то %самоцензура про неспособных рекрутеров% проебет и незапустится и все застрется

function Rectangle() и this.draw = () => {

тут сразу видно что ты это в разнов время писал. вначале медленно писал а потом спешил и использовал fat arrow

i--; l--;

        ну тут неприятно. невижу как подругому пока сделать но бля так точно не тру. зато тру то что ты закинул в фор инициализацию переменной

do while да хуета это. сделай также фором и не еби мозги страустраупу

// User score functions function updateUserScore() { scoreTable.innerHTML = userScore; } function resetUserScore() { userScore = 0; scoreTable.innerHTML = userScore;

понимаю но легко объединить их в

(value){ var = value } а чтобы не сильно тебя обижать - я бы написал комментарий в месте, где эти функции вызываются как // updating scoreTabl // reseting score to zero

function getRandomColor() { return "#"+((1<<24)*Math.random()|0).toString(16); }

тру но я б так не делал не говоря уже об отсутсвии отсупов и т.д. вот подумай как такую шляпу дебажитть если сломается чето?? +2 строчечки езе никого не убивали let value if let hex = value.16 return ""#""+hex

чтобы заценили эту крутоту можешь ее вначале в комментарии оставить. круто но бля ммерзко

if(this.coordinateY >= canvasHeight) this.coordinateY = startCoordinate; тру но обычно советуют не путать людей и и тут оставлять скобки

if (rectangleWidth > xAxisDifference > 0 && rectangleHeight > yAxisDifference > 0) return true; хуета спасибо что старался

let element = canvas, в таком случае можно назвать аргумент функции element и не ебать мозг

requestAnimationFrame где эта функций\я

startGameSession - может просто start?

atherdon commented 6 years ago

porn time!