このメールは、OSASK伝言板に書き込まれた内容です。 この書き込みに返事を書く場合は、下のURLから書き込みを行なって下さい http://www.imasy.org/~mone/osask/index.cgi?REFER=3dfd9fe7_bdef 2002/12/16 18:41 川合秀実 [OSASK 5462]へのレスです。 だれもコメントしないようなので、僕がコメントします。 ざっとみたところ、前よりもずっとすっきりしてよくなった感じがします。く ーみんさんのアドバイスと、あっきぃさんの頑張りのおかげでしょう。 少し気がついたことを。 > do{ > col = rand() % 16; > if (col == 7){break;} > if (col <= 2){break;} > if (col >= 3){continue;} > }while (col <= 8); これは何をやろうとしているのかなと思ったら、つまり、色の番号として、 0, 1, 2, 7, 9, 10, 11, 12, 13, 14, 15 を出そうとしているわけですね。それなら、まあいろいろと方法がありますが、 たとえばこんなのはどうでしょうか？ /* 0, 1, 2, 7, 9, 10, 11, 12, 13, 14, 15 を出す */ /* (0 1 2 3 4 5 6 7 8 9 10) */ col = rand() % 11; if (col >= 4) col += 5; if (col == 3) col += 4; もちろん今のままでも問題なく動いているでしょうから、直さなくていいです。 ただ、こういう風に書くこともできますよ、というただのアドバイスです。よか ったら参考にしてください。