C# と VB.NET の質問掲示板

■No39666 (.SHO さん) に返信
> ■No39664 (へっぽこ さん) に返信
> 
>>たいへんよくわかりました。(^-^)これでは左端にはすすみませんね。
>>う〜ん。乏しい脳みそでもう一度考えてみますm(_ _)m
> 
> だっから、最初からこれを確認しろって言ってるのに…orz

すみません。
解決法は'idouryou = rn.Next(10) の部分の数字を(200,400)くらいにしないと左端へは進まないことが

わかりました。でもそうすると点滅がはやくなってしまうようです。（一種のモグラたたきのゲームがつくれそうですが(T_T)
結局は、おかしなものをつくろうとしていたのかな〜^^;

> 解決法は'idouryou = rn.Next(10) の部分の数字を(200,400)くらいにしないと左端へは進まないことが

If idouryou > 0 Then
  idouryou = rn.Next(10)
Else
  idouryou = -rn.Next(10)
EndIf

ってやったらどうなります?

> だっから、PictureBox1.Left の値がどう変化するかを確認すればいい。

だよねぇ。
ひょっとして、「ブレークポイント」を知らなかったりするのかなぁ?

> If PictureBox1.Left > Me.Width - PictureBox1.Width Or PictureBox1.Left < 0 Then
>     idouryou *= -1 '上記の条件のときの移動方向を反転
>
> End If
>
> PictureBox1.Left += idouryou '←この行にブレークポイントを置いて、
        'idouryou の値が想定どおりになっているか調べてみるべし。 > へっぽこ さん
