Feature/cache layer

[Yujin1014]

캐싱 관련한 기능을 구현해본 적이 처음이라 캐싱 관련된 코드를 중점적으로 봐주시면 좋을 것 같습니다!

• 바코드 스캔 기능 구현
• OCR 데이터 캐싱 기능 추가
• 폴더명 및 클래스명 변경

[abdul0986]

[ISSUE #3] Feature/cache layer 와 같은 식으로 ISSUE와 연결해서 PR을 등록해주세요

[abdul0986]

Suggested change

	Barcode(String value) : value = value.trim();
	Barcode(String value): value = value.trim();

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	Future<void> saveToCache(String barcode, String data) async {
	Future<void> save(String barcode, String data) async {

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	Future<String?> loadFromCache(String barcode) async {
	Future<String?> load(String barcode) async {

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	String _getCacheKey(String barcode) {
	String _getKey(String barcode) {

[Yujin1014]

수정했습니다!

[abdul0986]

현재 startBarcodeScan()이 포함하고 있는 기능 목록

• 스캔
• 바코드 업데이트
• 에러 처리
• 디버그용 임시 데이터 처리

[abdul0986]

startBarcodeScan() : Refactoring

• Future<void> startBarcodeScan(BuildContext context)
• Future<Barcode?> _performBarcodeScan(BuildContext context)
• Future<void> _handleScannerBarcode(BuildContext context, String barcodeValue) async
• Future<void> _checkDebugModeCachedData(BuildContext context) async
• void _setDebugBarcodeTemp()

[Yujin1014]

메소드를 더 잘게 나누어 수정했습니다!

[abdul0986]

context.mounded처럼 조건이 명확하고, 실패할 경우 이후 코드를 진행할 필요가 없을 때는 early return 처리하기!!!

if (!context.mounted) return; 

final cachedData = await _cacheService.load(barcode ?? ''); 
if (cachedData != null) {
  _navigateToIntakeGuide(context, cachedData); 
} else {
  _navigateToNutrientScan(context); 
}

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	Future<void> saveToCache(String key, String value) async {
	Future<void> save(String key, String value) async {

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	Future<String?> loadFromCache(String key) async {
	Future<String?> load(String key) async {

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	Future<void> removeFromCache(String key) async {
	Future<void> remove(String key) async {

[Yujin1014]

수정했습니다!

[abdul0986]

Suggested change

	final prefs = await SharedPreferences.getInstance();
	final cachedValue = prefs.getString(key);
	final expiryDateStr = prefs.getString('${key}_expiry');
	if (cachedValue != null && expiryDateStr != null) {
	final expiryDate = DateTime.parse(expiryDateStr);
	if (DateTime.now().isBefore(expiryDate)) {
	return cachedValue;
	} else {
	// Remove expired cache
	await prefs.remove(key);
	await prefs.remove('${key}_expiry');
	}
	}
	return null;
	final prefs = await SharedPreferences.getInstance();
	final cachedValue = _getCachedValue(prefs, key);
	final expiryDate = _getExpiryDate(prefs, key);
	if (cachedValue == null || expiryDateStr == null) {
	return null;
	}
	if (_isExpired(expiryDate)
	await prefs.remove(key);
	await prefs.remove('${key}_expiry');
	}
	return cachedValue;

String? _getCachedValue(SharedPreferences pref, String key) {
    return prefs.getString(key);
}

Datetime _getExpiryDate(SharedPreferences pref, String key) {
    final expireDateStr = prefs.getString('${key}_expiry'); 
    if (expiryDateStr == null) return null; 

    try {
        return Datetime.parse(expireDateStr); 
    } catch (_) {
        return null; 
    }
}

bool _isExpired(Datetime expiryDate) {
    return Datetime.now().isAfter(expiryDate);
}

[Yujin1014]

명확하게 리팩토링 하였습니다!

[abdul0986]

BarcodeCacheService는 XXXService이기 때문에, 최대한 비즈니스 디펜던시가 있는 부분을 모두 포함하는 객체라고 생각

[abdul0986]

Suggested change

	if (kDebugMode) {
	_setDebugBarcodeTemp();
	if (context.mounted) {
	await _checkCachedData(context, scannedBarcode ?? '');
	}
	}
	if (!kDebugMode) {
	return;
	}
	_setDebugBarcodeTemp();
	await _checkCachedData(context, scannedBarcode ?? '');

[abdul0986]

Suggested change

	if (cachedAnswer != null) {
	return AnalysisResult(cachedAnswer);
	}
	return null;
	return cachedAnswer != null ? AnalysisResult(cachedAnswer) : null;

[abdul0986]

피드백 반영 부탁드려요

[Yujin1014]

피드백 반영하여 코드 다시 정리하였습니다.

[abdul0986]

LGTM