• Trong thời gian này, bạn không thể tạo bài mới, đăng comment, hay tạo tài nguyên mới để diễn đàn dọn dẹp nội dung.
    Diễn đàn sẽ sớm mở trở lại, chi tiết hãy theo dõi tại Discord
UI Nạp Thẻ GeyserMC - TheSieuToc

Plugin Việt Nam UI Nạp Thẻ GeyserMC - TheSieuToc v1.1

  • Người tạo chủ đề Người tạo chủ đề khoa22
  • Ngày gửi Ngày gửi
Ý tưởng tốt, cảm ơn bạn đã đăng plugin để đóng góp cho diễn đàn
Tuy nhiên mình có đọc qua source code và thấy một vài chỗ bạn xử lý hơi phức tạp, mình xin phép góp ý chút để bạn sửa và hoàn thiện plugin hơn trong những release tiếp theo :ok_hand:

41383


File config của bạn vẫn đang sử dụng kí tự § để định danh màu sắc, mình không rõ bên bedrock như nào, nhưng bạn nên viết một hàm colorize sử dụng ChatColor#translateAlternateColorCodes để replace các kí tự § thành & để mọi người dễ config hơn.
Đồng thời ở key xacnhan mình thấy bạn có kí tự \n để ngắt dòng, bạn có thể chuyển sang dùng stringlist để trông config của bạn gọn hơn, và đương nhiên sẽ giúp mọi người dễ tùy chỉnh hơn

41384


Package của bạn đang đặt sai theo convention của Java, hãy sửa lại thành me.anhkhoaaa.sosnapthepe nhé (tất cả viết lowercase)

41385


Về phần logger, bạn hãy dùng getLogger#log(Level, messages). Level của thể là INFO hoặc WARNING tùy theo nội dung log của bạn

41386


Cái này có vẻ không cần thiết lắm, bạn có thể xóa check này đi và register lệnh trong hàm onEnable()

41387


Đoạn này nếu sender là console hoặc commandblock thì sẽ ra sao ?
Bạn nên check nếu sender có phải là người chơi không, sau đó cast CommandSender -> Player

41388


Giảm thiểu việc trùng lặp code bằng cách lưu instance của FloodgateAPI vào biến trong class SOSNapthePE

41389


Nhà mạng và mệnh giá + message dropdown và một vài message khác đang hardcode nên được làm configurable

41390


If else hell, cái này nên đưa vào Enum thay vì get theo index như này

41391


Tương tự cho phần này, kiểu dữ liệu cho mệnh giá nên là int

41393


Không nên đặt output dir như này
 

Đính kèm

  • 1661951651718.png
    1661951651718.png
    1.6 KB · Lượt xem: 144
Thanks ạ mình luôn đón nhận góp ý để bản thân tốt hơn <3
Ý tưởng tốt, cảm ơn bạn đã đăng plugin để đóng góp cho diễn đàn
Tuy nhiên mình có đọc qua source code và thấy một vài chỗ bạn xử lý hơi phức tạp, mình xin phép góp ý chút để bạn sửa và hoàn thiện plugin hơn trong những release tiếp theo :ok_hand:

Xem tệp đính kèm 41383

File config của bạn vẫn đang sử dụng kí tự § để định danh màu sắc, mình không rõ bên bedrock như nào, nhưng bạn nên viết một hàm colorize sử dụng ChatColor#translateAlternateColorCodes để replace các kí tự § thành & để mọi người dễ config hơn.
Đồng thời ở key xacnhan mình thấy bạn có kí tự \n để ngắt dòng, bạn có thể chuyển sang dùng stringlist để trông config của bạn gọn hơn, và đương nhiên sẽ giúp mọi người dễ tùy chỉnh hơn

Xem tệp đính kèm 41384

Package của bạn đang đặt sai theo convention của Java, hãy sửa lại thành me.anhkhoaaa.sosnapthepe nhé (tất cả viết lowercase)

Xem tệp đính kèm 41385

Về phần logger, bạn hãy dùng getLogger#log(Level, messages). Level của thể là INFO hoặc WARNING tùy theo nội dung log của bạn

Xem tệp đính kèm 41386

Cái này có vẻ không cần thiết lắm, bạn có thể xóa check này đi và register lệnh trong hàm onEnable()

Xem tệp đính kèm 41387

Đoạn này nếu sender là console hoặc commandblock thì sẽ ra sao ?
Bạn nên check nếu sender có phải là người chơi không, sau đó cast CommandSender -> Player

Xem tệp đính kèm 41388

Giảm thiểu việc trùng lặp code bằng cách lưu instance của FloodgateAPI vào biến trong class SOSNapthePE

Xem tệp đính kèm 41389

Nhà mạng và mệnh giá + message dropdown và một vài message khác đang hardcode nên được làm configurable

Xem tệp đính kèm 41390

If else hell, cái này nên đưa vào Enum thay vì get theo index như này

Xem tệp đính kèm 41391

Tương tự cho phần này, kiểu dữ liệu cho mệnh giá nên là int

Xem tệp đính kèm 41393

Không nên đặt output dir như này
 
Plugin rất hay nhưng mong chủ tus cho plugin hỗ trợ nhiều phiên bản hơn như 1.16.5 trở lên
 
Nội dung tương tự Xem nhiều nhất Xem thêm

Thống kê nội dung

Tạo
khoa22,
Trả lời lần cuối từ
runtime123,
Trả lời
17
Lượt xem
2,276
Quay lại
Trên cùng