Lesson 4 (+1cm fix) #181

furoraru · 2016-10-22T18:40:39Z

No description provided.

(+1cm fix)

ice-phoenix

Все довольно неплохо, исправляйте замечания и можно работать дальше

ice-phoenix · 2016-10-24T12:09:38Z

src/lesson1/task1/Simple.kt

 */
-fun lengthInMeters(sagenes: Int, arshins: Int, vershoks: Int): Double = TODO()
+fun lengthInMeters(sagenes: Int, arshins: Int, vershoks: Int) : Double =
+        ((vershoks * 4.445 + arshins * 16* 4.445 + sagenes * 48* 4.445) / 100)


Лишние скобки вокруг законченного выражения

ice-phoenix · 2016-10-24T12:10:14Z

src/lesson1/task1/Simple.kt

 */
-fun angleInRadian(grad: Int, min: Int, sec: Int): Double = TODO()
+fun angleInRadian(grad: Int, min: Int, sec: Int): Double =
+        (( grad +(( min + sec / 60.0) /60.0)) * Math.PI /180)


Почему здесь открывающие скобки внезапно испугались и отделились от остальных частей пробелами?

ice-phoenix · 2016-10-24T12:11:15Z

src/lesson1/task1/Simple.kt

 * Определить третью цифру справа в этом числе (в данном случае 8).
 */
-fun thirdDigit(number: Int): Int = TODO()
+fun thirdDigit(number: Int) = (number/100)%10


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

ice-phoenix · 2016-10-24T12:12:15Z

src/lesson1/task1/Simple.kt

-fun accountInThreeYears(initial: Int, percent: Int): Double = TODO()
+fun accountInThreeYears(initial: Int, percent: Int): Double {
+    val k = initial + initial*0.01*percent
+    return (k + k*0.01*percent) + (k + k*0.01*percent)*0.01*percent


Это выражение можно упростить еще дальше

ice-phoenix · 2016-10-24T12:12:51Z

src/lesson2/task1/IfElse.kt

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


Этот if может быть частью следующего за ним when

ice-phoenix · 2016-10-24T12:26:33Z

src/lesson4/task1/List.kt

 */
-fun center(list: MutableList<Double>): MutableList<Double> = TODO()
+fun center(list: MutableList<Double>): MutableList<Double> {
+    if (list.isEmpty()) return list


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

ice-phoenix · 2016-10-24T12:26:52Z

src/lesson4/task1/List.kt

-fun times(a: List<Double>, b: List<Double>): Double = TODO()
+fun times(a: List<Double>, b: List<Double>): Double {
+    if (a.isEmpty() || b.isEmpty()) return 0.0
+    var C = 0.0


Называть переменные с большой буквы в Котлине не принято

ice-phoenix · 2016-10-24T12:27:10Z

src/lesson4/task1/List.kt

 */
-fun polynom(p: List<Double>, x: Double): Double = TODO()
+fun polynom(p: List<Double>, x: Double): Double {
+    if (p.isEmpty()) return 0.0


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

ice-phoenix · 2016-10-24T12:27:42Z

src/lesson4/task1/List.kt

 */
-fun accumulate(list: MutableList<Double>): MutableList<Double> = TODO()
+fun accumulate(list: MutableList<Double>): MutableList<Double> {
+    var Z = 0.0


Называть переменные с большой буквы в Котлине не принято

Переменным лучше давать осмысленные имена, соответствующие их назначению в коде

ice-phoenix · 2016-10-24T12:28:33Z

src/lesson4/task1/List.kt

+    val result = mutableListOf<Int>()
+    var n0 = n
+    while (n0 > 1) {
+        result.add(minDivisor(n0))


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

Извините, но я плохо понимаю, как это можно исправить, ведь переменная, которая вызывается в тяжелой функции, меняется в цикле ;c

А что вам мешает определить эту переменную прямо в цикле?

furoraru · 2016-10-27T21:52:36Z

Я не могу загрузить изменения из-за ошибки, что делать?

Push failed
Failed with error: Error: Could not find or load main class org.jetbrains.git4idea.http.GitAskPassApp
error: unable to read askpass response from 'C:\Users\Алиса.IdeaIC2016.2\system\tmp\intellij-git-askpass.bat'
error: failed to execute prompt script (exit code 1)
fatal: could not read Username for 'https://github.com': No error

ice-phoenix · 2016-10-28T10:11:47Z

Можете приложить лог идеи (Help->Show Log in...)?

furoraru · 2016-10-28T10:48:51Z

2.txt

ice-phoenix · 2016-10-28T11:05:51Z

А давайте вы теперь снова попробуете сделать Git push и покажете новый лог после этого

furoraru · 2016-10-28T11:10:31Z

2.txt

ice-phoenix · 2016-10-28T11:18:14Z

Попробуйте переуказать ваши данные для GitHub и попробуйте снова

furoraru · 2016-10-28T11:21:10Z

Не помогло :с

ice-phoenix · 2016-10-28T11:36:02Z

А если открыть консоль и попробовать сделать git push origin master из директории, в которой у вас лежит проект?

furoraru · 2016-10-28T11:44:49Z

Я через консоль могу зайти в папку, но что делать дальше?

ice-phoenix · 2016-10-28T11:46:03Z

Дальше попробовать выполнить B:\Program Files\Git\Git\bin\git.exe push origin master

ice-phoenix · 2016-10-28T11:56:48Z

Запустите CRD в режиме шаринга и скиньте мне Access Code на почту akhin[at]kspt.icc.spbstu.ru

furoraru · 2016-12-12T20:49:54Z

recheck all

                   override fun set(row: Int, column: Int, value: E) {
-                      TODO()
+                      map[Pair(row, column)] = value

+                              row > 0 -> result[row - 1, column]
+                              column > 0 -> result[row, column - 1]
+                              else -> 0
+                          }

-                  TODO()
+                  val result = mutableMapOf<String, Int>()
+                  for (line in File(inputName).readLines()) {
+                      for (string in substrings) {

+                          if (parts.isNotEmpty()) {
+                              if (!result.contains(string))
+                                  result.put(string, parts.size - 1)
+                              else result.set(string, result.get(string)!! + parts.size -1)

                */
-              fun flattenPhoneNumber(phone: String): String = TODO()
+              fun flattenPhoneNumber(phone: String): String =
+                      if (phone.matches(Regex("""[\d\s()+-]+"""))) phone.filter { it !in "()- " }

Lesson 4 (+1cm fix) #181

Lesson 4 (+1cm fix) #181

Uh oh!

Conversation

furoraru commented Oct 22, 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!

Choose a reason for hiding this comment

Uh oh!

furoraru commented Oct 27, 2016

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

furoraru commented Oct 28, 2016

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

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

Uh oh!

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

furoraru commented Oct 28, 2016

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

furoraru commented Oct 28, 2016

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

ice-phoenix commented Oct 28, 2016

Uh oh!

furoraru commented Dec 12, 2016

Uh oh!

kotlin-polytech-bot commented Dec 12, 2016

author

lesson1.task1

lesson2.task1

lesson2.task2

lesson3.task1

lesson4.task1

lesson5.task1

lesson6.task1

lesson7.task1

lesson7.task2

lesson8.task1

owner

total

Uh oh!

furoraru commented Dec 14, 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

furoraru commented Oct 28, 2016 •

edited by ice-phoenix

Loading

furoraru commented Dec 16, 2016 •

edited

Loading