Blog - LINE ENGINEERING

TAG ARCHIVES: Code review

【DroidKaigi 2023】Code Review Challenge 5問目解説
Hidetsugu Tamaki / Ralph2023-09-29
LINEのAndroidアプリを作っています。
こんにちは。コミュニケーションアプリLINEのAndroid版クライアントを開発している玉木です。この記事では、DroidKaigi 2023の企業ブースで行ったCode Review Challengeの5問目の解説をします。 Code Review Challengeについてはこちらを参照してください。出題タイトル: Bitter Sweets 5問目では、以下のコードが出題されました。 class MainActivity : AppCompatActivity() { override fun onCreate(savedInstanceState: Bundle?) { super.onCreate(savedInstanceState) setContentView(R.layout.activity_main) val request: Request = if (Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU) { @Sup
- Android
- Code review
【DroidKaigi 2023】Code Review Challenge 4問目解説
Yuki Ando2023-09-27
iOS/Androidエクスペリエンス開発チーム
こんにちは。コミュニケーションアプリLINEのクライアントを開発している安藤です。この記事では、DroidKaigi2023の企業ブースで行ったCode Review Challengeの4問目の解説をします。 Code Review Challengeについてはこちらを参照してください。出題タイトル: Caught on Running 4問目では、以下のコードが出題されました。 class SomeActivity : AppCompatActivity() { private val logger: Logger = Logger() private val userDataRepository = UserDataRepository() override fun onCreate(savedInstanceState: Bundle?) { super.onCreate(savedInstanceState) setContentView(R.layout.activity_huge_legacy
- Android
- Code review
【DroidKaigi 2023】Code Review Challenge 3問目解説
Ishikawa Munetoshi2023-09-26
Developer Experience Development
こんにちは。コミュニケーションアプリLINEのクライアントを開発している石川です。この記事では、DroidKaigi 2023 の企業ブースで行った Code Review Challenge の 3 問目の解説をします。 Code Review Challengeについてはこちらを参照してください。出題タイトル: Slip Through My Async 3 問目では、以下のコードが出題されました。 /** * A repository class to save/load a [ShopItem] to the local storage. */ class ShopItemListRepository( private val shopItemDao: ShopItemDao, private val shopItemThumbnailDao: ShopItemThumbnailDao, private val errorReporter: ErrorReporter ) { /** [CoroutineScope] for DAO re
- Android
- Code review
【DroidKaigi 2023】Code Review Challenge 2問目解説
Atsushi Mori2023-09-26
メッセンジャープロダクトアプリ開発1チーム
こんにちは、コミュニケーションアプリ LINE のAndroidクライアントを開発している森です。この記事では、DroidKaigi2023の企業ブースで行ったCode Review Challengeの2問目の解説をします。Code Review Challengeについてはこちらを参照してください。出題タイトル : A Composable for everything, and everything in its Composable 2問目は以下のコードを出題しました。 @Composable fun LoginScreen() { Column( modifier = Modifier .padding(DEFAULT_SIZE.dp) ) { Spacer( modifier = Modifier.weight(2f) ) title() Spacer( modifier = Modifier.weigh
- Android
- Code review
【DroidKaigi 2023】Code Review Challenge 1問目解説
Yuta Takanashi2023-09-25
LINEアプリ基盤機能開発チーム
こんにちは。コミュニケーションアプリLINEのクライアントを開発している高梨です。この記事では、DroidKaigi2023の企業ブースで行ったCode Review Challengeの1問目の解説をします。 Code Review Challengeについてはこちらを参照してください。出題タイトル: Tip of the Ice-func 1問目では、以下のコードが出題されました。 object BackupCreator { private val localDataSource = LocalDataSource() private val remoteDataSource = RemoteDataSource() private lateinit var progressDialog: ProgressDialogForBackup @WorkerThread fun createBackup(target: BackupTarget, since: Date) { showProgressDialog()
- Android
- Code review
Code Review Challenge 5問目解説
Hidetsugu Tamaki / Ralph2022-12-09
LINEのAndroidアプリを作っています。
こんにちは。コミュニケーションアプリ LINE のクライアントを開発している玉木です。この記事では、DroidKaigi 2022 の企業ブースで行った Code Review Challenge の 5 問目の解説をします。Code Review Challenge についてはこちらを参照してください。出題タイトル: Coroutine's hard-to-understand trap that is easy to use for us 5 問目では、以下のようなコードが出題されました。 class MyContentProvider : ContentProvider() { private val singleThreadCoroutineDispatcher: CoroutineDispatcher = Executors.newSingleThreadExecutor().asCoroutineDispatcher() // Do not request again if succeeded once. privat
Code Review Challenge 4問目解説
Yuki Ando2022-12-09
iOS/Androidエクスペリエンス開発チーム
こんにちは。コミュニケーションアプリ LINE のクライアントを開発している安藤です。この記事では、DroidKaigi 2022 の企業ブースで行った Code Review Challenge の 4 問目の解説をします。Code Review Challenge についてはこちらを参照してください。出題タイトル: What layout state class was really needed? 4 問目では、以下のようなコードが出題されました。 data class StickerLayoutState( val stickerID: StickerID, val stickerName: String, val stickerImage: String, val stickerType: StickerType, val creatorName: String, val creatorImage: String, val reviewPageIndex: Int, val reviewPageTo
Code Review Challenge 3問目解説
Ishikawa Munetoshi2022-12-09
Developer Experience Development
こんにちは。コミュニケーションアプリ LINE のクライアントを開発している石川です。この記事では、DroidKaigi 2022 の企業ブースで行った Code Review Challenge の 3 問目の解説をします。Code Review Challenge についてはこちらを参照してください。出題タイトル: Truth in deep nests 3 問目では、以下のようなコードが出題されました。 class ProfileImageRepository( private val context: Context, private val metadataDao: ProfileImageMetadataDao = ProfileImageMetadataDao(), private val apiClient: RemoteApiClient = RemoteApiClient(context), private val ioScheduler: CoroutineContext = Dispatchers.IO ) { pri
Code Review Challenge 2問目解説
Kenichi Ikenaga2022-12-09
LINEコミュニケーション基盤開発チーム
こんにちは。コミュニケーションアプリ LINE のAndroidクライアントを開発している池永です。この記事では、DroidKaigi 2022 の企業ブースで行った Code Review Challenge の 2 問目の解説をします。Code Review Challenge についてはこちらを参照してください。出題タイトル: Split by object, not condition 2 問目では、以下のようなコードが出題されました。 class PlayerPresenter( private val loadingView: View, private val playButton: View, private val pauseButton: View, private val progressView: TextView, private val player: Player, private val lifecycleScope: CoroutineScope ) { private var isPlayi
Code Review Challenge 1問目解説
Kazuki Chigita 2022-12-09
LINEコミュニケーションサービス開発チーム
こんにちは。コミュニケーションアプリ LINE のクライアントを開発している千北です。この記事では、DroidKaigi 2022 の企業ブースで行った Code Review Challenge の 1 問目の解説をします。Code Review Challenge についてはこちらを参照してください。出題タイトル : Everything is something 1 問目では、以下のコードが出題されました。 sealed class ContactType { object Friend : ContactType() object Bot : ContactType() } class friend_item_Presenter( private val friendNameProvider: FriendNameProvider, private val coroutineScope: CoroutineScope ) { fun updateViews(itemId: String) { coroutineS

1 2