質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Windows

Windowsは、マイクロソフト社が開発したオペレーティングシステムです。当初は、MS-DOSに変わるOSとして開発されました。 GUIを採用し、主にインテル系のCPUを搭載したコンピューターで動作します。Windows系OSのシェアは、90%を超えるといわれています。 パソコン用以外に、POSシステムやスマートフォンなどの携帯端末用、サーバ用のOSもあります。

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

button

HTMLで用いる<button>タグです。

Q&A

2回答

3152閲覧

C#のForm上のボタンの移動について

Shiro_neko

総合スコア8

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Windows

Windowsは、マイクロソフト社が開発したオペレーティングシステムです。当初は、MS-DOSに変わるOSとして開発されました。 GUIを採用し、主にインテル系のCPUを搭載したコンピューターで動作します。Windows系OSのシェアは、90%を超えるといわれています。 パソコン用以外に、POSシステムやスマートフォンなどの携帯端末用、サーバ用のOSもあります。

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

button

HTMLで用いる<button>タグです。

0グッド

0クリップ

投稿2016/07/08 14:34

編集2022/01/12 10:55

###前提・実現したいこと
C#にてスライドパズルを作成するため、Windows Formの上に画像を張り付けたボタンを置き、クリックイベントで並びを変えるプログラムを作成しています

###発生している問題・エラーメッセージ

現在、同じボタンを二回押すと配列内では並び替えがうまくいくんですが、Form上のボタンが移動しなかったり 一つ隣のボタンが移動してしまうなど意図しない挙動が起きてしまいます。

###該当のソースコード

static void bt1_Click(object sender, EventArgs e) { //押下したボタン(画像)の座標(左上基準) btx = Cursor.Position.X - mf.Location.X; if (btx < 105) { btx = 0; } else if (btx < 205) { btx = 100; } else if (btx < 305) { btx = 200; } else if (btx < 405) { btx = 300; } bty = Cursor.Position.Y - mf.Location.Y; if (bty < 120) { bty = 0; } else if (bty < 220) { bty = 100; } else if (bty < 320) { bty = 200; } else if (bty < 420) { bty = 300; } lx = btx / 100; ly = bty / 100; for (int i = 0; i < 4; i++) { for (int j = 0; j < 4; j++) { //空白ピースの座標(左上基準) if (hit[i, j] == -1) { x = j * 100; y = i * 100; } } } //ボタン(画像)の移動 if (bty == y) { if ((btx - x) == 100) { bt1[ly, lx].Left = btx - 100; //ボタン(画像)の移動 tmp = hit[(bty / 100), (btx / 100)]; hit[(bty / 100), (btx / 100)] = -1; //空白の移動(配列内) hit[(y / 100), (x / 100)] = tmp; } else if((x - btx) == 100) { bt1[ly, lx].Left = btx + 100; //ボタン(画像)の移動 tmp = hit[(bty / 100), (btx / 100)]; hit[(bty / 100), (btx / 100)] = -1; //空白の移動(配列内) hit[(y / 100), (x / 100)] = tmp; } } else if (btx == x) { if ((bty - y) == 100) { bt1[ly, lx].Top = bty - 100; //ボタン(画像)の移動 tmp = hit[(bty / 100), (btx / 100)]; hit[(bty / 100), (btx / 100)] = -1; //空白の移動(配列内) hit[(y / 100), (x / 100)] = tmp; } else if((y - bty) == 100) { bt1[ly, lx].Top = bty + 100; //ボタン(画像)の移動 tmp = hit[(bty / 100), (btx / 100)]; hit[(bty / 100), (btx / 100)] = -1; //空白の移動(配列内) hit[(y / 100), (x / 100)] = tmp; } } mf.Visible = true; //画面(フォーム)の更新 }

###補足情報(言語/FW/ツール等のバージョンなど)
言語はC#、ツールはVisualStudio2015を使っています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kgtkr

2016/07/08 14:50

ソースは重要な部分だけ抜き出して、整形機能を使った方がいいと思います。
guest

回答2

0

配列では移動ができているということなので、意図した動作にななっているのでしょう。
内部処理を行い、結果をUIに表示する。という基本的なスタイル(Document & view わりと古い基本概念なので、検索してみてください)ができていれば、ちゃんと表示されるとおもいます。

あと、同じようなコードも多いのでまとめられる部分はきちんと共通化してスッキリさせましょう。

投稿2016/07/10 20:09

mugicya

総合スコア1046

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

どのボタンが押されたかの認識がうまくいってなさそうな..
イベントが発生した時点で、クリックされた座標と認識されたボタン番号(Index)を表示させるようにしてみては?

あとこの辺りを参考に基本的なロジックを押さえた方がよいと思いますよ
Javaから楽しく学ぶ!ゲームプログラミング専門学校 > 15パズル制作

投稿2016/07/08 19:53

dojikko

総合スコア3939

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問