mirror of
https://github.com/bggRGjQaUbCoE/PiliPlus.git
synced 2026-04-23 04:00:28 +08:00
opt: unnecessary_non_null_assertion (#1762)
This commit is contained in:
committed by
GitHub
parent
b7a277a57c
commit
2b3ec77e92
@@ -67,8 +67,8 @@ abstract class CommonWhisperController<R>
|
||||
if (res.isSuccess) {
|
||||
if (loadingState.value.isSuccess) {
|
||||
List<Session>? list = loadingState.value.data;
|
||||
if (list?.isNotEmpty == true) {
|
||||
for (var item in list!) {
|
||||
if (list != null && list.isNotEmpty) {
|
||||
for (var item in list) {
|
||||
if (item.hasUnread()) {
|
||||
item.clearUnread();
|
||||
}
|
||||
|
||||
@@ -149,9 +149,9 @@ abstract class CommonDynPageState<T extends StatefulWidget> extends State<T>
|
||||
itemBuilder: (context, index) => const VideoReplySkeleton(),
|
||||
),
|
||||
Success(:var response) =>
|
||||
response?.isNotEmpty == true
|
||||
response != null && response.isNotEmpty
|
||||
? SliverList.builder(
|
||||
itemCount: response!.length + 1,
|
||||
itemCount: response.length + 1,
|
||||
itemBuilder: (context, index) {
|
||||
if (index == response.length) {
|
||||
controller.onLoadMore();
|
||||
|
||||
@@ -93,8 +93,8 @@ mixin CommonMultiSelectMixin<T extends MultiSelectData>
|
||||
void handleSelect({bool checked = false, bool disableSelect = true}) {
|
||||
if (loadingState.value.isSuccess) {
|
||||
final list = loadingState.value.data;
|
||||
if (list?.isNotEmpty == true) {
|
||||
for (var item in list!) {
|
||||
if (list != null && list.isNotEmpty) {
|
||||
for (var item in list) {
|
||||
item.checked = checked;
|
||||
}
|
||||
loadingState.refresh();
|
||||
@@ -165,7 +165,7 @@ mixin DeleteItemMixin<R, T extends MultiSelectData>
|
||||
// void handleSelect([bool checked = false, bool disableSelect = true]) {
|
||||
// if (loadingState.value.isSuccess) {
|
||||
// final list = loadingState.value.data;
|
||||
// if (list?.isNotEmpty == true) {
|
||||
// if (list != null && list.isNotEmpty) {
|
||||
// if (checked) {
|
||||
// selected.addAll(list!.map(getId));
|
||||
// } else {
|
||||
|
||||
@@ -95,8 +95,8 @@ abstract class CommonSearchPageState<S extends StatefulWidget, R, T>
|
||||
return switch (loadingState) {
|
||||
Loading() => const HttpError(),
|
||||
Success(:var response) =>
|
||||
response?.isNotEmpty == true
|
||||
? buildList(response!)
|
||||
response != null && response.isNotEmpty
|
||||
? buildList(response)
|
||||
: HttpError(onReload: controller.onReload),
|
||||
Error(:var errMsg) => HttpError(
|
||||
errMsg: errMsg,
|
||||
|
||||
Reference in New Issue
Block a user