feat: 日志优化

This commit is contained in:
zdl
2025-10-18 12:26:08 +08:00
parent 4ebb17190f
commit a7695c7365
7 changed files with 127 additions and 36 deletions

View File

@@ -50,6 +50,7 @@ import ProtectedRoute from "components/ProtectedRoute";
import ErrorBoundary from "components/ErrorBoundary"; import ErrorBoundary from "components/ErrorBoundary";
import AuthModalManager from "components/Auth/AuthModalManager"; import AuthModalManager from "components/Auth/AuthModalManager";
import ScrollToTop from "components/ScrollToTop"; import ScrollToTop from "components/ScrollToTop";
import { logger } from "utils/logger";
function AppContent() { function AppContent() {
const { colorMode } = useColorMode(); const { colorMode } = useColorMode();
@@ -172,13 +173,19 @@ export default function App() {
// 全局错误处理:捕获未处理的 Promise rejection // 全局错误处理:捕获未处理的 Promise rejection
useEffect(() => { useEffect(() => {
const handleUnhandledRejection = (event) => { 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(); event.preventDefault();
}; };
const handleError = (event) => { 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(); event.preventDefault();
}; };

View File

@@ -9,6 +9,7 @@ import {
VStack, VStack,
Container Container
} from '@chakra-ui/react'; } from '@chakra-ui/react';
import { logger } from '../utils/logger';
class ErrorBoundary extends React.Component { class ErrorBoundary extends React.Component {
constructor(props) { constructor(props) {
@@ -28,14 +29,20 @@ class ErrorBoundary extends React.Component {
componentDidCatch(error, errorInfo) { componentDidCatch(error, errorInfo) {
// 开发环境:打印错误到控制台,但不显示错误边界 // 开发环境:打印错误到控制台,但不显示错误边界
if (process.env.NODE_ENV === 'development') { if (process.env.NODE_ENV === 'development') {
console.error('🔴 ErrorBoundary 捕获到错误(开发模式,不拦截):'); logger.error('ErrorBoundary', 'componentDidCatch', error, {
console.error('错误:', error); componentStack: errorInfo.componentStack,
console.error('错误信息:', errorInfo); developmentMode: true
});
// 不更新 state让错误继续抛出 // 不更新 state让错误继续抛出
return; return;
} }
// 生产环境:保存错误信息到 state // 生产环境:保存错误信息到 state
logger.error('ErrorBoundary', 'componentDidCatch', error, {
componentStack: errorInfo.componentStack,
productionMode: true
});
this.setState({ this.setState({
error: error, error: error,
errorInfo: errorInfo errorInfo: errorInfo

View File

@@ -37,6 +37,7 @@ import { FiStar, FiCalendar } from 'react-icons/fi';
import { useNavigate, useLocation } from 'react-router-dom'; import { useNavigate, useLocation } from 'react-router-dom';
import { useAuth } from '../../contexts/AuthContext'; import { useAuth } from '../../contexts/AuthContext';
import { useAuthModal } from '../../contexts/AuthModalContext'; import { useAuthModal } from '../../contexts/AuthModalContext';
import { logger } from '../../utils/logger';
/** 二级导航栏组件 - 显示当前一级菜单下的所有二级菜单项 */ /** 二级导航栏组件 - 显示当前一级菜单下的所有二级菜单项 */
const SecondaryNav = () => { const SecondaryNav = () => {
@@ -366,11 +367,11 @@ export default function HomeNavbar() {
const toast = useToast(); const toast = useToast();
// 添加调试信息 // 添加调试信息
console.log('HomeNavbar Debug:', { logger.debug('HomeNavbar', '组件渲染状态', {
user, hasUser: !!user,
isAuthenticated, isAuthenticated,
isLoading, isLoading,
userKeys: user ? Object.keys(user) : 'no user' userId: user?.id
}); });
// 获取显示名称的函数 // 获取显示名称的函数
@@ -389,7 +390,9 @@ export default function HomeNavbar() {
setShowCompletenessAlert(false); setShowCompletenessAlert(false);
// logout函数已经包含了跳转逻辑这里不需要额外处理 // logout函数已经包含了跳转逻辑这里不需要额外处理
} catch (error) { } catch (error) {
console.error('Logout error:', error); logger.error('HomeNavbar', 'handleLogout', error, {
userId: user?.id
});
} }
}; };
@@ -434,7 +437,9 @@ export default function HomeNavbar() {
setWatchlistQuotes([]); setWatchlistQuotes([]);
} }
} catch (e) { } catch (e) {
console.warn('加载自选股实时行情失败:', e); logger.warn('HomeNavbar', '加载自选股实时行情失败', {
error: e.message
});
setWatchlistQuotes([]); setWatchlistQuotes([]);
} finally { } finally {
setWatchlistLoading(false); setWatchlistLoading(false);
@@ -482,7 +487,9 @@ export default function HomeNavbar() {
setFollowingEvents([]); setFollowingEvents([]);
} }
} catch (e) { } catch (e) {
console.warn('加载关注事件失败:', e); logger.warn('HomeNavbar', '加载关注事件失败', {
error: e.message
});
setFollowingEvents([]); setFollowingEvents([]);
} finally { } finally {
setEventsLoading(false); setEventsLoading(false);
@@ -550,12 +557,16 @@ export default function HomeNavbar() {
// 如果已经检查过,跳过(避免重复请求) // 如果已经检查过,跳过(避免重复请求)
if (hasCheckedCompleteness.current) { if (hasCheckedCompleteness.current) {
console.log('[Profile] 已检查过资料完整性,跳过重复请求'); logger.debug('HomeNavbar', '已检查过资料完整性,跳过重复请求', {
userId: user?.id
});
return; return;
} }
try { try {
console.log('[Profile] 开始检查资料完整性...'); logger.debug('HomeNavbar', '开始检查资料完整性', {
userId: user?.id
});
const base = getApiBase(); const base = getApiBase();
const resp = await fetch(base + '/api/account/profile-completeness', { const resp = await fetch(base + '/api/account/profile-completeness', {
credentials: 'include' credentials: 'include'
@@ -569,11 +580,17 @@ export default function HomeNavbar() {
setShowCompletenessAlert(data.data.needsAttention); setShowCompletenessAlert(data.data.needsAttention);
// 标记为已检查 // 标记为已检查
hasCheckedCompleteness.current = true; hasCheckedCompleteness.current = true;
console.log('[Profile] 资料完整性检查完成:', data.data); logger.debug('HomeNavbar', '资料完整性检查完成', {
userId: user?.id,
completeness: data.data.completenessPercentage
});
} }
} }
} catch (error) { } catch (error) {
console.warn('检查资料完整性失败:', error); logger.warn('HomeNavbar', '检查资料完整性失败', {
userId: user?.id,
error: error.message
});
} }
}, [isAuthenticated, user]); // 移除 getApiBase 依赖,因为它现在在组件外部 }, [isAuthenticated, user]); // 移除 getApiBase 依赖,因为它现在在组件外部

View File

@@ -5,6 +5,7 @@ import ReactECharts from 'echarts-for-react';
import * as echarts from 'echarts'; import * as echarts from 'echarts';
import moment from 'moment'; import moment from 'moment';
import { stockService } from '../../services/eventService'; import { stockService } from '../../services/eventService';
import { logger } from '../../utils/logger';
const StockChartModal = ({ const StockChartModal = ({
isOpen, isOpen,
@@ -36,14 +37,20 @@ const StockChartModal = ({
adjustedEventTime = nextDay.format('YYYY-MM-DD HH:mm'); adjustedEventTime = nextDay.format('YYYY-MM-DD HH:mm');
} }
} catch (e) { } catch (e) {
console.warn('事件时间解析失败:', e); logger.warn('StockChartModal', '事件时间解析失败', {
eventTime,
error: e.message
});
} }
} }
const response = await stockService.getKlineData(stock.stock_code, type, adjustedEventTime); const response = await stockService.getKlineData(stock.stock_code, type, adjustedEventTime);
setPreloadedData(prev => ({...prev, [type]: response})); setPreloadedData(prev => ({...prev, [type]: response}));
} catch (err) { } 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'); adjustedEventTime = nextDay.format('YYYY-MM-DD HH:mm');
} }
} catch (e) { } catch (e) {
console.warn('事件时间解析失败:', e); logger.warn('StockChartModal', '事件时间解析失败', {
eventTime,
error: e.message
});
} }
} }
@@ -113,7 +123,10 @@ const StockChartModal = ({
chartInstanceRef.current.setOption(option, true); chartInstanceRef.current.setOption(option, true);
} }
} catch (err) { } catch (err) {
console.error('加载图表数据失败:', err); logger.error('StockChartModal', 'loadChartData', err, {
stockCode: stock?.stock_code,
chartType: type
});
} finally { } finally {
setLoading(false); setLoading(false);
} }

View File

@@ -4,12 +4,17 @@
* 对应Flask后端的所有财务API接口 * 对应Flask后端的所有财务API接口
*/ */
import { logger } from '../utils/logger';
const isProduction = process.env.NODE_ENV === 'production'; const isProduction = process.env.NODE_ENV === 'production';
const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL; const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL;
const apiRequest = async (url, options = {}) => { const apiRequest = async (url, options = {}) => {
try { 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}`, { const response = await fetch(`${API_BASE_URL}${url}`, {
...options, ...options,
@@ -22,16 +27,24 @@ const apiRequest = async (url, options = {}) => {
if (!response.ok) { if (!response.ok) {
const errorText = await response.text(); 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}`); throw new Error(`HTTP error! status: ${response.status}`);
} }
const data = await response.json(); 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; return data;
} catch (error) { } catch (error) {
console.error(`Financial API request failed for ${url}:`, error); logger.error('financialService', 'apiRequest', error, { url });
throw error; throw error;
} }
}; };

View File

@@ -4,12 +4,17 @@
* 对应Flask后端的所有市场API接口 * 对应Flask后端的所有市场API接口
*/ */
import { logger } from '../utils/logger';
const isProduction = process.env.NODE_ENV === 'production'; const isProduction = process.env.NODE_ENV === 'production';
const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL; const API_BASE_URL = isProduction ? "" : process.env.REACT_APP_API_URL;
const apiRequest = async (url, options = {}) => { const apiRequest = async (url, options = {}) => {
try { 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}`, { const response = await fetch(`${API_BASE_URL}${url}`, {
...options, ...options,
@@ -22,16 +27,24 @@ const apiRequest = async (url, options = {}) => {
if (!response.ok) { if (!response.ok) {
const errorText = await response.text(); 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}`); throw new Error(`HTTP error! status: ${response.status}`);
} }
const data = await response.json(); 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; return data;
} catch (error) { } catch (error) {
console.error(`Market API request failed for ${url}:`, error); logger.error('marketService', 'apiRequest', error, { url });
throw error; throw error;
} }
}; };
@@ -449,7 +462,7 @@ export const marketCacheManager = {
if (cached) { if (cached) {
const { data, timestamp } = cached; const { data, timestamp } = cached;
if (Date.now() - timestamp < maxAge) { if (Date.now() - timestamp < maxAge) {
console.log(`Cache hit for key: ${key}`); logger.debug('marketCacheManager', '缓存命中', { key });
return data; return data;
} }
this.cache.delete(key); this.cache.delete(key);
@@ -467,15 +480,16 @@ export const marketCacheManager = {
data, data,
timestamp: Date.now() timestamp: Date.now()
}); });
console.log(`Data cached for key: ${key}`); logger.debug('marketCacheManager', '数据已缓存', { key });
}, },
/** /**
* 清除所有缓存 * 清除所有缓存
*/ */
clear: function() { clear: function() {
const cacheSize = this.cache.size;
this.cache.clear(); this.cache.clear();
console.log('All market cache cleared'); logger.debug('marketCacheManager', '清除所有缓存', { clearedCount: cacheSize });
}, },
/** /**
@@ -483,12 +497,17 @@ export const marketCacheManager = {
* @param {string} stockCode - 股票代码 * @param {string} stockCode - 股票代码
*/ */
clearStock: function(stockCode) { clearStock: function(stockCode) {
let clearedCount = 0;
for (const key of this.cache.keys()) { for (const key of this.cache.keys()) {
if (key.includes(`market_${stockCode}_`)) { if (key.includes(`market_${stockCode}_`)) {
this.cache.delete(key); this.cache.delete(key);
clearedCount++;
} }
} }
console.log(`Cache cleared for stock: ${stockCode}`); logger.debug('marketCacheManager', '清除股票缓存', {
stockCode,
clearedCount
});
} }
}; };

View File

@@ -1,6 +1,8 @@
// src/utils/citationUtils.js // src/utils/citationUtils.js
// 引用数据处理工具 // 引用数据处理工具
import { logger } from './logger';
/** /**
* 处理后端返回的引用数据 * 处理后端返回的引用数据
* *
@@ -41,12 +43,16 @@
export const processCitationData = (rawData) => { export const processCitationData = (rawData) => {
// 验证输入数据 // 验证输入数据
if (!rawData || !rawData.data || !Array.isArray(rawData.data)) { 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; return null;
} }
if (rawData.data.length === 0) { if (rawData.data.length === 0) {
console.warn('citationUtils: Empty data array'); logger.warn('citationUtils', '数据数组为空');
return null; return null;
} }
@@ -57,7 +63,11 @@ export const processCitationData = (rawData) => {
rawData.data.forEach((item, index) => { rawData.data.forEach((item, index) => {
// 验证必需字段(至少需要 query_part 或 sentences 之一) // 验证必需字段(至少需要 query_part 或 sentences 之一)
if (!item.query_part && !item.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; return;
} }
@@ -83,7 +93,9 @@ export const processCitationData = (rawData) => {
// 如果没有有效的片段,返回 null // 如果没有有效的片段,返回 null
if (segments.length === 0) { if (segments.length === 0) {
console.warn('citationUtils: No valid segments found'); logger.warn('citationUtils', '没有找到有效的片段', {
totalItems: rawData.data.length
});
return null; return null;
} }
@@ -111,7 +123,10 @@ const formatDate = (dateStr) => {
return `${year}-${month}-${day}`; return `${year}-${month}-${day}`;
} catch (e) { } catch (e) {
console.warn('citationUtils: Date formatting error:', e); logger.warn('citationUtils', '日期格式化错误', {
dateStr,
error: e.message
});
return '--'; return '--';
} }
}; };