Hello world and bubbleSort #14

akhmetov-dev · 2020-09-12T15:27:46Z

No description provided.

xthebat · 2020-09-16T10:17:06Z

ahmetoff/02/src/main/kotlin/ru/bmstu/iu4/Tests.kt

+        val data : Array <Int> = arrayOf(1,5,3,6,7,3,2,9)
+        println(Arrays.toString(data))
+        Starter.bubbleSort(data)
+        println(Arrays.toString(data))


В тесте нет assert - это по факту и не тест

xthebat · 2020-09-16T10:17:41Z

ahmetoff/02/src/main/kotlin/ru/bmstu/iu4/Starter.kt

+            }
+        }
+    }
+    }


Выравнивайте форматирование кода (есть автоформат в IDEA CTRL+ALT+L)

xthebat · 2020-09-16T10:17:58Z

ahmetoff/02/src/main/kotlin/ru/bmstu/iu4/Starter.kt

+        bubbleSort(data)
+        println(Arrays.toString(data))
+    }
+    fun bubbleSort(toSort: Array<Int>) {


Функция должна быть в отдельном файле

xthebat · 2020-09-16T10:18:25Z

ahmetoff/02/build.gradle

@@ -0,0 +1,16 @@
+plugins {
+    id 'org.jetbrains.kotlin.jvm' version '1.3.72'


Версии в dependencies и плагина должны совпадать

xthebat · 2020-09-18T06:40:33Z

ahmetoff/02/src/main/kotlin/ru/bmstu/iu4/Tests.kt

-    }
-
+inter    @Test
+fun bubbleSortTest() {


Вы хотя бы тесты запускали?

Случайно полетело, исправил

xthebat · 2020-09-18T06:41:09Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Starter.kt

@@ -0,0 +1,10 @@
+package ru.bmstu.iu4
+
+typealias Matrix = MutableList<MutableList<Int>>


typealias должен быть в файле с функциями

Исправил

xthebat · 2020-09-18T06:42:32Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

@@ -0,0 +1,111 @@
+package ru.bmstu.iu4
+
+fun Matrix.init() {


функция должна называться не init, а random, причем почему-то ограниченный до 10

Исправил

xthebat · 2020-09-18T06:42:44Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    if (this[rows - 1][cols - 1] != null) {
+        for (i in 0 until rows) {
+            for (j in 0 until cols) {
+                print(this[i][j])


для этого есть println

Исправил

xthebat · 2020-09-18T06:43:01Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+fun Matrix.print() {
+    val rows = this.size
+    val cols = this[0].size
+    if (this[rows - 1][cols - 1] != null) {


зачем эта проверка?

я задумывал ее как проверку на то, что передаваемая матрциа не пустая

Зачем ее печатать если она пустая

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

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

xthebat · 2020-09-18T06:43:21Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    val cols2 = matrix2[0].size
+
+    if (rows1 == rows2 && cols1 == cols2) {
+        if (matrix1[rows1 - 1][cols1 - 1] != null) {


аналогично, зачем эта проверка?

ну сложить поэлементно матрицы можно, только если количество строк и столбцов у них равно

xthebat · 2020-09-18T06:44:07Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    val rows2 = matrix2.size
+    val cols2 = matrix2[0].size
+
+    if (rows1 == rows2 && cols1 == cols2) {


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

Исправлено

xthebat · 2020-09-18T06:44:19Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    }
+}
+
+fun Matrix.subtr(matrix1: Matrix, matrix2: Matrix) {


xthebat · 2020-09-18T06:44:27Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    }
+}
+
+fun Matrix.dotMult(matrix1: Matrix, matrix2: Matrix) {


xthebat · 2020-09-18T06:46:02Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Tests.kt

+        println()
+
+        val matrix3: Matrix = MutableList(rows1, { MutableList(cols1, { 0 }) })
+        matrix3.subtr(matrix1, matrix2)


все функции должны быть в infix нотации
matrix3 = matrix1 subtr matrix2

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

xthebat · 2020-09-18T06:46:42Z

Опять в корень закомитили .gitignore

xthebat

Опять бинарные файлы закоммитили :(

xthebat · 2020-09-25T15:34:31Z

ahmetoff/04/src/main/kotlin/Functions.kt

+
+import kotlin.test.Test
+
+fun countCache(json: Persons): Double {


Должно быть сделано с использованием функций работы с коллекциями filter, map и т.п.

xthebat

Зачем вы удалил чужие файлы? О_о

xthebat · 2020-10-04T15:23:39Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+fun Matrix.print() {
+    val rows = this.size
+    val cols = this[0].size
+    if (this[rows - 1][cols - 1] != null) {


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

xthebat · 2020-10-04T15:24:28Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+        }
+    }
+    else {
+        throw IllegalArgumentException ("Неправильно заданы размеры матриц.")


Сообщения в исключения лучше писать на английском

xthebat · 2020-10-04T15:24:53Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    val rows2 = matrix2.size
+    val cols2 = matrix2[0].size
+
+    if (cols1 == rows2) {


Можно обойстись без этой дикой вложенности, инвертировав условие if

xthebat · 2020-10-04T15:25:03Z

ahmetoff/03/src/main/kotlin/ru/bmstu/iu4/Functions.kt

+    val tmp: Matrix = MutableList(cols1, { MutableList(rows1, { 0 }) })
+
+    for (i in 0..tmp.size - 1) {
+        for (j in 0..tmp[0].size - 1) {


можно использовать until

Добавлены тесты

Восстановил папки соучащихся

Удален дублирущий гитигнор файл

Разрешение конфликтов

1

Разрешение конфликтов

akhmetov-dev added 6 commits September 11, 2020 22:31

чистка

e3589d1

hello world project

601ab65

sort bubble

762b586

Сортировка пузырьком

f52c959

check gitignore

965252a

check gitigniore

6fc047e

xthebat reviewed Sep 16, 2020

View reviewed changes

akhmetov-dev added 2 commits September 17, 2020 16:03

I have corrected my issues

df82142

Task 03 was completed

2041c14

xthebat requested changes Sep 18, 2020

View reviewed changes

akhmetov-dev added 2 commits September 25, 2020 14:50

Поправки ко 2 и 3 заданию, добавление 4 задания

71a43e0

Create .gitignore

418a9e5

xthebat requested changes Sep 25, 2020

View reviewed changes

xthebat reviewed Sep 25, 2020

View reviewed changes

akhmetov-dev added 4 commits September 26, 2020 21:37

удалил бин файлы

dc289b3

Update Tests.kt

be9fc61

Delete .gitignore

dbe75ce

выравнивание по центру, удаление двойных границ

23dd378

xthebat requested changes Oct 4, 2020

View reviewed changes

akhmetov-dev added 7 commits October 4, 2020 21:07

5 задание

f269263

Задание 5

0fd031e

Задание 5

9d8c6c6

Добавлены тесты

folders of other students was restored

1a01839

Восстановил папки соучащихся

.gitgnore file was removed

5273578

Удален дублирущий гитигнор файл

resolving conflicts

afad393

Разрешение конфликтов

1

a689c3b

1

akhmetov-dev closed this Nov 11, 2020

akhmetov-dev reopened this Nov 11, 2020

akhmetov-dev closed this Nov 11, 2020

akhmetov-dev added 2 commits November 11, 2020 16:09

Merge remote-tracking branch 'upstream/master'

f399adc

Разрешение конфликтов

fe6c0b3

Разрешение конфликтов

akhmetov-dev reopened this Nov 11, 2020

Copy sem 10

6a7f595

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Hello world and bubbleSort #14

Hello world and bubbleSort #14

akhmetov-dev commented Sep 12, 2020

xthebat Sep 16, 2020

xthebat Sep 16, 2020

xthebat Sep 16, 2020

xthebat Sep 16, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

akhmetov-dev Sep 25, 2020

xthebat Oct 4, 2020

akhmetov-dev Oct 30, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat Sep 18, 2020

xthebat Sep 18, 2020

xthebat Sep 18, 2020

akhmetov-dev Sep 25, 2020

xthebat commented Sep 18, 2020

xthebat left a comment

xthebat Sep 25, 2020

xthebat left a comment

xthebat Oct 4, 2020

xthebat Oct 4, 2020

xthebat Oct 4, 2020

xthebat Oct 4, 2020

		@@ -0,0 +1,16 @@
		plugins {
		id 'org.jetbrains.kotlin.jvm' version '1.3.72'

		@@ -0,0 +1,10 @@
		package ru.bmstu.iu4

		typealias Matrix = MutableList<MutableList<Int>>


		import kotlin.test.Test

		fun countCache(json: Persons): Double {

Hello world and bubbleSort #14

Are you sure you want to change the base?

Hello world and bubbleSort #14

Conversation

akhmetov-dev commented Sep 12, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xthebat commented Sep 18, 2020

xthebat left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xthebat left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment