Need help with 7'th last. Lesson 8 by tiger31 · Pull Request #40 · Kotlin-Polytech/KotlinAsFirst2016

tiger31 · 2016-09-10T19:44:14Z

Check out 1st, 2nd, 3rd lessons, 4th in progress

ice-phoenix · 2016-09-14T14:37:01Z

src/lesson1/task1/Simple.kt

 */
-fun angleInRadian(grad: Int, min: Int, sec: Int): Double = TODO()
+fun angleInRadian(grad: Int, min: Int, sec: Int): Double {
+    var value: Double = grad + min.toDouble()/60 + sec.toDouble()/3600;


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

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

tiger31 · 2016-11-26T19:28:57Z

Как я понял последня задача седьмого урока решается графом + A*? Или есть другие пути её решения?

ice-phoenix · 2016-11-28T09:45:26Z

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

tiger31 · 2016-11-28T09:58:23Z

Исправлениями я займусь, а эвристикой будет что-то вроде f(x)=(количество перестановок до текущего состояния + количество ячеек не на своём месте)

tiger31 · 2016-12-01T09:43:14Z

recheck all

tiger31 · 2016-12-01T15:05:11Z

Изменил эвристику и пуллинг новых вершин, но алгоритм все еще решает только простые задачи с небольшим количество ходов. Что не так в алгоритме?

tiger31 · 2016-12-01T15:06:40Z

Я знаю, что он и не вернет список ходов, но он даже не может найти конечную вершину(решение)

mglukhikh

Я взял на себя смелость поревьюить fifteenGameSolution (и только её), поскольку вы первый из студентов, кто пытается решить эту задачу. Ответ на ваш вопрос в ревью содержится. Дерзайте. Эта задача, по-видимому, самая сложная во всём проекте, так что не огорчайтесь, если не получится решить её быстро. Если у моих коллег есть дополнительные замечания к решению, они напишут их отдельно.

NB: не считайте, пожалуйста, что вначале надо заставить всё работать, а потом решение причёсывать и приводить в порядок стиль. В нормальном случае эти процессы идут параллельно, поскольку заставить работать аккуратно написанное решение проще. Многократно проверено.

mglukhikh · 2016-12-01T17:07:56Z

src/lesson7/task2/Matrices.kt

+        listOf(9, 10, 11, 12),
+        listOf(13, 15, 14, 0)))
+
+fun h(matrix: Matrix<Int>): Int {


Старайтесь не называть так функции. Насколько я понимаю, эта функция возвращает некоторую оценку, насколько далеко данная позиция от исходной. В соответствии с этим придумайте название. Простой идеей может быть, например, evaluation (оценка)

mglukhikh · 2016-12-01T17:08:26Z

src/lesson7/task2/Matrices.kt

+
+fun h(matrix: Matrix<Int>): Int {
+    var n = 0
+    for (i in 0..3)


В таких конструкциях лучше ставить фигурные скобки везде, иначе их тяжелее читать. Кроме этого, есть смысл назвать в данном случае переменные цикла row и column -- так понятнее.

mglukhikh · 2016-12-01T17:09:33Z

src/lesson7/task2/Matrices.kt

+    var n = 0
+    for (i in 0..3)
+        for (j in 0..3)
+            if (matrix[i,j] != 0)


Вот здесь я не уверен в разумности этой проверки. Интуиция подсказывает, что расстояние пустой ячейки от её оригинальной позиции тоже имеет некоторое значение. Но здесь, конечно, нужна проверка на практике.

mglukhikh · 2016-12-01T17:12:16Z

src/lesson7/task2/Matrices.kt

+        for (j in 0..3)
+            if (matrix[i,j] != 0)
+                if (matrix[i,j] != i * 4 + j + 1) {
+                    n += abs(matrix[i, j] / 4 - i) + abs(matrix[i, j] % 4 - j - 1)


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

mglukhikh · 2016-12-01T17:15:08Z

src/lesson7/task2/Matrices.kt

+            for (j in 0..i - 1)
+                if (list[j] > list[i])
+                    N++
+        } else N += 1 + i / 4


Вы уверены, что этот кусок формулы, касающийся пустой клетки, справедлив? (спрашиваю потому, что первую часть этого мини-алгоритма в математических книгах я видел, а вторую -- нет)

Кроме этого, это определение чётности позиции стоило бы вынести в отдельную функцию

mglukhikh · 2016-12-01T17:16:17Z

src/lesson7/task2/Matrices.kt

+    val solution = if (N % 2 == 0) solution1 else solution2
+    println("Solution: \r\n $solution")
+
+    // Попытка A* + поиск в шиину


Всё начиная отсюда стоило бы вытащить в отдельную функцию. Кроме этого, A* пока не виден

mglukhikh · 2016-12-01T17:18:24Z

src/lesson7/task2/Matrices.kt

+    println("Solution: \r\n $solution")
+
+    // Попытка A* + поиск в шиину
+    data class State(val m: Matrix<Int>, val priority: Int) : Comparable<State> {


m --> matrix. Такие короткие имена почти не используются для свойств

mglukhikh · 2016-12-01T17:30:11Z

src/lesson7/task2/Matrices.kt

+    queue.add(State(matrix, h(matrix)))
+
+    val visited = mutableMapOf(matrix to 0)
+    while (queue.isNotEmpty()) {


Я боюсь, что с подобной идеей алгоритма исправление ошибок (например, не учитывается приоритет, не запоминаются ходы, может быть, я ещё чего-либо не вижу) не поможет. Дело в том, что у игры в 15 всего имеется 16! позиций, чёт / нечёт уменьшает это число вдвое. Такой вот полный поиск в ширину будет вынужден обойти весьма существенную часть от этого количества, а оно измеряется триллионами. То есть шансов получить решение за обозримое время таким способом вроде как не имеется (IMHO).

Что можно рекомендовать:

в данной задаче DFS должен быть лучше BFS, хотя бы за счёт того, что можно избежать постоянного копирования матриц

независимо от того, используете вы DFS или BFS, поиск необходимо ограничивать, то есть не искать дальше, чем на какое-то ограниченное количество ходов вперёд; вы при этом не достигнете завершающей позиции, но сможете найти позицию с лучшей оценкой и ход, ведущий в неё; найденный ход следует запомнить в списке, сделать и повторить всю процедуру поиска уже с новой позиции для поиска лучшего второго хода; процедура повторяется до тех пор, пока сделанный ход не приведёт вас в solution.

ice-phoenix · 2016-12-05T11:44:42Z

Очень много неисправленных ранее замечаний. Давайте вы сперва исправите их все (да, скучно, да, нудно, да, не приближает вас к получению зачета напрямую), а потом уже будем смотреть, что вы наделали дальше.

… not the 65-steps example given

kotlin-polytech-bot · 2016-12-05T18:23:35Z

author

tiger31 [tiger31839@gmail.com]

owner

tiger31 []

ice-phoenix

Чего-то посмотрел и пописал в задачке про 15

ice-phoenix · 2016-12-07T19:06:37Z

src/lesson7/task2/Matrices.kt

-                    n += abs(matrix[i, j] / 4 - i) + abs(matrix[i, j] % 4 - j - 1)
+class State(val matrix: Matrix<Int>, val moved: Int?)  {
+
+    object companion {


Вообще-то, companion object задается несколько не так

ice-phoenix · 2016-12-07T19:10:49Z

src/lesson7/task2/Matrices.kt


+//Extensions of Cell data class
+// По-нормальноу это должен быть статический параметр класса, но са класс Cell я менять не могу
+val NOT_EXISTS = Cell(-1, -1)


Вы не поверите, но можно сделать extension value у класса =)

ice-phoenix · 2016-12-07T19:11:26Z

src/lesson7/task2/Matrices.kt

+val NOT_EXISTS = Cell(-1, -1)
+
+fun Cell.near(other: Cell): Boolean {
+    if (this == other) throw IllegalArgumentException("Cells are the same")


Можно в виде one-liner

ice-phoenix · 2016-12-07T19:13:04Z

src/lesson7/task2/Matrices.kt

+
+fun <E> Matrix<E>.subMatrix(height: Int, width: Int, heightShift: Int, widthShift: Int): Matrix<E> {
+    if (this.height < height + heightShift || this.width < width + widthShift)
+        throw IllegalArgumentException("Sub Matrix is out of bounds of original")


Раз уж мы все проверяем, то я бы проверил границы и в другую сторону

ice-phoenix · 2016-12-07T19:14:27Z

src/lesson7/task2/Matrices.kt

+fun <E> Matrix<E>.contains(cell: Cell): Boolean = this.contains(cell.row, cell.column)
+
+fun <E> Matrix<E>.subMatrix(height: Int, width: Int, heightShift: Int, widthShift: Int): Matrix<E> {
+    if (this.height < height + heightShift || this.width < width + widthShift)


Тут у вас клэш по именам, лучше его избегать --- я бы переименовал аргументы функции как-нибудь по другому

ice-phoenix · 2016-12-07T19:51:41Z

src/lesson7/task2/Matrices.kt

+                }
+            }
+        }
+        state.matrix.swap(zero, move)


Кажется, что во всем этом деле (см. выше) вы слишком много раз копируете матрицу там, где можно было бы обойтись и без этого

ice-phoenix · 2016-12-07T19:55:00Z

src/lesson7/task2/Matrices.kt

+    var solved = false
+
+    if (state.matrix == State.companion.SOLUTION)
+        return listOf()


Вы уж разберитесь в коде, либо через ==, либо через solved

ice-phoenix · 2016-12-07T19:56:46Z

src/lesson7/task2/Matrices.kt

+            println("$count searches are complete, pathes = $finished, current depth = $limit, visited = ${visited.size}")
+        limit += 2
+        visited.clear()
+        moves.clear()


Зачем перезапускать поиск с нуля?

ice-phoenix · 2016-12-07T19:58:11Z

src/lesson7/task2/Matrices.kt

+
+fun fifteenGameSolution(matrix: Matrix<Int>): List<Int> {
+
+    State.companion.START = matrix


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

ice-phoenix · 2016-12-07T19:59:17Z

src/lesson7/task2/Matrices.kt

+        } else {
+
+            if (!visited.contains(state.matrix.copyN())) {
+                if (state.estimate() + path + 1 > limit) {


Зачем вы вообще ограничиваете глубину поиска, чтобы потом (если не получилось) перезапускать все с нуля и снова проходить все те же самые позиции?

ice-phoenix · 2016-12-18T22:28:27Z

recheck all

ice-phoenix · 2016-12-18T22:39:52Z

Несмотря на большое количество замечаний и не очень чистый код, по совокупной статистике решения задач и результатам тестирования у вас "Отлично"! Так держать!

ice-phoenix · 2016-12-18T22:40:19Z

PS: Но тесты все же лучше, чтобы завершались =)

kotlin-polytech-bot · 2016-12-18T22:42:16Z

author

tiger31 [tiger31839@gmail.com]

owner

tiger31 []

tiger31 · 2016-12-18T23:15:05Z

Я сейчас несколько исправлений еще добавляю. Очень хотелось бы увидеть правильное решение последней задачи в седьмом, потому что пока это единственная задача, которая меня "сломала" :)

…n-bot

kotlin-polytech-bot · 2016-12-18T23:42:20Z

author

tiger31 [tiger31839@gmail.com]

lesson7.task2

Author: tiger31 [tiger31839@gmail.com]

Owner: tiger31 []

Total: 14 / 15

Example: 2 / 2
Easy: 1 / 1
Normal: 5 / 5
Hard: 6 / 6
Impossible: 0 / 1

Succeeded:

[Example] lesson7.task2/plus
[Example] lesson7.task2/transpose
[Hard] lesson7.task2/generateSpiral
[Hard] lesson7.task2/generateRectangles
[Hard] lesson7.task2/generateSnake
[Hard] lesson7.task2/isLatinSquare
[Normal] lesson7.task2/sumNeighbours
[Normal] lesson7.task2/findHoles
[Normal] lesson7.task2/sumSubMatrix
[Hard] lesson7.task2/canOpenLock
[Easy] lesson7.task2/unaryMinus
[Normal] lesson7.task2/times
[Hard] lesson7.task2/fifteenGameMoves
[Normal] lesson7.task2/rotate

Seed: 4571634249296578062

lesson8.task1

Author: tiger31 [tiger31839@gmail.com]

Owner: tiger31 []

Total: 4 / 13

Example: 1 / 1
Normal: 3 / 7
Hard: 0 / 3
Impossible: 0 / 2

Succeeded:

[Example] lesson8.task1/alignFile
[Normal] lesson8.task1/countSubstrings
[Normal] lesson8.task1/sibilants
[Normal] lesson8.task1/top20Words

Failed:

[Normal] lesson8.task1/centerFile

Expected:

АЙАВ -- АЭВЖБВХЧБЗБАЙ / бАбОбвваб -- бббччбяб бббччбяб  вабтбажбббю / ВБВАЫУВЛВБАЬУВЮБ: ебвавшдббаоу баятбббвюввшатюарозб; бэфбб / ювХввьбатвЬавбйСбЙрЦ: АЭВЖБВХЧБЗБАЙ --- ВААЭВШББЕАРББЪАВЗВЧГ -- ййщдёв ВБВАЫУВЛВБАЬУВЮБ - ЙЙЩДЁВ - ацпавбжъааутб - ебвавшдббаоу; ЙБАШББЛНЙА / ЩИбЕёбвбаХабб - лгвббвжббебмыарбааб  ебвАвШдббаОу --- двавввачвямбрвббацаб: ввввжбйббвд --- РБББВЕВВБВЧБВТДВЗ - БАЯТБББВЮВВШАТЮАРОЗБ / ДВАВВВАЧВЯМБРВББАЦАБ; быбб  ЩИбЕёбвбаХабб  вЬббаЕбвИабцбИвЕвм / ббХбб - ЙбАШбблНЙа ДвавввачвЯмбРвббаЦаб, аббе, вЬббаЕбвИабцбИвЕвм / ШвббТабщёва --

Actual:

АЙАВ -- АЭВЖБВХЧБЗБАЙ / бАбОбвваб -- бббччбяб бббччбяб  вабтбажбббю / ВБВАЫУВЛВБАЬУВЮБ: ебвавшдббаоу баятбббвюввшатюарозб; бэфбб / ювХввьбатвЬавбйСбЙрЦ: АЭВЖБВХЧБЗБАЙ --- ВААЭВШББЕАРББЪАВЗВЧГ -- ййщдёв ВБВАЫУВЛВБАЬУВЮБ - ЙЙЩДЁВ - ацпавбжъааутб - ебвавшдббаоу; ЙБАШББЛНЙА / ЩИбЕёбвбаХабб - лгвббвжббебмыарбааб  ебвАвШдббаОу --- двавввачвямбрвббацаб: ввввжбйббвд --- РБББВЕВВБВЧБВТДВЗ - БАЯТБББВЮВВШАТЮАРОЗБ / ДВАВВВАЧВЯМБРВББАЦАБ; быбб  ЩИбЕёбвбаХабб  вЬббаЕбвИабцбИвЕвм / ббХбб - ЙбАШбблНЙа ДвавввачвЯмбРвббаЦаб, аббе, вЬббаЕбвИабцбИвЕвм / ШвббТабщёва --

Inputs:

text ->

АЙАВ -- АЭВЖБВХЧБЗБАЙ / бАбОбвваб -- бббччбяб бббччбяб  вабтбажбббю / ВБВАЫУВЛВБАЬУВЮБ: ебвавшдббаоу баятбббвюввшатюарозб; бэфбб / ювХввьбатвЬавбйСбЙрЦ: АЭВЖБВХЧБЗБАЙ --- ВААЭВШББЕАРББЪАВЗВЧГ -- ййщдёв ВБВАЫУВЛВБАЬУВЮБ - ЙЙЩДЁВ - ацпавбжъааутб - ебвавшдббаоу; ЙБАШББЛНЙА / ЩИбЕёбвбаХабб - лгвббвжббебмыарбааб  ебвАвШдббаОу --- двавввачвямбрвббацаб: ввввжбйббвд --- РБББВЕВВБВЧБВТДВЗ - БАЯТБББВЮВВШАТЮАРОЗБ / ДВАВВВАЧВЯМБРВББАЦАБ; быбб  ЩИбЕёбвбаХабб  вЬббаЕбвИабцбИвЕвм / ббХбб - ЙбАШбблНЙа ДвавввачвЯмбРвббаЦаб, аббе, вЬббаЕбвИабцбИвЕвм / ШвббТабщёва --

Exception: null

[Hard] lesson8.task1/alignFileByWidth

Expected:

ЬбжбблфявЬ ЬбжбблфявЬ -

Actual:

ЬбжбблфявЬ ЬбжбблфявЬ -

Inputs:

text ->

ЬбжбблфявЬ ЬбжбблфявЬ -

Exception: null

Seed: 4571634249296578062

owner

tiger31 []

total

Author: tiger31 [tiger31839@gmail.com]

Owner: tiger31 []

Total: 18 / 28

Example: 3 / 3
Easy: 1 / 1
Normal: 8 / 12
Hard: 6 / 9
Impossible: 0 / 3

lessons 1,2,3 solved, lesson 4 in progress

86c93e1

gagarski added the grader failure label Sep 11, 2016

lessons 1-4 are solved, lesson 5 in progress

39c75cb

mglukhikh added the to be investigated label Sep 12, 2016

gagarski removed the grader failure label Sep 12, 2016

lesson5 except last

1cfa6a4

ice-phoenix self-assigned this Sep 14, 2016

ice-phoenix added tests failed and removed to be investigated labels Sep 14, 2016

ice-phoenix reviewed Sep 14, 2016
View reviewed changes

tiger31 changed the title ~~lesson 6 is done~~ lesson 7 almost Nov 26, 2016

fifteenGameMoves solved

20f2a8d

ice-phoenix added wontfix and removed stale labels Nov 28, 2016

tiger31 added 2 commits December 1, 2016 12:35

Can't solve last of 7'th lesson, so need help. Lesson 8 tasks

ed9fee9

Names conflict

f2e0443

Still can't sole last of 7'th. 8 lesson

66d2fcc

tiger31 changed the title ~~lesson 7 almost~~ Need help with 7'th last. Lesson 8 Dec 1, 2016

mglukhikh suggested changes Dec 1, 2016

View reviewed changes

IDA* 7'th last. Still need help, 'cause it's solving 15-20 steps, but…

f685768

… not the 65-steps example given

ice-phoenix suggested changes Dec 7, 2016

View reviewed changes

ice-phoenix added excellent and removed waiting for fix wontfix labels Dec 18, 2016

some fixes lesson 7,8. Last task of 7'th has been commented for kotli…

6703df7

…n-bot

mglukhikh closed this Jul 26, 2017


		fun fifteenGameSolution(matrix: Matrix<Int>): List<Int> {

		State.companion.START = matrix

Conversation

tiger31 commented Sep 10, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tiger31 commented Nov 26, 2016

Uh oh!

ice-phoenix commented Nov 28, 2016

Uh oh!

tiger31 commented Nov 28, 2016

Uh oh!

tiger31 commented Dec 1, 2016

Uh oh!

tiger31 commented Dec 1, 2016

Uh oh!

tiger31 commented Dec 1, 2016

Uh oh!

mglukhikh 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!

ice-phoenix commented Dec 5, 2016

Uh oh!

kotlin-polytech-bot commented Dec 5, 2016

author

owner

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!

ice-phoenix commented Dec 18, 2016

Uh oh!

ice-phoenix commented Dec 18, 2016

Uh oh!

ice-phoenix commented Dec 18, 2016

Uh oh!

kotlin-polytech-bot commented Dec 18, 2016

author

owner

Uh oh!

tiger31 commented Dec 18, 2016

Uh oh!

kotlin-polytech-bot commented Dec 18, 2016