Skip to content

Dx Huong#12

Open
Lazygarde wants to merge 13 commits intomasterfrom
DXHuong
Open

Dx Huong#12
Lazygarde wants to merge 13 commits intomasterfrom
DXHuong

Conversation

@Lazygarde
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@DuongLH-Sudo DuongLH-Sudo left a comment

Choose a reason for hiding this comment

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

Cách làm của em lạ chứ k muốn nói là sai yêu cầu
đọc kỹ lại yêu cầu của anh
chú ý cách đặt tên biến, tên class

val secondFragment = sign_up_frm()

val sign_up_or_in = findViewById<Button>(R.id.sign_up_or_in)
val have_account_or_not = findViewById<TextView>(R.id.have_account_or_not)
Copy link
Collaborator

Choose a reason for hiding this comment

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

sao lại đặt tên biến như này?

commit()
}
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

sao k dùng binding?
sao k tạo 2 fragment riêng ra?

import android.os.Bundle
import android.widget.Button

class MainActivity2 : AppCompatActivity() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

cần đặt tên class có ý nghĩa

import android.widget.Button
import androidx.fragment.app.Fragment

class login_frm : Fragment(R.layout.frm_login) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ai dạy đặt tên class nư này đấy?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants