alexanderLP-dev / Homework

0 stars 0 forks source link

homework 2019-10-25 #2

Open filonitta opened 4 years ago

filonitta commented 4 years ago

https://github.com/alexanderLP-dev/Homework/blob/4074d9833fc89ae4381928411a419be9aa75beef/homework%202019-10-25/javaScript.js#L66 не смутило, что при отмене в name попадает строка "null", хотя должен быть обычный null?

https://github.com/alexanderLP-dev/Homework/blob/4074d9833fc89ae4381928411a419be9aa75beef/homework%202019-10-25/javaScript.js#L90 в switch-case обычно определяют значение переменной, а после всех проверок, когда значение получено, с ним работают. Например, отображают

https://github.com/alexanderLP-dev/Homework/blob/4074d9833fc89ae4381928411a419be9aa75beef/homework%202019-10-25/javaScript.js#L108 два момента. Во-первых, зачем исходную строку конкатенировать с новой? Во-вторых, наклонные кавычки не используются для обычной строки. Они специальные - для специальных случаев: когда нужно в строке значение переменной вывести. Итого, правильно использовать их так: `You say: ${greeting}`

https://github.com/alexanderLP-dev/Homework/blob/4074d9833fc89ae4381928411a419be9aa75beef/homework%202019-10-25/javaScript.js#L133 бесполезная строчка :) Во-первых, month никак не используется в дальнейшем. Во-вторых, даже new Date().getMonth(10000) - все равно выдаст 9, текущий месяц, то есть x ни на что не влияет

filonitta commented 4 years ago

https://github.com/alexanderLP-dev/Homework/blob/325154cebb319c2a1af1e52bee2c34c3dfd8a351/homework%202019-10-25/javaScript.js#L23 при вариантах x = 0, y = 1, z = -1 найдет, что меньшее - x