Canvas. Прямоугольник рисуется не там где надо.

[Andrej_F]

Здравствуйте!

В меню первой формы есть кнопка "Открыть" с помощью которой мы выбираем файл изображения, после этого открывается вторая форма, в которой находится Image, отображающий выбранное изображение. Я хочу на этом изображении выделить мышкой прямоугольник, после чего этот прямоугольник нарисовался бы на изображении.

Текст первой формы:

unit images;

interface

uses
  Winapi.Windows, Winapi.Messages, System.SysUtils, System.Variants, System.Classes, Vcl.Graphics,
  Vcl.Controls, Vcl.Forms, Vcl.Dialogs, Vcl.Menus, pngimage, JPEG, Vcl.StdCtrls, images2;

type
  TForm3 = class(TForm)
    MainMenu1: TMainMenu;
    N1: TMenuItem;
    N2: TMenuItem;
    N3: TMenuItem;
    N4: TMenuItem;
    N5: TMenuItem;
    N6: TMenuItem;
    N7: TMenuItem;
    N8: TMenuItem;
    N9: TMenuItem;
    N10: TMenuItem;
    OpenDialog1: TOpenDialog;
    N11: TMenuItem;
    procedure N4Click(Sender: TObject);
    procedure N7Click(Sender: TObject);
  private
    { Private declarations }
  public
    { Public declarations }
  end;

var
  Form3: TForm3;
  height_image, width_image : word;

implementation

{$R *.dfm}

procedure TForm3.N4Click(Sender: TObject);
var
filename : Tfilename;
begin
OpenDialog1.Execute();
filename := OpenDialog1.FileName;
if (filename='') then else begin
  Form4.Show;
  Form4.Image1.Picture.LoadFromFile(filename);
  height_image := Form4.Image1.Picture.Height;
  width_image := Form4.Image1.Picture.Width;
  Form4.Image1.AutoSize := False;
  Form4.Image1.Height := 768;
  Form4.Image1.Width := Round((width_image*Form4.Image1.Height)/height_image);
  Form4.Image1.Stretch := True;
  height_koeff := Round(height_image/Form4.Image1.Height);
  width_koeff := Round(width_image/Form4.Image1.Width);
end;
end;

procedure TForm3.N7Click(Sender: TObject);
begin
Halt;
end;

end.

Вторая форма:

unit images2;

interface

uses
  Winapi.Windows, Winapi.Messages, System.SysUtils, System.Variants, System.Classes, Vcl.Graphics,
  Vcl.Controls, Vcl.Forms, Vcl.Dialogs, Vcl.ExtCtrls, Vcl.StdCtrls, Vcl.Menus;

type
  TForm4 = class(TForm)
    Image1: TImage;
    MainMenu1: TMainMenu;
    Ht1: TMenuItem;
    N1: TMenuItem;
    N2: TMenuItem;
    N3: TMenuItem;
    N4: TMenuItem;
    procedure N2Click(Sender: TObject);
    procedure Image1MouseUp(Sender: TObject; Button: TMouseButton;
      Shift: TShiftState; X, Y: Integer);
    procedure Image1MouseDown(Sender: TObject; Button: TMouseButton;
      Shift: TShiftState; X, Y: Integer);
  private
    { Private declarations }
  public
    { Public declarations }
  end;

var
  Form4: TForm4;
  x1, y1, x2, y2 : integer;
  Mouse : TMouse;
  Flag_mouseup, Flag_mousedown : Boolean;
  height_koeff, width_koeff : word;

implementation


{$R *.dfm}

procedure TForm4.Image1MouseDown(Sender: TObject; Button: TMouseButton;
  Shift: TShiftState; X, Y: Integer);
begin
  if (Flag_mousedown = True) then begin
    x1 := Mouse.CursorPos.X;
    y1 := Mouse.CursorPos.Y;
    Flag_mouseup := True;
    Flag_mousedown := False;
  end;
end;

procedure TForm4.Image1MouseUp(Sender: TObject; Button: TMouseButton;
  Shift: TShiftState; X, Y: Integer);
begin
if (Flag_mouseup = True) then begin
  x2 := Mouse.CursorPos.X;
  y2 := Mouse.CursorPos.Y;
  Flag_mouseup := False;
  Screen.Cursor := crDefault;
  //Canvas.Pen.Color := clGreen;
  //Canvas.Pen.Style := psSolid;
  //Canvas.Pen.Width := 2;
  Image1.Canvas.Rectangle(x1,y1,x2,y2);
end;
end;

procedure TForm4.N2Click(Sender: TObject);
begin
Screen.Cursor := crCross;
Flag_mousedown := True;
end;

end.

Т.е. идея такая: по нажатию кнопки мыши выбирать x1 и y1, а по отжатию кнопки выбирать x2 и y2.

Почему-то прямоугольник рисуется не там где нужно((( Подскажите пожалуйста что делаю не так?

[indexx]

Проект скинь. Нет желания копировать всё это.

[Foreseen]

Подскажите пожалуйста что делаю не так?

Ошибка у вас здесь

procedure TForm4.Image1MouseDown(Sender: TObject; Button: TMouseButton;
  Shift: TShiftState; X, Y: Integer);
begin
  if (Flag_mousedown = True) then begin
    x1 := Mouse.CursorPos.X;
    y1 := Mouse.CursorPos.Y;
    Flag_mouseup := True;
    Flag_mousedown := False;
  end;
end;

Вы получаете координаты указателя относительно экрана, а не относительно Image1. Для проверки посмотрите, какие значения у вас хранятся в x1, y1 после нажатия мыши. Аналогично и с процедурой Image1MouseUp.

[min@y™]

Выводи координаты курсора куда-нить в обработчике OnMouseMove.

З.Ы. Как вы все заколебали рисовать на TImage! Сколько лет борюсь с этим триппером в интернете - всё бестолку!

[Yelenka]

Сообщение от min@y™

Выводи координаты курсора куда-нить в обработчике OnMouseMove.

З.Ы. Как вы все заколебали рисовать на TImage! Сколько лет борюсь с этим триппером в интернете - всё бестолку!

А с помощью какого элемента вы посоветуете рисовать?

[Andrej_F]

Foreseen, Спасибо!

Также, так как я сжал размер изображения под высоту 768 пикселей, пришлось считать коэффициент пропорциональности реального размера изображения к отображаемому размеру

height_image := Form4.Image1.Picture.Height;
width_image := Form4.Image1.Picture.Width;
Form4.Image1.Height := 768;
koeff := height_image/Form4.Image1.Height;
Form4.Image1.Width := Round(width_image/koeff);

и пересчитывать координаты:

x1 := Round(X*koeff);
y1 := Round(Y*koeff);

[min@y™]

А с помощью какого элемента вы посоветуете рисовать?

TPaintBox, например.

[Foreseen]

На здоровье

[Andrej_F]

Сообщение от min@y™

TPaintBox, например.

дело в том, что мне нужно на выбираемом изображении указывать определенную область, поэтому я рисую на TImage.

[min@y™]

дело в том, что мне нужно на выбираемом изображении указывать определенную область, поэтому я рисую на TImage.

Не вижу связи.