millis()の使い方

Question

millisを使いたくて以下のプログラムを作成したのですが上手く動きません。
改善点を是非教えてほしいです。
#include<Servo.h>
Servo myservo;
Servo myservo2;
Servo myservo3;
int pos=0;
int flag=0;
int sw1=1;
long beforetime = 0;
void setup(){
myservo.attach(9);
myservo2.attach(10);
myservo3.attach(11);
pinMode(13, OUTPUT);
}
void loop()
{
flag=digitalRead(sw1);
if(millis() - beforetime > 5000){
for( pos=0 ;pos<=180; pos=pos+1){
myservo.write(180-pos);
myservo2.write(180-pos);
myservo3.write(180-pos);
}
for( pos=180 ;pos>=0; pos-=1){
myservo.write(180-pos);
myservo2.write(180-pos);
myservo3.write(180-pos);
}
}
else{
}
}

Comments

[botamochi6277]

「うまく動かない」をもっと具体的に説明してください（例：コンパイルが通らない，サーボが動かない）．
また，配線に問題がある可能性もあるので，回路図や配線したArduinoの写真を貼付してもらえるとなお良いです．

[kei344]

回答が付いた質問の編集は慎重に行ってください。タイプミス程度なら修正する事もありますが、本文を削除するのはやめましょう。

Answer 1

ざっと見たところ５秒毎にサーボを左右に振り切らせたいのかな？

beforetime が初期化しただけで更新されてないのが悪いのでは？

setup() 内でbeforetime = mills(); として
if(millis() - beforetime > 5000){ の直下にも
beforetime = mills();
を入れればとりあえず動くと思いますよ。