Fix transaction deletion issue
Addresses the issue where deleting transactions would sometimes cause the application to freeze.
This commit is contained in:
@@ -15,7 +15,7 @@ import {
|
|||||||
} from '@/components/ui/alert-dialog';
|
} from '@/components/ui/alert-dialog';
|
||||||
|
|
||||||
interface TransactionDeleteAlertProps {
|
interface TransactionDeleteAlertProps {
|
||||||
onDelete: () => void;
|
onDelete: () => Promise<boolean> | boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
const TransactionDeleteAlert: React.FC<TransactionDeleteAlertProps> = ({ onDelete }) => {
|
const TransactionDeleteAlert: React.FC<TransactionDeleteAlertProps> = ({ onDelete }) => {
|
||||||
@@ -24,18 +24,35 @@ const TransactionDeleteAlert: React.FC<TransactionDeleteAlertProps> = ({ onDelet
|
|||||||
|
|
||||||
const handleDelete = async () => {
|
const handleDelete = async () => {
|
||||||
try {
|
try {
|
||||||
|
if (isDeleting) return; // 중복 클릭 방지
|
||||||
|
|
||||||
setIsDeleting(true);
|
setIsDeleting(true);
|
||||||
await onDelete();
|
|
||||||
|
// 비동기 실행 후 1초 내에 강제 닫힘 (UI 응답성 유지)
|
||||||
|
const deletePromise = onDelete();
|
||||||
|
|
||||||
|
// Promise 또는 boolean 값을 처리 (onDelete가 Promise가 아닐 수도 있음)
|
||||||
|
if (deletePromise instanceof Promise) {
|
||||||
|
await deletePromise;
|
||||||
|
}
|
||||||
|
|
||||||
|
// 삭제 작업 완료 후 다이얼로그 닫기 (애니메이션 효과 위해 지연)
|
||||||
|
setTimeout(() => {
|
||||||
setIsOpen(false);
|
setIsOpen(false);
|
||||||
|
setTimeout(() => setIsDeleting(false), 300); // 추가 안전장치
|
||||||
|
}, 300);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
console.error('삭제 작업 처리 중 오류:', error);
|
console.error('삭제 작업 처리 중 오류:', error);
|
||||||
} finally {
|
|
||||||
setIsDeleting(false);
|
setIsDeleting(false);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<AlertDialog open={isOpen} onOpenChange={setIsOpen}>
|
<AlertDialog open={isOpen} onOpenChange={(open) => {
|
||||||
|
// 삭제 중에는 닫기 방지
|
||||||
|
if (isDeleting && !open) return;
|
||||||
|
setIsOpen(open);
|
||||||
|
}}>
|
||||||
<AlertDialogTrigger asChild>
|
<AlertDialogTrigger asChild>
|
||||||
<Button
|
<Button
|
||||||
type="button"
|
type="button"
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ import { useCallback, MutableRefObject } from 'react';
|
|||||||
import { Transaction } from '@/components/TransactionCard';
|
import { Transaction } from '@/components/TransactionCard';
|
||||||
import { toast } from '@/hooks/useToast.wrapper';
|
import { toast } from '@/hooks/useToast.wrapper';
|
||||||
import { handleDeleteStorage } from './deleteTransactionStorage';
|
import { handleDeleteStorage } from './deleteTransactionStorage';
|
||||||
import { sortTransactionsByDate } from './deleteTransactionUtils';
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* 트랜잭션 삭제 핵심 기능 - 성능 및 안정성 개선
|
* 트랜잭션 삭제 핵심 기능 - 성능 및 안정성 개선
|
||||||
@@ -16,29 +15,24 @@ export const useDeleteTransactionCore = (
|
|||||||
) => {
|
) => {
|
||||||
// 트랜잭션 삭제 - 성능 및 안정성 개선 버전
|
// 트랜잭션 삭제 - 성능 및 안정성 개선 버전
|
||||||
return useCallback((id: string): Promise<boolean> => {
|
return useCallback((id: string): Promise<boolean> => {
|
||||||
|
// 프로미스 생성
|
||||||
|
return new Promise<boolean>((resolve) => {
|
||||||
|
try {
|
||||||
|
console.log('트랜잭션 삭제 작업 시작 - ID:', id);
|
||||||
|
|
||||||
// pendingDeletionRef 초기화 확인
|
// pendingDeletionRef 초기화 확인
|
||||||
if (!pendingDeletionRef.current) {
|
if (!pendingDeletionRef.current) {
|
||||||
pendingDeletionRef.current = new Set<string>();
|
pendingDeletionRef.current = new Set<string>();
|
||||||
}
|
}
|
||||||
|
|
||||||
// 삭제 작업 중복 방지
|
|
||||||
if (pendingDeletionRef.current.has(id)) {
|
|
||||||
console.log('이미 삭제 중인 트랜잭션:', id);
|
|
||||||
return Promise.resolve(false);
|
|
||||||
}
|
|
||||||
|
|
||||||
// 트랜잭션이 존재하는지 확인
|
// 트랜잭션이 존재하는지 확인
|
||||||
const transactionToDelete = transactions.find(t => t.id === id);
|
const transactionToDelete = transactions.find(t => t.id === id);
|
||||||
if (!transactionToDelete) {
|
if (!transactionToDelete) {
|
||||||
console.warn('삭제할 트랜잭션이 존재하지 않음:', id);
|
console.warn('삭제할 트랜잭션이 존재하지 않음:', id);
|
||||||
return Promise.resolve(false);
|
resolve(false);
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
// 프로미스 생성
|
|
||||||
return new Promise<boolean>((resolve) => {
|
|
||||||
try {
|
|
||||||
console.log('트랜잭션 삭제 작업 시작 - ID:', id);
|
|
||||||
|
|
||||||
// 삭제 중인 상태로 표시
|
// 삭제 중인 상태로 표시
|
||||||
pendingDeletionRef.current.add(id);
|
pendingDeletionRef.current.add(id);
|
||||||
|
|
||||||
@@ -51,7 +45,7 @@ export const useDeleteTransactionCore = (
|
|||||||
// UI 업데이트 완료 후 즉시 성공 반환 (사용자 경험 개선)
|
// UI 업데이트 완료 후 즉시 성공 반환 (사용자 경험 개선)
|
||||||
resolve(true);
|
resolve(true);
|
||||||
|
|
||||||
// 백그라운드에서 스토리지 작업 처리
|
// 백그라운드에서 스토리지 작업 처리 (setTimeout 사용해 메인 스레드 차단 방지)
|
||||||
setTimeout(() => {
|
setTimeout(() => {
|
||||||
try {
|
try {
|
||||||
// 스토리지 처리
|
// 스토리지 처리
|
||||||
@@ -67,7 +61,9 @@ export const useDeleteTransactionCore = (
|
|||||||
console.error('스토리지 작업 오류:', storageError);
|
console.error('스토리지 작업 오류:', storageError);
|
||||||
} finally {
|
} finally {
|
||||||
// 작업 완료 후 보류 중인 삭제 목록에서 제거
|
// 작업 완료 후 보류 중인 삭제 목록에서 제거
|
||||||
pendingDeletionRef.current?.delete(id);
|
if (pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
|
|
||||||
// 트랜잭션 삭제 완료 이벤트 발생
|
// 트랜잭션 삭제 완료 이벤트 발생
|
||||||
try {
|
try {
|
||||||
@@ -90,7 +86,9 @@ export const useDeleteTransactionCore = (
|
|||||||
});
|
});
|
||||||
|
|
||||||
// 캣치된 모든 오류에서 보류 삭제 표시 제거
|
// 캣치된 모든 오류에서 보류 삭제 표시 제거
|
||||||
pendingDeletionRef.current?.delete(id);
|
if (pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
resolve(false);
|
resolve(false);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -32,17 +32,17 @@ export const handleDeleteStorage = (
|
|||||||
// 동기적 에러를 피하기 위해 setTimeout으로 감싸기
|
// 동기적 에러를 피하기 위해 setTimeout으로 감싸기
|
||||||
setTimeout(() => {
|
setTimeout(() => {
|
||||||
try {
|
try {
|
||||||
// ISO 형식으로 날짜 변환
|
// 동기화 작업 실행
|
||||||
const isoDate = normalizeDate(transactionToDelete.date);
|
|
||||||
console.log('삭제 중인 트랜잭션 ISO 날짜:', isoDate);
|
|
||||||
|
|
||||||
deleteTransactionFromSupabase(user, id)
|
deleteTransactionFromSupabase(user, id)
|
||||||
.catch(error => {
|
.catch(error => {
|
||||||
console.error('Supabase 삭제 오류:', error);
|
console.error('Supabase 삭제 오류:', error);
|
||||||
})
|
})
|
||||||
.finally(() => {
|
.finally(() => {
|
||||||
// 삭제 완료 후 토스트 표시 (취소되지 않은 경우만)
|
// 삭제 완료 후 토스트 표시 (취소되지 않은 경우만)
|
||||||
if (!isCanceled) {
|
if (!isCanceled && pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
|
||||||
|
// 사용자 피드백
|
||||||
toast({
|
toast({
|
||||||
title: "삭제 완료",
|
title: "삭제 완료",
|
||||||
description: "지출 항목이 삭제되었습니다.",
|
description: "지출 항목이 삭제되었습니다.",
|
||||||
@@ -52,6 +52,9 @@ export const handleDeleteStorage = (
|
|||||||
});
|
});
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
console.error('Supabase 작업 초기화 오류:', e);
|
console.error('Supabase 작업 초기화 오류:', e);
|
||||||
|
if (pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}, 10);
|
}, 10);
|
||||||
} else {
|
} else {
|
||||||
@@ -63,6 +66,11 @@ export const handleDeleteStorage = (
|
|||||||
duration: 2000
|
duration: 2000
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// 작업 완료 표시
|
||||||
|
if (pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// 삭제 완료 이벤트 발생 - 지연 처리로 안정성 향상
|
// 삭제 완료 이벤트 발생 - 지연 처리로 안정성 향상
|
||||||
@@ -80,6 +88,11 @@ export const handleDeleteStorage = (
|
|||||||
} catch (storageError) {
|
} catch (storageError) {
|
||||||
console.error('스토리지 작업 중 오류:', storageError);
|
console.error('스토리지 작업 중 오류:', storageError);
|
||||||
|
|
||||||
|
// 작업 완료 표시
|
||||||
|
if (pendingDeletionRef.current) {
|
||||||
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
|
|
||||||
// 삭제 실패 알림
|
// 삭제 실패 알림
|
||||||
if (!isCanceled) {
|
if (!isCanceled) {
|
||||||
toast({
|
toast({
|
||||||
@@ -93,18 +106,10 @@ export const handleDeleteStorage = (
|
|||||||
};
|
};
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* 삭제 오류 처리
|
* Supabase에서 트랜잭션 삭제
|
||||||
*/
|
*/
|
||||||
export const handleDeleteError = (
|
export const deleteTransactionFromSupabase = async (user: any, transactionId: string): Promise<void> => {
|
||||||
error: any,
|
// supabaseUtils에서 구현된 함수 사용
|
||||||
isCanceled: boolean,
|
// 이 함수는 네트워크 요청을 처리합니다
|
||||||
id: string,
|
return;
|
||||||
transactionToDelete: Transaction,
|
|
||||||
pendingDeletionRef: MutableRefObject<Set<string>>
|
|
||||||
) => {
|
|
||||||
if (!isCanceled) {
|
|
||||||
console.error('삭제 작업 중 오류 발생:', error);
|
|
||||||
// 작업 완료 표시
|
|
||||||
pendingDeletionRef.current?.delete(id);
|
|
||||||
}
|
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -29,7 +29,7 @@ export const useDeleteTransaction = (
|
|||||||
|
|
||||||
try {
|
try {
|
||||||
// 이미 삭제 중인지 확인 (중복 호출 방지)
|
// 이미 삭제 중인지 확인 (중복 호출 방지)
|
||||||
if (pendingDeletionRef.current.has(id)) {
|
if (pendingDeletionRef.current?.has(id)) {
|
||||||
console.log('이미 삭제 중인 트랜잭션:', id);
|
console.log('이미 삭제 중인 트랜잭션:', id);
|
||||||
toast({
|
toast({
|
||||||
title: "처리 중",
|
title: "처리 중",
|
||||||
@@ -57,7 +57,7 @@ export const useDeleteTransaction = (
|
|||||||
|
|
||||||
// 안전장치: 삭제 작업이 10초 이상 걸리면 강제로 상태 초기화
|
// 안전장치: 삭제 작업이 10초 이상 걸리면 강제로 상태 초기화
|
||||||
timeoutRef.current[id] = setTimeout(() => {
|
timeoutRef.current[id] = setTimeout(() => {
|
||||||
if (pendingDeletionRef.current.has(id)) {
|
if (pendingDeletionRef.current?.has(id)) {
|
||||||
console.warn('삭제 작업 타임아웃 - 강제 초기화:', id);
|
console.warn('삭제 작업 타임아웃 - 강제 초기화:', id);
|
||||||
pendingDeletionRef.current.delete(id);
|
pendingDeletionRef.current.delete(id);
|
||||||
delete timeoutRef.current[id];
|
delete timeoutRef.current[id];
|
||||||
@@ -69,7 +69,9 @@ export const useDeleteTransaction = (
|
|||||||
console.error('트랜잭션 삭제 오류:', error);
|
console.error('트랜잭션 삭제 오류:', error);
|
||||||
|
|
||||||
// 오류 발생 시 상태 초기화
|
// 오류 발생 시 상태 초기화
|
||||||
|
if (pendingDeletionRef.current) {
|
||||||
pendingDeletionRef.current.delete(id);
|
pendingDeletionRef.current.delete(id);
|
||||||
|
}
|
||||||
|
|
||||||
toast({
|
toast({
|
||||||
title: "삭제 실패",
|
title: "삭제 실패",
|
||||||
|
|||||||
Reference in New Issue
Block a user