boostcampwm-2022 / web18-PRV

논문 인용관계를 밤하늘의 별 처럼 표시해보자 🌟-🌟
https://paperef.com
143 stars 3 forks source link

10-3 [FE][fix] 논문 Title 최대 글자수 설정, 검색결과 Articles 숫자 세자리당 콤마 설정 #65

Closed Palwol closed 1 year ago

Palwol commented 1 year ago

개요

자동완성 시 논문 제목이 비정상적으로 길면 키워드 highlight 기능에 오류가 발생해서 브라우저가 멈춥니다. (정규표현식 과부하로 추정) 제목이 비정상적으로 긴 경우 유저 사용성도 해칠 수 있기 때문에 제목이 일정 글자수를 넘어가는 경우 예외 처리를 해주었습니다.

검색결과 페이지에서도 키워드 highlight 기능이 사용되기 때문에 동일하게 처리 해주었습니다.

작업사항

리뷰 요청사항

Palwol commented 1 year ago

64

Palwol commented 1 year ago

너무 긴 제목들이나 string들의 상한선을 두는 건 좋지만, 추후에 긴 제목의 전체 부분을 볼 수 있게 (커서를 올리면 모든 내용이 보인다거나) 원본을 가지고 있는 방향도 좋다고 생각합니다

서버에서 받아온 원본 데이터 자체는 수정하지 않고 타이틀을 보여줄 때 글자수를 자르고 하이라이트를 적용하기 때문에, 추후에 원본 데이터가 필요하다면 사용할 수 있을 것 같습니다!

yeynii commented 1 year ago

덧붙이자면 문자열 비교 알고리즘(kmp) 으로 하이라이팅할 문자열을 탐색하면 정규식보다 훨씬 빠르게 동작할듯 하네요. (나중에 시간이 남는다면 도전..?) 현재는 N(rows x maxlength x split된 단어수 x 단어길이) 만큼의 시간복잡도가 나올 것 같아요.