Skip to content

zoomAtメソッドのズーム計算ロジックの修正 #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kmryuk400144 wants to merge 1 commit into
base: develop-desktop
Choose a base branch
from
Open

zoomAtメソッドのズーム計算ロジックの修正 #75

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
85 changes: 63 additions & 22 deletions src/main/java/org/example/view/View.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
import javax.swing.event.ChangeListener;
import javax.swing.JLabel;
import java.util.Hashtable;
import java.text.DecimalFormat; // DecimalFormatをインポート
import java.text.DecimalFormat;

/**
* スピログラフアプリケーションのViewクラス。
Expand All @@ -52,8 +52,8 @@ public class View extends JPanel {

/** 拡大縮小率 */
private double scale = 1.0;
private static final double MIN_SCALE = 0.1; // 最小スケールをより小さく設定
private static final double MAX_SCALE = 5.0; // 最大スケールをより大きく設定
private static final double MIN_SCALE = 0.1;
private static final double MAX_SCALE = 5.0;

/** ビューのオフセット（パン用） */
private Point2D.Double viewOffset = new Point2D.Double(0, 0);
Expand Down Expand Up @@ -125,7 +125,7 @@ public View(Model model) {
}
};

// --- メインボタン群をJMenuItemとしてJPopupMenuに追加 ---
// メインボタン群をJMenuItemとしてJPopupMenuに追加
String[] mainButtonNames = { "Start", "Stop", "Clear", "Save", "Load" };
for (String name : mainButtonNames) {
JMenuItem item = new JMenuItem(name);
Expand All @@ -135,7 +135,7 @@ public View(Model model) {

MenuDisplay.addSeparator(); // 区切り線を追加

// --- ペンサイズをPenSizeサブメニューにまとめる ---
// ペンサイズをPenSizeサブメニューにまとめる
JMenu penSizeMenu = new JMenu("PenSize");
String[] penSizes = { "Small", "Medium", "Large" };
for (String size : penSizes) {
Expand All @@ -145,7 +145,7 @@ public View(Model model) {
}
MenuDisplay.add(penSizeMenu);

// --- カラーパレット表示メニュー項目を追加 ---
// カラーパレット表示メニュー項目を追加
JMenuItem chooseColorItem = new JMenuItem("色を選択...");
chooseColorItem.addActionListener(e -> {
Color newColor = JColorChooser.showDialog(View.this, "色を選択", model.getPenColor());
Expand All @@ -157,7 +157,7 @@ public View(Model model) {
});
MenuDisplay.add(chooseColorItem);

// --- スピード選択用JSliderを追加 ---
// スピード選択用JSliderを追加
MenuDisplay.addSeparator(); // 区切り線を追加

// スライダーとラベルを格納するためのパネル
Expand Down Expand Up @@ -239,7 +239,7 @@ protected void paintComponent(Graphics g) {
g2d.translate(viewOffset.x, viewOffset.y);
g2d.scale(scale, scale);

// --- 中心点の描画（r=2） ---
// 中心点の描画（r=2）
// スパーギア中心
Point2D.Double spurPosition = model.getSpurGearPosition();
if (spurPosition != null) {
Expand Down Expand Up @@ -472,7 +472,9 @@ public void changeSpurAndPinionRadius(double newSpurRadius) {
*/
public void zoomAt(Point screenPoint, double zoomFactor) {
// 現在のスクリーン座標での中心点をワールド座標に変換
Point2D.Double worldPointBeforeZoom = screenToWorld(screenPoint);
// viewOffsetとscaleを考慮して、現在の表示状態でのワールド座標を取得
double worldX = (screenPoint.getX() - viewOffset.x) / scale;
double worldY = (screenPoint.getY() - viewOffset.y) / scale;

// 新しいスケールを計算し、範囲内に収める
double newScale = scale * zoomFactor;
Expand All @@ -490,15 +492,12 @@ public void zoomAt(Point screenPoint, double zoomFactor) {
// 新しいスケールを設定
scale = newScale;

// ズーム後のスクリーン座標での中心点
Point2D.Double screenPointAfterZoom = new Point2D.Double(
worldPointBeforeZoom.x * scale + viewOffset.x,
worldPointBeforeZoom.y * scale + viewOffset.y
);

// ズーム後のオフセットを計算
viewOffset.x += (screenPoint.x - screenPointAfterZoom.x);
viewOffset.y += (screenPoint.y - screenPointAfterZoom.y);
// 新しいスケールとワールド座標のズーム中心点から、新しいオフセットを計算する
// (screenPoint.x - newViewOffsetX) / newScale = worldX
// screenPoint.x - newViewOffsetX = worldX * newScale
// newViewOffsetX = screenPoint.x - (worldX * newScale)
viewOffset.x = screenPoint.getX() - (worldX * newScale);
viewOffset.y = screenPoint.getY() - (worldY * newScale);

repaint();
}
Expand Down Expand Up @@ -676,7 +675,7 @@ public void setViewOffset(Point2D.Double offset) {
public void setLocusData(List<Point2D.Double> locus, Color penColor, double penSize) {
this.loadedLocusData = locus;
this.loadedPenColor = penColor;
this.loadedPenSize = penSize; // ここを修正：-1.0ではなくpenSizeをセット
this.loadedPenSize = penSize;
repaint();
}

Expand Down Expand Up @@ -738,11 +737,11 @@ public void updateCursor(Point mousePoint) {
Point2D pinionCenter = model.getPinionGearPosition();
double pinionRadius = model.getPinionGearRadius();

if (spurCenter != null && world.distance(spurCenter) < 10 / scale) { // スケールを考慮
if (spurCenter != null && world.distance(spurCenter) < 10 / scale) {
setCursor(Cursor.getPredefinedCursor(Cursor.MOVE_CURSOR));
} else if (spurCenter != null && Math.abs(world.distance(spurCenter) - spurRadius) < 10 / scale) { // スケールを考慮
} else if (spurCenter != null && Math.abs(world.distance(spurCenter) - spurRadius) < 10 / scale) {
setCursor(Cursor.getPredefinedCursor(Cursor.SE_RESIZE_CURSOR));
} else if (pinionCenter != null && world.distance(pinionCenter) < 10 / scale) { // スケールを考慮
} else if (pinionCenter != null && world.distance(pinionCenter) < 10 / scale) {
setCursor(Cursor.getPredefinedCursor(Cursor.HAND_CURSOR));
} else {
setCursor(Cursor.getDefaultCursor());
Expand All @@ -760,4 +759,46 @@ public void showPenTip() {
showPenTip = true;
repaint();
}

// --- テスト用のゲッターメソッド ---

/**
* 現在の保存メッセージを取得
* @return 保存メッセージ
*/
public String getSaveMessage() {
return saveMessage;
}

/**
* スパーギア定義中フラグの状態を取得
* @return スパーギア定義中であればtrue
*/
public boolean isDefiningSpurGear() {
return isDefiningSpurGear;
}

/**
* スパーギア中心（スクリーン座標）を取得
* @return スパーギア中心のスクリーン座標
*/
public Point getSpurGearCenterScreen() {
return spurGearCenterScreen;
}

/**
* スパーギアドラッグ点（スクリーン座標）を取得
* @return スパーギアドラッグ点のスクリーン座標
*/
public Point getCurrentDragPointScreen() {
return currentDragPointScreen;
}

/**
* ペン先表示フラグの状態を取得
* @return ペン先が表示されていればtrue
*/
public boolean isShowPenTip() {
return showPenTip;
}
}