Skip to content

Commit

Permalink
[Add] 매수/매도호가 관련 더미 데이터 추가로직 주석 처리
Browse files Browse the repository at this point in the history
- BE 에서 매수/매도호가 관련 데이터 추가하기로 결정하여 기존에 활용하던 더미 데이터 추가 로직 주석 처리 (서버 구현 오류 가능성 감안하여 삭제하지 않고 주석처리 함)
- 이외 변경사항 : 입력가 거래가능 체크 관련 UseEffect 훅의 의존성 배열에 조건 추가 (orderType 상태, 매수/매도 전환 시 현재 설정한 가격으로 거래가능 체크 발생하도록)

Issues #17
  • Loading branch information
novice1993 committed Sep 12, 2023
1 parent c40696f commit d980f1d
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 11 deletions.
4 changes: 2 additions & 2 deletions client/src/components/StockOrderSection/PriceSetting.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ const PriceSetting = (props: OwnProps) => {

useEffect(() => {
handleCheckTradePossibility();
}, [orderPrice]);
}, [orderPrice, orderType]);

// 가격 설정란에서 포커스 제거 -> 안내 메세지 제거
const handleRemoveNoVolumeNotification = () => {
Expand Down Expand Up @@ -248,5 +248,5 @@ const UnitContent = styled.div`
const NoTradingVolume = styled.div`
position: absolute;
top: 222px;
right: 10px;
right: 4%;
`;
19 changes: 10 additions & 9 deletions client/src/components/StockOrderSection/StockPriceList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -44,22 +44,23 @@ const StockPriceList = () => {
}

/*
🔴 삭제 예정인 코드
[문제점] 주가 리스트 개수가 너무 적음 (매도호가 5개 + 매수호가 5개 = 총 10개) → UX를 저해하는 요소로 판단되어, 더미데이터를 추가 (매도/매수 각각 5개씩)
[해결방안] 1) fetching 해온 데이터 중 가격 0인 데이터 제외 (한국투자증권 API에서 간혹 보내는 경우 있음) → 호가 간격 계산 후, 더미 데이터 추가 (거래량은 0으로 설정)
*/
const existSellingPrice = sellingPrice.filter((selling) => selling.price !== 0);
const existBuyingPrice = buyingPrice.filter((buyingPrice) => buyingPrice.price !== 0);
const priceInterval: number = existSellingPrice[existSellingPrice.length - 1].price - existBuyingPrice[0].price;
// const priceInterval: number = existSellingPrice[existSellingPrice.length - 1].price - existBuyingPrice[0].price;

for (let i = 0; existSellingPrice.length < 10; i++) {
const dummySellingData = { price: existSellingPrice[0].price + priceInterval, volume: 0 };
existSellingPrice.unshift(dummySellingData);
}
// for (let i = 0; existSellingPrice.length < 10; i++) {
// const dummySellingData = { price: existSellingPrice[0].price + priceInterval, volume: 0 };
// existSellingPrice.unshift(dummySellingData);
// }

for (let i = 0; existBuyingPrice.length < 10; i++) {
const dummyBuyingData = { price: existBuyingPrice[existBuyingPrice.length - 1].price - priceInterval, volume: 0 };
existBuyingPrice.push(dummyBuyingData);
}
// for (let i = 0; existBuyingPrice.length < 10; i++) {
// const dummyBuyingData = { price: existBuyingPrice[existBuyingPrice.length - 1].price - priceInterval, volume: 0 };
// existBuyingPrice.push(dummyBuyingData);
// }

// 1) 매도/매수호가 종합 2) 매수/매도호가 거래량 종합
const sellingAndBuyingPrice = [...existSellingPrice, ...existBuyingPrice];
Expand Down

0 comments on commit d980f1d

Please sign in to comment.