1,2,3 Lessons were done full by StarKRE · Pull Request #152 · Kotlin-Polytech/KotlinAsFirst2016

StarKRE · 2016-09-30T19:41:00Z

Михаил Игоревич, извините за предыдущий Commit, буду относиться с пониманием...

…IIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIII

…lllllllllllllllllllllllllllllllllllll

ice-phoenix

Просьба внимательнее относиться к форматированию кода

ice-phoenix · 2016-10-01T08:14:42Z

src/lesson1/task1/Simple.kt

+    minutes >= 0
+    minutes <= 60
+    seconds >= 0
+    seconds <= 60


Что, по вашему мнению, делает данная цепочка логических выражений?

ice-phoenix · 2016-10-01T08:15:00Z

src/lesson1/task1/Simple.kt

 */
-fun lengthInMeters(sagenes: Int, arshins: Int, vershoks: Int): Double = TODO()
+fun lengthInMeters(sagenes: Int, arshins: Int, vershoks: Int): Double {
+    return vershoks * 0.04445 + arshins * 16 * 0.04445 + sagenes * 48 * 0.04445


Функцию с телом в виде одного return expr можно записать как fun foo(...) = expr

ice-phoenix · 2016-10-01T08:15:54Z

src/lesson1/task1/Simple.kt

 */
-fun angleInRadian(grad: Int, min: Int, sec: Int): Double = TODO()
+fun angleInRadian(grad: Int, min: Int, sec: Int): Double {
+    return (grad.toDouble() + min.toDouble()/60 + sec.toDouble()/3600) / (180 / Math.PI)


Функцию с телом в виде одного return expr можно записать как fun foo(...) = expr

Для того, чтобы результат деления имел тип Double, обычно делают a / 60.0, а не a.toDouble() / 60

ice-phoenix · 2016-10-01T08:16:15Z

src/lesson1/task1/Simple.kt

 */
-fun trackLength(x1: Double, y1: Double, x2: Double, y2: Double): Double = TODO()
+fun trackLength(x1: Double, y1: Double, x2: Double, y2: Double): Double {
+    return sqrt(sqr(x2-x1)+sqr(y2-y1))


Функцию с телом в виде одного return expr можно записать как fun foo(...) = expr

ice-phoenix · 2016-10-01T08:16:28Z

src/lesson1/task1/Simple.kt

 */
-fun thirdDigit(number: Int): Int = TODO()
+fun thirdDigit(number: Int): Int {
+    return (number/100)%10


Функцию с телом в виде одного return expr можно записать как fun foo(...) = expr

ice-phoenix · 2016-10-01T08:36:52Z

src/lesson3/task1/Loop.kt

+        while (number < n) {
+            k = k + 1
+            s = fib(k)
+            while (s > 0 ){


См. замечания выше

ice-phoenix · 2016-10-01T08:37:16Z

src/lesson4/task1/List.kt

+fun q2(g:Double) = g * g
+
+fun abs(v: List<Double>): Double{
+   var sum: Double = 0.0


Система типов Котлина выведет тип переменной автоматически

ice-phoenix · 2016-10-01T08:38:13Z

src/lesson4/task1/List.kt

+
+fun abs(v: List<Double>): Double{
+   var sum: Double = 0.0
+    for (i in 0 .. v.size-1){


Если вам явно не нужен индекс значений в списке, лучше воспользоваться циклом в foreach стиле

for (element in list) { foo(element) }

ice-phoenix · 2016-10-01T08:38:52Z

src/lesson4/task1/List.kt

-fun mean(list: List<Double>): Double = TODO()
+fun mean(list: List<Double>): Double {
+    var sum : Double = 0.0
+    for (i in 0 .. list.size - 1){


Можно просто list.sum()

ice-phoenix · 2016-10-01T08:39:41Z

src/lesson4/task1/List.kt

+fun times(a: List<Double>, b: List<Double>): Double {
+    var min: Int = 0
+    var sum: Double = 0.0
+    if (a.size < b.size){


Скалярное произведение векторов разных размерностей неопределено

Определил, нашел наименьшую длину min из двух списков и в цикле пробежался от 0 до min -1 , потому что от i min и до длины большей строки произведения скалярных векторов равно 0.

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

StarKRE · 2016-10-01T22:13:04Z

В 1-м уроке в задании с поездом, где нужно рассчитать время пути, бот вывел ошибку:
Expected: 0
Actual: 1440
Inputs:
hoursDepart -> 1
minutesDepart -> 1
hoursArrive -> 1
minutesArrive -> 1
НО Поезд мог проехать ровно сутки и прибыть в то же время только уже на другой станции.

ice-phoenix · 2016-10-01T22:51:13Z

Ваш последний коммит не компилируется. Убедительная просьба проверять коммит на работоспособность локально перед отправкой PR.

ice-phoenix

Ваш код все еще не компилируется

ice-phoenix · 2016-10-03T07:59:09Z

src/lesson1/task1/Simple.kt

 * Вывести значение того же угла в радианах (например, 0.63256).
 */
-fun angleInRadian(grad: Int, min: Int, sec: Int): Double = TODO()
+fun angleInRadian(grad: Int, min: Int, sec: Int): Double = (grad.toDouble() + min /60.0 + sec /3600.0) / (180.0 / Math.PI)


Форматирование кода --- старайтесь придерживаться единого стиля форматирования везде.

Можно просто grad вместо grad.toDouble()

ice-phoenix · 2016-10-03T08:02:38Z

src/lesson1/task1/Simple.kt

 */
-fun travelMinutes(hoursDepart: Int, minutesDepart: Int, hoursArrive: Int, minutesArrive: Int): Int = TODO()
+fun travelMinutes(hoursDepart: Int, minutesDepart: Int, hoursArrive: Int, minutesArrive: Int): Int {
+    var min: Int = (hoursArrive - hoursDepart) * 60 + (minutesArrive - minutesDepart)


Система типов Котлина выведет тип переменной автоматически

Данная переменная может быть объявлена как val

min я объявил var, потому что условие изменяет переменную: =)
if (min < 0){
min += 24 * 60
}

Не подскажите, почему возникает фатальная ошибка IDE internal error occured?

А можно поподробнее об ошибке? Скриншот или копию описания?

ice-phoenix · 2016-10-03T08:03:24Z

src/lesson1/task1/Simple.kt

 * Например, 100 рублей под 10% годовых превратятся в 133.1 рубля
 */
-fun accountInThreeYears(initial: Int, percent: Int): Double = TODO()
+fun accountInThreeYears(initial: Int, percent: Int): Double = ((1+percent.toDouble()/100)*(1+percent.toDouble()/100)*(1+percent.toDouble()/100) * initial.toDouble())


Повторно вычисляемые выражения лучше вынести в отдельную переменную

Вместо a.toDouble() / 42 обычно делают a / 42.0 при необходимости вычислений с плавающей запятой

ice-phoenix · 2016-10-03T08:05:05Z

src/lesson2/task1/IfElse.kt

-fun ageDescription(age: Int): String = TODO()
+fun ageDescription(age: Int): String {
+
+    if ((age % 10 == 1) && (age / 10 != 1) && (age / 10 != 11)) {


Данное выражение лучше оформить как when

ice-phoenix · 2016-10-03T08:05:20Z

src/lesson2/task1/IfElse.kt

+                   t2: Double,v2: Double,
+                   t3: Double,v3: Double): Double {
+
+    var ss1: Double = v1 * t1


Система типов Котлина выведет тип переменной автоматически

Данная переменная может быть объявлена как val

ice-phoenix · 2016-10-03T08:12:23Z

src/lesson3/task1/Loop.kt

+    } else {
+        min = n
+    }
+    for (i in 1..min) {


Зачем при поиске наибольшего общего делителя перебирать делители от минимального к максимальному?

ice-phoenix · 2016-10-03T08:12:49Z

src/lesson3/task1/Loop.kt

+    }
+    for (i in 1..min) {
+        if (n % i == 0 && m % i == 0) {
+            k = i


Почитайте про алгоритмы поиска НОД, которые работают быстрее полного перебора

ice-phoenix · 2016-10-03T08:14:26Z

src/lesson3/task1/Loop.kt

-fun squareBetweenExists(m: Int, n: Int): Boolean = TODO()
+fun sqr(g: Double) = g * g
+
+fun squareBetweenExists(m: Int, n: Int): Boolean = m <= sqr(sqrt(n.toDouble()).toInt().toDouble()) && sqr(sqrt(n.toDouble()).toInt().toDouble()) <= n


Повторно используемые выражения лучше вынести в отдельные переменные

Для округления лучше использовать специальные функции из пакета Math: round / floor / ceil

ice-phoenix · 2016-10-03T08:15:53Z

src/lesson3/task1/Loop.kt

+        number = number + count(numberi * numberi)
+    }
+    result = numberi * numberi
+    (n..number - 1).forEach { numberi -> result = result / 10 }


Изменять внешнюю переменную в лямбде --- очень плохой стиль. В данном случае лучше воспользоваться обычным циклом for.

ice-phoenix · 2016-10-03T08:17:04Z

src/lesson3/task1/Loop.kt

+        number = number + count(fib(numberi))
+    }
+    result = fib(numberi)
+    (n..number - 1).forEach { i -> result = result / 10 }


Изменять внешнюю переменную в лямбде --- очень плохой стиль. В данном случае лучше воспользоваться обычным циклом for.

StarKRE · 2016-12-16T22:20:50Z

Подскажите, пожалуйста, работал на другом компьютере, склонировал репозиторий с GitHub, при push ошибка: Failed with error: fatal: unable to access 'https://github.com/StarKRE/KotlinAsFirst2016.git/': The requested URL returned error: 403

mglukhikh · 2016-12-17T21:18:06Z

Возможно, логин / пароль неверные или что-то не так в этот момент было с Интернет

ice-phoenix · 2016-12-18T20:04:13Z

src/lesson1/task1/Simple.kt

 *Необходимо вывести число, полученное из заданного перестановкой цифр в обратном порядке (например, 874).
 */
-fun numberRevert(number: Int): Int = TODO()
+fun numberRevert(number: Int): Int = ((number % 10) * 100) + ((number / 100)) + (number - ((number / 100) * 100) - (number % 10))


Длинные строки не есть очень хорошо

ice-phoenix · 2016-12-18T20:06:58Z

src/lesson2/task1/IfElse.kt

+}
+
+fun minOfThree(a: Double, b: Double, c: Double): Double{
+    var min= 0.0


Форматирование, ну е-мое!

ice-phoenix · 2016-12-18T20:07:20Z

src/lesson2/task1/IfElse.kt

+            d >= b && a >= c -> b - a
+            d < b && a > d -> -1
+            d < b && d >= a && c <= a -> d - a
+            else -> d - c


Все еще актуально

ice-phoenix · 2016-12-18T20:08:50Z

src/lesson3/task1/Loop.kt

+fun sin(x: Double, eps: Double): Double {
+    var k = 0
+    var sin = x % (2 * Math.PI)
+    var number = x % (2 * Math.PI)


Может быть, можно не считать два раза одно и то же?

ice-phoenix · 2016-12-18T20:10:19Z

src/lesson4/task1/List.kt

+         * Центрировать заданный список list, уменьшив каждый элемент на среднее арифметическое всех элементов.
+         * Если список пуст, не делать ничего. Вернуть изменённый список.
+         */
+        fun center(list: MutableList<Double>): MutableList<Double> {


Неужели так сложно исправить форматирование было, е-мое???

ice-phoenix · 2016-12-18T20:33:18Z

src/lesson7/task2/Matrices.kt

+        downWall--
+        if (downWall - topWall <= 0) return matrix
+        for (k in downWall - 1 downTo topWall){
+            j = k


А почему бы просто не поиспользовать переменную k здесь и выше???

не понял...

ice-phoenix · 2016-12-18T20:33:48Z

src/lesson7/task2/Matrices.kt

+    while (count <=  width * height) {
+        if (rightWall - leftWall <= 0) return matrix
+        for (k in leftWall..rightWall - 1) {
+            i = k


См. выше...

ice-phoenix · 2016-12-18T20:34:19Z

src/lesson7/task2/Matrices.kt

 * 3 1 2
 */
-fun isLatinSquare(matrix: Matrix<Int>): Boolean = TODO()
+fun isLatinSquare(matrix: Matrix<Int>): Boolean {


Решили не думать внимательно. Тоже вариант!

ice-phoenix · 2016-12-18T20:34:38Z

src/lesson7/task2/Matrices.kt

-fun sumNeighbours(matrix: Matrix<Int>): Matrix<Int> = TODO()
+fun sumNeighbours(matrix: Matrix<Int>): Matrix<Int> {
+    val extendedMatrix = createMatrix<Int>(matrix.height + 2, matrix.width + 2, 0)
+    val NeighboursMatrix = matrix // создание расширенной и преобразованной матриц


camelCase

ice-phoenix · 2016-12-18T20:35:25Z

src/lesson7/task2/Matrices.kt

 */
-fun sumNeighbours(matrix: Matrix<Int>): Matrix<Int> = TODO()
+fun sumNeighbours(matrix: Matrix<Int>): Matrix<Int> {
+    val extendedMatrix = createMatrix<Int>(matrix.height + 2, matrix.width + 2, 0)


А вот это неплохой трюк

ice-phoenix · 2016-12-18T20:38:13Z

recheck all

Conversation

StarKRE commented Sep 30, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ice-phoenix left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

StarKRE commented Oct 1, 2016

Uh oh!

ice-phoenix commented Oct 1, 2016

Uh oh!

ice-phoenix left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ice-phoenix Oct 3, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ice-phoenix Oct 3, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

StarKRE commented Dec 16, 2016

Uh oh!

mglukhikh commented Dec 17, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

StarKRE commented Sep 30, 2016 •

edited

Loading

ice-phoenix Oct 3, 2016 •

edited

Loading

ice-phoenix Oct 3, 2016 •

edited

Loading