※ …って、このレスを書いてるあいだに、みきぬ さんが答えてくださっちゃいましたね f(^^;

# あと、初心者はこういう混乱によく陥るので、メンバーの参照には this / Me を付けろ、
# メンバー変数の名前は区別しやすい命名 (アタマにアンダースコア付けるとか) をしろ、
# って言うわけですよ f(^^;
# ※ 「★…★の部分を挿入すると」って話から、そう思える。


> なのでbiacさんは、「移動量と方向」の2つに分けた。移動量を変えても、方向は変わらないからね。

ですます。
2次元空間で移動させるとなれば、ベクトルを扱うことになるわけだけど。
それを、2つの成分を持つものと見るか、大きさと方向を持つものと見るか…?
乱数でベクトルの大きさを設定すると決めた瞬間に、後者で行くぞという暗黙の前提が。 f(^^;



さて。
どうしても大きさと方向を idouryou という変数ひとつでいっぺんに扱いたいらしいので、
コメントに突っ込みを。(追記分を 【】 で。 <del>〜</del> は削除。)

■No39648 (へっぽこ さん) に返信

> idouryou = rn.Next(10) '【右向きの】移動量を0〜9で発生★
    '【この場所では、idouryou は 0〜9 のいずれかになっている。つまり必ず右向き。】
>
> If PictureBox1.Left > Me.Width - PictureBox1.Width Or PictureBox1.Left < 0 Then
      '【この場所でも、idouryou は 0〜9 のいずれかのままである。つまり必ず右向き。】
>     idouryou *= -1 '上記の条件のときの移動方向を<del>反転</del>【左向きに変える】
>
      '【この場所では、idouryou は左向き。】
> End If
>
  '【1】
> PictureBox1.Left += idouryou

さらに。
1. 「上記の条件」というのを日本語で説明してください。
2. 「上記の条件」のとき、idouryou は左向きになるわけですが、それでよいですか?
3. 「上記の条件」以外のときは、【1】での idouryou は右向きになりますか、左向きになりますか?
   また、それでよいですか?


> 移動量がランダムに変化することとの関係がいまいち理解できません。(-_-;)

そういうときは、問題をシンプルにしましょう。
いま直面している問題に、「移動量がランダムに変化すること」が関係しているかどうか…?
次のようにコードを (仮に) 変更してみれば、関係しているかどうか分かるのではないかな?

> Private Sub Timer1_Tick(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Timer1.Tick
>
>     PictureBox1.Top = rn.Next(400) 'Y座標を0〜399で発生
>     '★ 'idouryou = rn.Next(10) '移動量を0〜9で発生★
      idouryou = 5 '移動量を固定値5で発生 (「移動量がランダムに変化すること」を排除する)
>
>     If PictureBox1.Left > Me.Width - PictureBox1.Width Or PictureBox1.Left < 0 Then
>         idouryou *= -1 '上記の条件のときの移動方向を反転
>
>     End If
>
>     PictureBox1.Left += idouryou
>
> End Sub


それと。
行き詰ったときは、コードを捨てる勇気を持ちましょう。
(実際には、捨てる前にバックアップコピーを取っておくものですけれど。)

今回ですと、 Timer1_Tick() メソッドの中身全部と、 Timer1_Tick() メソッドだけで使っている
変数 (idouryou) を削除してしまって、心機一転ゼロから書き直したほうが良いです。
そうしないと、いつまでも既存の「おかしな」コードに引き摺られてしまいますよ。

■No39670 (biac さん) に返信
> 1. 「上記の条件」というのを日本語で説明してください。

 PictureBox1.Left が右端を越えそうなとき、あるいは左端を越えそうなとき

 2. 「上記の条件」のとき、idouryou は左向きになるわけですが、それでよいですか?

いえ、前者のときは、左向き、後者のときは右向きになってほしいです

3. 「上記の条件」以外のときは、【1】での idouryou は右向きになりますか、左向きになりますか?
    右向き

　＞また、それでよいですか?
　いえ

■No39669 (επιστημη さん) に返信
> If idouryou > 0 Then
>   idouryou = rn.Next(10)
> Else
>   idouryou = -rn.Next(10)
> EndIf

弘法も筆の誤り、επιさんでもバグることはある。
# 承知の上、かな。 ( Next(9) で 1つゲタ履かせる? ) f(^^;

これは、想定されているのとはわずかに違う動きをすることがあると思いますが、
試してみてくださいね > へっぽこ さん

で、いろいろ試して、スマートに直せればよし。
やっぱりグチャグチャになって行き詰っちゃったら、また全部捨ててゼロスタート。

■No39669 (επιστημη さん) に返信
> If idouryou > 0 Then
>   idouryou = rn.Next(10)
> Else
>   idouryou = -rn.Next(10)
> EndIf
> 
> ってやったらどうなります?
> 

アドバイスに従ってやってみました（ちがうかもしれませんが）
＜実行結果＞
右端にいきたくてもスムーズにいけず、すこし移動しては戻りを何度も繰り返してジタバタしています。

Private Sub Timer1_Tick(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Timer1.Tick


        PictureBox1.Top = rn.Next(400) 'Y座標を0〜399で発生
        
        If PictureBox1.Left > Me.Width - PictureBox1.Width Or PictureBox1.Left < 0 Then


            idouryou *= -1 '上記の条件のときの移動方向を反転
        Else
            idouryou *= 1

            If idouryou > 0 Then
                idouryou = rn.Next(10)
            Else
                idouryou = -rn.Next(10)
            End If
        End If

        PictureBox1.Left += idouryou


    End Sub

■No39686 (やじゅ さん) に返信
>>

> 
> Visual Studio 2008 開発ガイドブック  第 6 章 デバッグ
> http://www.microsoft.com/japan/msdn/vstudio/2008/tech/howto/VSGuide/chapter06.aspx
> 
> 特定の条件のときにデバッグをブレーク（停止）させるには？［C#、VB］
> http://www.atmarkit.co.jp/fdotnet/dotnettips/688debugbreakcond/debugbreakcond.html

やじゅさん、ありがとうございます。(^_^)
しっかり学びたいと思います。

> ほっほう。何ででしょうねぇ。
> 
> コードの意味をよく考えるか、見える化をして idouryou が予期せずマイナスになるところの法則を見れば分かるかもしれませんねぇ。

さらに。
επιστημη さんが No39669 で提案してくれたのとは、違う場所に επιστημη さんのコードを突っ込んでるし〜

No39669 をよく読んでください。 επιστημη さんが 「> 解決法は…」 という部分をわざわざ引用しているのは、なぜでしょう?

■No39707 (επιστημη さん) に返信
> あー...このコード片の"意図"が伝わらなかったか orz