ViewModelの責務について

[YumNumm]

問題

• ViewModel層で処理すべき実装がView層でされている
  

  github-repository-search/lib/ui/page/search_view/component/repository_search_list.dart

  Lines 24 to 47 in 64817a0

  	onNotification: (notification) {
  	// 検索結果を取得していない場合・検索結果がない場合は
  	// 何もしない
  	if (!data.hasValue) {
  	return false;
  	}
  	final scrollPosition = notification.metrics.pixels /
  	notification.metrics.maxScrollExtent;
  	if (scrollPosition > 0.8 && (data.value!.isNotEmpty)) {
  	// 既に検索結果を全て読み込んでいる場合は、
  	// 次のページを読み込む処理を行わない
  	if (data.value!.length ==
  	(ref.read(totalRepositoryCountProvider).value ?? 0)) {
  	return false;
  	}
  	ref
  	.read(repositorySearchViewModel.notifier)
  	.loadMoreRepositories();
  	return true;
  	}
  	return false;
  	},

  
  

  github-repository-search/lib/ui/page/search_view/component/repository_search_list.dart

  Lines 51 to 66 in 64817a0

  	itemBuilder: (context, index) {
  	if (index == items.length) {
  	// InfiniteScroll Indicator
  	return const Padding(
  	padding: EdgeInsets.symmetric(vertical: 32),
  	child: Center(
  	child: CircularProgressIndicator.adaptive(),
  	),
  	);
  	}
  	final item = items[index];
  	return RepositoryItemWidget(
  	item: item,
  	term: ref.watch(searchRepositoryNameProvider) ?? '',
  	);
  	},

参考

• ViewModelのメソッド名はイベントを表す命名にしたいという話
• MVVMとはなんぞやを公理から求めてみる
• アーキテクチャ系の本も探してみる