Skip to content
This repository has been archived by the owner on Mar 1, 2021. It is now read-only.

Students list #15

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Students list #15

wants to merge 7 commits into from

Conversation

yaroslaver
Copy link

No description provided.

@otopba
Copy link

otopba commented Feb 28, 2019

Проект не собирается. Похоже забыл подключить библиотеку RecyclerView
image

@otopba otopba self-requested a review February 28, 2019 20:13
@otopba
Copy link

otopba commented Mar 5, 2019

@yaroslaver

  1. Пожалуйста, удали из репозитория папку .idea
  2. Кнопка "добавить студента" выглядит странно
  3. Клавиатура перекрывает поля ввода
  4. Можно сохранить студента с пустым именем и фамилией
    image

@otopba
Copy link

otopba commented Mar 27, 2019

  1. Нет выделения выбранного студента
  2. При заполнении полей и нажатии "добавить студента" хотелось бы что бы введенные данные применялись к новому студенту. А сейчас всегда создается дефолтный студент
  3. Длинные имена ломают верстку
  4. Создал несколько студентов, потом проскролил список вверх, там почему-то оказался студент без имени и фамилии и какие-то отступы (скрин прикрепляю)

image

private final String DEFAULTSECONDNAME = "Secondname";

private Student prevStudent;
private final Student DEFAULTPREV = new Student(" ", " ", false, android.R.color.transparent);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше DEFAULT_PREV

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему вместо id фото стоит цвет?

@@ -45,4 +47,12 @@ public int getPhoto() {
public void setPhoto(int photo) {
this.photo = photo;
}

public void setInFocus(boolean focus) {
this.inFocus = focus;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут наверное стоит сделать название поля класса тоже focus

}

private String reflectTextInTextView(String text) {
if (text.length() > 10) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для этого у TextView есть ellipsize

@otopba
Copy link

otopba commented Apr 29, 2019

  1. Ввод двух строк все также режется построчно
  2. Можно создавать студентов просто кликая "добавить студента" не введя никаких полей

@otopba
Copy link

otopba commented Apr 29, 2019

Работа принята
7 балов

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants