쿼리 편집기 — 자동완성 위치 버그와 목록 가독성
■ 팝업이 화면 구석으로 날아가던 문제
두 가지가 겹쳐 있었다.
1. WPF Popup 은 열려 있는 동안 Placement 를 다시 계산하지 않는다. 글자를 칠 때마다
HorizontalOffset/VerticalOffset 만 바꿔서는 처음 열린 자리에 그대로 머문다.
2. GetRectFromCharacterIndex 는 스크롤 밖이나 범위를 벗어난 위치에서 Empty 를 준다.
그대로 쓰면 좌표가 무한대가 되어 팝업이 화면 밖으로 나간다.
기준점을 커서가 아니라 <b>완성 중인 낱말의 시작</b>으로 바꾸고(글자마다 흔들리지 않는다),
그 기준점이 달라졌을 때만 닫았다 다시 연다. Empty·무한대는 캐럿 → 원점 순으로 물러선다.
■ 별칭 없이 컬럼을 칠 때 아무것도 안 뜨던 문제
select * FROM P_COMINF where comcht|
점을 찍어야만 컬럼을 제안하고 있었다. 그런데 별칭을 안 쓰는 쪽이 오히려 흔하고,
WHERE 절이야말로 컬럼 이름이 가장 많이 필요한 자리다.
이제 일반 위치에서도 이 쿼리가 참조하는 테이블(FROM/JOIN)의 컬럼을 먼저 제안한다.
FROM 뒤에 키워드가 오면 테이블로 보지 않고, 같은 테이블이 두 번 조인돼도 후보는 한 번만 만든다.
■ 치환 변수 목록 가독성
토큰이 60자가 넘고 앞 40자(M.CMM.HISOperatingInfo.bzPatientInfo.)가 모든 줄에서 똑같았다.
그대로 두 줄로 깔면 정작 다른 부분인 속성명이 오른쪽 끝에서 잘려 무엇이 무엇인지 구분되지 않는다.
설명 + 속성명 한 줄로 바꾸고 전체 토큰은 툴팁으로 옮겼다. 줄 수도 절반이 됐다.
DataRow 형태는 꼬리(.item("컬럼명"))를 빼고 속성명만 남긴다 — 컬럼 채우기라는 사실은 설명이 말해 준다.
■ 현재 줄 강조
긴 쿼리에서 지금 어디를 고치고 있는지 잃지 않도록 커서 줄에 옅은 띠를 깐다.
선택 중일 때는 끈다 — 선택 색과 겹치면 오히려 읽기 어렵다.
단위 테스트 6건 추가(별칭 없는 테이블 참조·중복 조인·FROM 뒤 키워드 배제·
짧은 형태에 클래스 이름 없음·길이 상한·DataRow 속성명).
회귀: 테스트 216/216, 편집 스모크 실패 0, 검증 실행 점검 10/10,
DB 왕복 1,271건 diff 0/예외 0, 종이 렌더 P062 바이트 동일.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
79db31cb4d
commit
f7c90979b9
@@ -112,11 +112,23 @@ public partial class QueryEditorWindow : Window
|
||||
return;
|
||||
}
|
||||
|
||||
// 팝업은 '완성 중인 낱말의 시작'에 붙인다 — 커서를 따라다니면 글자마다 흔들린다.
|
||||
// WPF Popup 은 열려 있는 동안 Placement 를 다시 계산하지 않으므로,
|
||||
// 기준점이 바뀐 경우에만 닫았다 다시 연다.
|
||||
var anchorChanged = completionStart != query.ReplaceStart;
|
||||
completionStart = query.ReplaceStart;
|
||||
CompletionList.ItemsSource = items;
|
||||
CompletionList.SelectedIndex = 0;
|
||||
PlaceCompletionPopup();
|
||||
CompletionPopup.IsOpen = true;
|
||||
|
||||
if (anchorChanged && CompletionPopup.IsOpen)
|
||||
{
|
||||
CompletionPopup.IsOpen = false;
|
||||
}
|
||||
if (!CompletionPopup.IsOpen)
|
||||
{
|
||||
PlaceCompletionPopup();
|
||||
CompletionPopup.IsOpen = true;
|
||||
}
|
||||
}
|
||||
|
||||
private IEnumerable<SqlCompletionItem> CandidatesFor(SqlCompletionQuery query)
|
||||
@@ -136,7 +148,12 @@ public partial class QueryEditorWindow : Window
|
||||
return Columns(table);
|
||||
|
||||
default:
|
||||
return SqlCompletion.BuiltIns().Concat(Tables());
|
||||
// 이 쿼리가 참조하는 테이블의 컬럼을 먼저 준다 — 별칭을 안 쓰는 쪽이 오히려 흔하고,
|
||||
// 점을 찍어야만 컬럼이 뜨면 정작 필요한 자리(WHERE 절)에서 아무것도 안 나온다.
|
||||
var referenced = SqlCompletion.ReferencedTables(SqlBox.Text)
|
||||
.SelectMany(Columns)
|
||||
.ToList();
|
||||
return referenced.Concat(SqlCompletion.BuiltIns()).Concat(Tables());
|
||||
}
|
||||
}
|
||||
|
||||
@@ -169,13 +186,28 @@ public partial class QueryEditorWindow : Window
|
||||
return items;
|
||||
}
|
||||
|
||||
/// <summary>캐럿 바로 아래에 목록을 놓는다</summary>
|
||||
/// <summary>
|
||||
/// 완성 중인 낱말 바로 아래에 목록을 놓는다.
|
||||
///
|
||||
/// GetRectFromCharacterIndex 는 스크롤·범위를 벗어나면 Empty 를 준다 —
|
||||
/// 그대로 쓰면 좌표가 무한대가 되어 팝업이 화면 구석으로 날아간다. 그 경우 캐럿으로 물러선다.
|
||||
/// </summary>
|
||||
private void PlaceCompletionPopup()
|
||||
{
|
||||
var rect = SqlBox.GetRectFromCharacterIndex(SqlBox.CaretIndex);
|
||||
var index = Math.Clamp(completionStart, 0, SqlBox.Text.Length);
|
||||
var rect = SqlBox.GetRectFromCharacterIndex(index);
|
||||
if (rect.IsEmpty || double.IsInfinity(rect.Bottom) || double.IsNaN(rect.Bottom))
|
||||
{
|
||||
rect = SqlBox.GetRectFromCharacterIndex(SqlBox.CaretIndex);
|
||||
}
|
||||
if (rect.IsEmpty || double.IsInfinity(rect.Bottom) || double.IsNaN(rect.Bottom))
|
||||
{
|
||||
rect = new Rect(0, 0, 0, SqlBox.FontSize * 1.4);
|
||||
}
|
||||
CompletionPopup.PlacementTarget = SqlBox;
|
||||
CompletionPopup.HorizontalOffset = rect.X;
|
||||
CompletionPopup.VerticalOffset = rect.Bottom + 2;
|
||||
CompletionPopup.Placement = System.Windows.Controls.Primitives.PlacementMode.Relative;
|
||||
CompletionPopup.HorizontalOffset = Math.Max(0, rect.X);
|
||||
CompletionPopup.VerticalOffset = Math.Max(0, rect.Bottom) + 2;
|
||||
}
|
||||
|
||||
/// <summary>고른 후보를 커서 앞 낱말과 바꿔 넣는다</summary>
|
||||
|
||||
Reference in New Issue
Block a user