• Chào bạn, hãy đăng ký hoặc đăng nhập để tham gia cùng bọn mình và sử dụng được đầy đủ chức năng của diễn đàn :).
UI Nạp Thẻ GeyserMC - TheSieuToc

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

No permission to download

khoa22

THÀNH VIÊN
Tham gia
05/07/2018
Bài viết
23
Ý 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: 91
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:

View attachment 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

View attachment 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)

View attachment 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

View attachment 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()

View attachment 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

View attachment 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

View attachment 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

View attachment 41390

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

View attachment 41391

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

View attachment 41393

Không nên đặt output dir như này
 
Hóng plugin Login cho Pe ... 100đ
 
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
 
Similar content Most view Xem thêm
Back
Top Bottom