Arduino unoを用いてランダム関数を使いたいです。

Question

以下はArduino unoで書いたプログラムなのですが、うまくいきません。
本を参考にしたわけではなく、random関数の使い方を調べてif文と使い合わせたのですが、どうやらrandom関数に関してもif文に関してもうまく理解できていないようでエラーが出ます。
どうか載せたプログラムの原型を崩さずに何が間違っているのかアドバイスを頂けないでしょうか？

int a=300;
int b=200;
long randomNumber;

void setup() {
pinMode(12,OUTPUT) ;
pinMode(11,OUTPUT) ;
pinMode(10,OUTPUT) ;

Serial.begin(9600);
}

void loop() {


randomSeed(millis());
randomNumber = random(1,2);
if(randomNumber==1){
digitalWrite(12, HIGH);
delay(a);
digitalWrite(12, LOW);
delay(b);
digitalWrite(11, HIGH);
delay(a);
digitalWrite(11, LOW);
delay(b);
digitalWrite(10, HIGH);
delay(a);//光る間の時間
digitalWrite(10, LOW);
delay(1000);}

if (randomNumber==2){
   
   digitalWrite(12, HIGH);
  delay(10);
  digitalWrite(12, LOW);
  delay(10);
   digitalWrite(11, HIGH);
  delay(10);
  digitalWrite(11, LOW);
  delay(10);
  digitalWrite(10, HIGH);
  delay(10);//光る間の時間
  digitalWrite(10, LOW);
  delay(10);
   digitalWrite(11, HIGH);
  delay(10);
  digitalWrite(11, LOW);
  delay(10);
   digitalWrite(12, HIGH);
  delay(10);
  digitalWrite(12, LOW);


  delay(1000);}


  
 
  
  // put your main code here, to run repeatedly:
   

}

Answer 1

random( 1, 2 )は常に 1 を返します。

1以上、2未満、ということなので。
1と2がランダムに出るようにするのはrandom( 1, 3 )ではないでしょうか。
http://www.musashinodenpa.com/arduino/ref/index.php?f=0&pos=2901

Answer 2

randomSeed(millis());
は、乱数の初期設定の関数なので、setup関数の方で実行させましょう

Comments

[carnage0216]

どうもありがとうございます。
申し訳ないのですが、手元にArduino unoがないので正しくできているかはわからないのですが、検証は出来ました。エラーはありませんでした。
ちなみに、randomSeed(millis());がsetup関数の方に入っていなかった以外はプログラムに問題はなかったでしょうか？

[y_waiwai]

みたところ、コード自体にはエラーはなく、実行はできるでしょう。
ただ、どう動くのが正解なのかはわからないけど

これでエラーが出る、という場合はまた聞いてくだされ

[ttatsf]

millis()は起動してからの経過時間なので同じ値になりがち、それをrandomSeedにすると同じパターンになりがちです。
毎回違ったパターンにしたい場合には、使ってないアナログ入力をanalogRead()してrandomSeedにするっていうのをよくやります。