From a7695c7365424ec14529998d4977bb9eaddc4cb8 Mon Sep 17 00:00:00 2001 From: zdl <3489966805@qq.com> Date: Sat, 18 Oct 2025 12:26:08 +0800 Subject: [PATCH] =?UTF-8?q?feat:=20=E6=97=A5=E5=BF=97=E4=BC=98=E5=8C=96?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/App.js | 11 ++++-- src/components/ErrorBoundary.js | 13 +++++-- src/components/Navbars/HomeNavbar.js | 37 ++++++++++++++------ src/components/StockChart/StockChartModal.js | 21 ++++++++--- src/services/financialService.js | 21 ++++++++--- src/services/marketService.js | 35 +++++++++++++----- src/utils/citationUtils.js | 25 ++++++++++--- 7 files changed, 127 insertions(+), 36 deletions(-) diff --git a/src/App.js b/src/App.js index 32c87d9f..e09c0fc9 100755 --- a/src/App.js +++ b/src/App.js @@ -50,6 +50,7 @@ import ProtectedRoute from "components/ProtectedRoute"; import ErrorBoundary from "components/ErrorBoundary"; import AuthModalManager from "components/Auth/AuthModalManager"; import ScrollToTop from "components/ScrollToTop"; +import { logger } from "utils/logger"; function AppContent() { const { colorMode } = useColorMode(); @@ -172,13 +173,19 @@ export default function App() { // 全局错误处理：捕获未处理的 Promise rejection useEffect(() => { const handleUnhandledRejection = (event) => { - console.error('未捕获的 Promise rejection:', event.reason); + logger.error('App', 'unhandledRejection', event.reason instanceof Error ? event.reason : new Error(String(event.reason)), { + promise: event.promise + }); // 阻止默认的错误处理（防止崩溃） event.preventDefault(); }; const handleError = (event) => { - console.error('全局错误:', event.error); + logger.error('App', 'globalError', event.error || new Error(event.message), { + filename: event.filename, + lineno: event.lineno, + colno: event.colno + }); // 阻止默认的错误处理（防止崩溃） event.preventDefault(); }; diff --git a/src/components/ErrorBoundary.js b/src/components/ErrorBoundary.js index 317e79ae..ba808f55 100755 --- a/src/components/ErrorBoundary.js +++ b/src/components/ErrorBoundary.js @@ -9,6 +9,7 @@ import { VStack, Container } from '@chakra-ui/react'; +import { logger } from '../utils/logger'; class ErrorBoundary extends React.Component { constructor(props) { @@ -28,14 +29,20 @@ class ErrorBoundary extends React.Component { componentDidCatch(error, errorInfo) { // 开发环境：打印错误到控制台，但不显示错误边界 if (process.env.NODE_ENV === 'development') { - console.error('🔴 ErrorBoundary 捕获到错误（开发模式，不拦截）:'); - console.error('错误:', error); - console.error('错误信息:', errorInfo); + logger.error('ErrorBoundary', 'componentDidCatch', error, { + componentStack: errorInfo.componentStack, + developmentMode: true + }); // 不更新 state，让错误继续抛出 return; } // 生产环境：保存错误信息到 state + logger.error('ErrorBoundary', 'componentDidCatch', error, { + componentStack: errorInfo.componentStack, + productionMode: true + }); + this.setState({ error: error, errorInfo: errorInfo diff --git a/src/components/Navbars/HomeNavbar.js b/src/components/Navbars/HomeNavbar.js index ed545b15..1f6008b0 100644 --- a/src/components/Navbars/HomeNavbar.js +++ b/src/components/Navbars/HomeNavbar.js @@ -37,6 +37,7 @@ import { FiStar, FiCalendar } from 'react-icons/fi'; import { useNavigate, useLocation } from 'react-router-dom'; import { useAuth } from '../../contexts/AuthContext'; import { useAuthModal } from '../../contexts/AuthModalContext'; +import { logger } from '../../utils/logger'; /** 二级导航栏组件 - 显示当前一级菜单下的所有二级菜单项 */ const SecondaryNav = () => { @@ -366,11 +367,11 @@ export default function HomeNavbar() { const toast = useToast(); // 添加调试信息 - console.log('HomeNavbar Debug:', { - user, + logger.debug('HomeNavbar', '组件渲染状态', { + hasUser: !!user, isAuthenticated, isLoading, - userKeys: user ? Object.keys(user) : 'no user' + userId: user?.id }); // 获取显示名称的函数 @@ -389,7 +390,9 @@ export default function HomeNavbar() { setShowCompletenessAlert(false); // logout函数已经包含了跳转逻辑，这里不需要额外处理 } catch (error) { - console.error('Logout error:', error); + logger.error('HomeNavbar', 'handleLogout', error, { + userId: user?.id + }); } }; @@ -434,7 +437,9 @@ export default function HomeNavbar() { setWatchlistQuotes([]); } } catch (e) { - console.warn('加载自选股实时行情失败:', e); + logger.warn('HomeNavbar', '加载自选股实时行情失败', { + error: e.message + }); setWatchlistQuotes([]); } finally { setWatchlistLoading(false); @@ -482,7 +487,9 @@ export default function HomeNavbar() { setFollowingEvents([]); } } catch (e) { - console.warn('加载关注事件失败:', e); + logger.warn('HomeNavbar', '加载关注事件失败', { + error: e.message + }); setFollowingEvents([]); } finally { setEventsLoading(false); @@ -550,12 +557,16 @@ export default function HomeNavbar() { // 如果已经检查过，跳过（避免重复请求） if (hasCheckedCompleteness.current) { - console.log('[Profile] 已检查过资料完整性，跳过重复请求'); + logger.debug('HomeNavbar', '已检查过资料完整性，跳过重复请求', { + userId: user?.id + }); return; } try { - console.log('[Profile] 开始检查资料完整性...'); + logger.debug('HomeNavbar', '开始检查资料完整性', { + userId: user?.id + }); const base = getApiBase(); const resp = await fetch(base + '/api/account/profile-completeness', { credentials: 'include' @@ -569,11 +580,17 @@ export default function HomeNavbar() { setShowCompletenessAlert(data.data.needsAttention); // 标记为已检查 hasCheckedCompleteness.current = true; - console.log('[Profile] 资料完整性检查完成:', data.data); + logger.debug('HomeNavbar', '资料完整性检查完成', { + userId: user?.id, + completeness: data.data.completenessPercentage + }); } } } catch (error) { - console.warn('检查资料完整性失败:', error); + logger.warn('HomeNavbar', '检查资料完整性失败', { + userId: user?.id, + error: error.message + }); } }, [isAuthenticated, user]); // 移除 getApiBase 依赖，因为它现在在组件外部 diff --git a/src/components/StockChart/StockChartModal.js b/src/components/StockChart/StockChartModal.js index 4e82b99c..d83fd558 100644 --- a/src/components/StockChart/StockChartModal.js +++ b/src/components/StockChart/StockChartModal.js @@ -5,6 +5,7 @@ import ReactECharts from 'echarts-for-react'; import * as echarts from 'echarts'; import moment from 'moment'; import { stockService } from '../../services/eventService'; +import { logger } from '../../utils/logger'; const StockChartModal = ({ isOpen, @@ -36,14 +37,20 @@ const StockChartModal = ({ adjustedEventTime = nextDay.format('YYYY-MM-DD HH:mm'); } } catch (e) { - console.warn('事件时间解析失败:', e); + logger.warn('StockChartModal', '事件时间解析失败', { + eventTime, + error: e.message + }); } } const response = await stockService.getKlineData(stock.stock_code, type, adjustedEventTime); setPreloadedData(prev => ({...prev, [type]: response})); } catch (err) { - console.error(`预加载${type}数据失败:`, err); + logger.error('StockChartModal', 'preloadData', err, { + stockCode: stock?.stock_code, + type + }); } }; @@ -91,7 +98,10 @@ const StockChartModal = ({ adjustedEventTime = nextDay.format('YYYY-MM-DD HH:mm'); } } catch (e) { - console.warn('事件时间解析失败:', e); + logger.warn('StockChartModal', '事件时间解析失败', { + eventTime, + error: e.message + }); } } @@ -113,7 +123,10 @@ const StockChartModal = ({ chartInstanceRef.current.setOption(option, true); } } catch (err) { - console.error('加载图表数据失败:', err); + logger.error('StockChartModal', 'loadChartData', err, { + stockCode: stock?.stock_code, + chartType: type + }); } finally { setLoading(false); } diff --git a/src/services/financialService.js b/src/services/financialService.js index 8bd9149f..84feeb80 100644 --- a/src/services/financialService.js +++ b/src/services/financialService.js @@ -4,12 +4,17 @@ * 对应Flask后端的所有财务API接口 */ +import { logger } from '../utils/logger'; + const isProduction = process.env.NODE_ENV === 'production'; const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL; const apiRequest = async (url, options = {}) => { try { - console.log(`Making Financial API request to: ${API_BASE_URL}${url}`); + logger.debug('financialService', 'API请求', { + url: `${API_BASE_URL}${url}`, + method: options.method || 'GET' + }); const response = await fetch(`${API_BASE_URL}${url}`, { ...options, @@ -22,16 +27,24 @@ const apiRequest = async (url, options = {}) => { if (!response.ok) { const errorText = await response.text(); - console.error(`Financial API request failed: ${response.status} - ${errorText}`); + logger.error('financialService', 'apiRequest', new Error(`HTTP ${response.status}`), { + url, + status: response.status, + errorText: errorText.substring(0, 200) + }); throw new Error(`HTTP error! status: ${response.status}`); } const data = await response.json(); - console.log(`Financial API response from ${url}:`, data); + logger.debug('financialService', 'API响应', { + url, + success: data.success, + hasData: !!data.data + }); return data; } catch (error) { - console.error(`Financial API request failed for ${url}:`, error); + logger.error('financialService', 'apiRequest', error, { url }); throw error; } }; diff --git a/src/services/marketService.js b/src/services/marketService.js index 376d5c61..94e887a1 100644 --- a/src/services/marketService.js +++ b/src/services/marketService.js @@ -4,12 +4,17 @@ * 对应Flask后端的所有市场API接口 */ +import { logger } from '../utils/logger'; + const isProduction = process.env.NODE_ENV === 'production'; const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL; const apiRequest = async (url, options = {}) => { try { - console.log(`Making Market API request to: ${API_BASE_URL}${url}`); + logger.debug('marketService', 'API请求', { + url: `${API_BASE_URL}${url}`, + method: options.method || 'GET' + }); const response = await fetch(`${API_BASE_URL}${url}`, { ...options, @@ -22,16 +27,24 @@ const apiRequest = async (url, options = {}) => { if (!response.ok) { const errorText = await response.text(); - console.error(`Market API request failed: ${response.status} - ${errorText}`); + logger.error('marketService', 'apiRequest', new Error(`HTTP ${response.status}`), { + url, + status: response.status, + errorText: errorText.substring(0, 200) + }); throw new Error(`HTTP error! status: ${response.status}`); } const data = await response.json(); - console.log(`Market API response from ${url}:`, data); + logger.debug('marketService', 'API响应', { + url, + success: data.success, + hasData: !!data.data + }); return data; } catch (error) { - console.error(`Market API request failed for ${url}:`, error); + logger.error('marketService', 'apiRequest', error, { url }); throw error; } }; @@ -449,7 +462,7 @@ export const marketCacheManager = { if (cached) { const { data, timestamp } = cached; if (Date.now() - timestamp < maxAge) { - console.log(`Cache hit for key: ${key}`); + logger.debug('marketCacheManager', '缓存命中', { key }); return data; } this.cache.delete(key); @@ -467,15 +480,16 @@ export const marketCacheManager = { data, timestamp: Date.now() }); - console.log(`Data cached for key: ${key}`); + logger.debug('marketCacheManager', '数据已缓存', { key }); }, /** * 清除所有缓存 */ clear: function() { + const cacheSize = this.cache.size; this.cache.clear(); - console.log('All market cache cleared'); + logger.debug('marketCacheManager', '清除所有缓存', { clearedCount: cacheSize }); }, /** @@ -483,12 +497,17 @@ export const marketCacheManager = { * @param {string} stockCode - 股票代码 */ clearStock: function(stockCode) { + let clearedCount = 0; for (const key of this.cache.keys()) { if (key.includes(`market_${stockCode}_`)) { this.cache.delete(key); + clearedCount++; } } - console.log(`Cache cleared for stock: ${stockCode}`); + logger.debug('marketCacheManager', '清除股票缓存', { + stockCode, + clearedCount + }); } }; diff --git a/src/utils/citationUtils.js b/src/utils/citationUtils.js index d37f7f29..09aa8d8a 100644 --- a/src/utils/citationUtils.js +++ b/src/utils/citationUtils.js @@ -1,6 +1,8 @@ // src/utils/citationUtils.js // 引用数据处理工具 +import { logger } from './logger'; + /** * 处理后端返回的引用数据 * @@ -41,12 +43,16 @@ export const processCitationData = (rawData) => { // 验证输入数据 if (!rawData || !rawData.data || !Array.isArray(rawData.data)) { - console.warn('citationUtils: Invalid data format, expected { data: [...] }'); + logger.warn('citationUtils', '无效的数据格式', { + hasRawData: !!rawData, + hasDataField: !!(rawData && rawData.data), + isArray: !!(rawData && rawData.data && Array.isArray(rawData.data)) + }); return null; } if (rawData.data.length === 0) { - console.warn('citationUtils: Empty data array'); + logger.warn('citationUtils', '数据数组为空'); return null; } @@ -57,7 +63,11 @@ export const processCitationData = (rawData) => { rawData.data.forEach((item, index) => { // 验证必需字段（至少需要 query_part 或 sentences 之一） if (!item.query_part && !item.sentences) { - console.warn(`citationUtils: Missing both 'query_part' and 'sentences' fields in item ${index}`); + logger.warn('citationUtils', '缺少必需字段', { + itemIndex: index, + hasQueryPart: !!item.query_part, + hasSentences: !!item.sentences + }); return; } @@ -83,7 +93,9 @@ export const processCitationData = (rawData) => { // 如果没有有效的片段，返回 null if (segments.length === 0) { - console.warn('citationUtils: No valid segments found'); + logger.warn('citationUtils', '没有找到有效的片段', { + totalItems: rawData.data.length + }); return null; } @@ -111,7 +123,10 @@ const formatDate = (dateStr) => { return `${year}-${month}-${day}`; } catch (e) { - console.warn('citationUtils: Date formatting error:', e); + logger.warn('citationUtils', '日期格式化错误', { + dateStr, + error: e.message + }); return '--'; } };