webページからリンク先のPDFファイルが開けない不具合の修正 #163
Conversation
| extension = url.substring(url.lastIndexOf(".")); | ||
| } | ||
| if(!url.contains(GOOGLE_DOCS_BASE_VIEWER_URL) | ||
| && Arrays.asList(arrOfficeFilenameExt).contains(extension.toLowerCase(Locale.getDefault()))) { |
There was a problem hiding this comment.
@vfa-cancc
Vì sao cần Locale.getDefault() ?
Có thể gọi hàm toLowerCase() mà không có tham số.
There was a problem hiding this comment.
Nếu không có tham số Locale sẽ bị lint warning khi build.
There was a problem hiding this comment.
@vfa-cancc
Theo tài liệu của java thì toLowerCase() tương đương toLowerCase(Locale.getDefault())
(https://docs.oracle.com/javase/7/docs/api/java/lang/String.html#toLowerCase())
Ngoài ra, anh tìm thử open source thì thấy bên trong toLowerCase() cũng gọi toLowerCase(Locale.getDefault()).
(http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/687fd7c7986d/src/share/classes/java/lang/String.java#l2654)
Nên để tránh warning này thì nên dùng suppressLint.
Bởi vì code theo cách toLowerCase(Locale.getDefault()) sẽ làm cho source code trở nên rối hơn, kì quặt và ko thống nhất giữa các hệ thống (chẳng hạn trong powerpush vẫn có rất nhiều chỗ xài toLowerCase()
There was a problem hiding this comment.
Đã chuyển sang sử dụng @SuppressLint("DefaultLocale")
vfa-duyna
changed the title
vfa-cancc
force-pushed
the
fix/open_link_pdf_in_richpush_webview
branch
from
88b1296
to
1c0c01d
Compare
概要(Summary)
動作確認手順(Step for Confirmation)
Run the unit test.
Confirm operation.