keine vollständige Serial.print Ausgabe

hallo ,
der folgende Code ergibt keine vollständige Serielle Ausgabe
(original Code von hier, angepasst: https://stackoverflow.com/questions/...r-based-on-lcg)

Code:

---

//********************************************
// randomize functions
//********************************************


uint32_t getSeed() {
  return  (millis() + analogRead(A0)) ;
}


//-----------------------------------------------------
// Linear Congruence Generator, LCG 
//-----------------------------------------------------

static uint32_t  _RandNum_ = 1;

int rand_LCG() {
  uint32_t a = 1103515245ul, c = 12345;
  _RandNum_ = a * _RandNum_ + c;
  _RandNum_ = (uint16_t)(_RandNum_ / 65536);
  return (_RandNum_ % 32768) ;
}


void srand_LCG(uint32_t seed) {
    _RandNum_ = seed;
}


//--------------------------------------------
// setup
//--------------------------------------------
void setup() {
 
  Serial.begin(115200);
  delay(2000);
 
  Serial.println("starting..."); 
  Serial.println();

  volatile uint32_t  prng, seed;
  int16_t i;
  char buf[30];

  Serial.print("srand_LCG() seed=");
  seed=1;
  Serial.println(seed); 
  srand_LCG(seed);   
  //Serial.println(_RandNum_);
  for(i==0; i<10; i++) {
    prng=rand_LCG();
    Serial.print(i); Serial.print(" "); Serial.println(prng);   
  }
  Serial.println();


  Serial.print("srand_LCG() seed=");
  seed=2;
  Serial.println(seed); 
  srand_LCG(seed);
  //Serial.println(_RandNum_);
  for(i==0; i<10; i++) {
    prng=rand_LCG();
    Serial.print(i); Serial.print(" "); Serial.println(prng); 
  }
  Serial.println();
}

//--------------------------------------------
// loop
//--------------------------------------------
void loop() {
  delay(2000);
}

---

auf dem esp8266 wird nur die erste, aber nicht die 2. for-Schleife angezeigt, dann hängt er sich auf:

Code:

---

starting...

srand_LCG() seed=1
0 16838
1 14666
2 10953
3 11665
4 7451
5 26316
6 27974
7 27550
8 31532
9 5572

srand_LCG() seed=2

---

auf dem Due aber wird gar keine for-Schleife mehr angezeigt:

Code:

---

starting...

srand_LCG() seed=1

srand_LCG() seed=2

---

Wo zum .... steckt der Fehler?

So würde ich das machen:

Code:

---

//********************************************// randomize functions
//********************************************


uint32_t getSeed() {
  return  (millis() + analogRead(A0)) ;
}


//-----------------------------------------------------
// Linear Congruence Generator, LCG 
//-----------------------------------------------------

static uint32_t  _RandNum_ = 1;

int rand_LCG() {
  uint32_t a = 1103515245ul, c = 12345;
  _RandNum_ = a * _RandNum_ + c;
  _RandNum_ = (uint16_t)(_RandNum_ / 65536);
  return (_RandNum_ % 32768) ;
}


void srand_LCG(uint32_t seed) {
    _RandNum_ = seed;
}


//--------------------------------------------
// setup
//--------------------------------------------
void setup() {
 
  Serial.begin(115200);
  delay(2000);
 
  Serial.println("starting..."); 
  Serial.println();

  volatile uint32_t  prng, seed;
  int16_t i;
  char buf[30];

  Serial.print("srand_LCG() seed=");
  seed=1;
  Serial.println(seed); 
  srand_LCG(seed);   
  //Serial.println(_RandNum_);
  for(int i=0; i<10; i++) {
    prng=rand_LCG();
    Serial.print(i); Serial.print(" "); Serial.println(prng);   
  }
  Serial.println();


  Serial.print("srand_LCG() seed=");
  seed=2;
  Serial.println(seed); 
  srand_LCG(seed);
  //Serial.println(_RandNum_);
  for(int i=0; i<10; i++) {
    prng=rand_LCG();
    Serial.print(i); Serial.print(" "); Serial.println(prng); 
  }
  Serial.println();
}

//--------------------------------------------
// loop
//--------------------------------------------
void loop() {
  delay(2000); }

---

warum ist das besser?
(vorher hatte ich ja int16_t i)

- - - Aktualisiert - - -

update:
komisch, jetzt läuft es auf dem Due auch mit int16_t wie zuvor im ersten Code.... :confused:
und auf dem esp8266 auch ... :confused: :confused:


hast du noch was anderes abgeändert?

Nein, habe ich nicht. Deswegen habe ich das Rot markiert.

"I==0" ist falsch weil "==" ein Vergleichsoperator ist. Bin leider auch schon mindestens einmal drauf reingefallen. Den Typ schreibe ich davor, damit der Compiler weiß, was er genau für eine Variable anlegen soll. Ob das notwendig ist? - Keine Ahnung, schadet aber nicht.

ogottogottogott, dass mir sowas noch passiert ist ... ](*,) :Haue

aber nein, wenn das int i dafür bereits außerhalb deklariert wurde, muss man das nicht unbedingt innen auch noch mal neu für die Schleife lokal als int deklarieren.

Ich habe nur schnell drüber geschaut und das "int i" außen nicht gesehen. Weil Du das wieder mit Deiner Lieblingsmethode gemacht hast: int16_t i;

:)

Zitat:

---

Zitat von HaWe

aber nein, wenn das int i dafür bereits außerhalb deklariert wurde, muss man das nicht unbedingt innen auch noch mal neu für die Schleife lokal als int deklarieren.

---

Das ist so nicht richtig. Wenn man eine Variable innerhalb eines Blocks, einer Schleife, deklariert, wird eine neue lokale Variable angelegt. Und das auch, wenn außerhalb des Blocks schon eine des gleichen Namens existiert. Es gibt da kein "nicht unbedingt".

MfG Klebwax

Zitat:

---

Zitat von Klebwax

Das ist so nicht richtig. Wenn man eine Variable innerhalb eines Blocks, einer Schleife, deklariert, wird eine neue lokale Variable angelegt. Und das auch, wenn außerhalb des Blocks schon eine des gleichen Namens existiert. Es gibt da kein "nicht unbedingt".

MfG Klebwax

---

ich meinte das anders: nicht unbedingt dann, wenn man das i außerhalb der Schleife nicht unbedingt für andere Zwecke braucht als nur für einen Schleifenzähler dann und wann. Aber ich verstehe schon, was du meinst:
dass für for(int i....) immer wieder eine neue lokale Schleifenvariable angelegt wird, was ich bei mir allerdings nicht unbedingt brauchte ;)

@moppi: int16_t verwende ich gerne dann, wenn ich vermeiden will, dass bei größeren MCUs der Compiler für ein int 32bit Speicher reserviert, was ansonsten bei 32bit ARM cpus passiert.

Streng genommen sollte für die Schleife eine neue Variable erzeugt werden, die nur innerhalb der Schleife gültig ist. Weiß aber nicht, ob das hier in diesem C-Programmierkram auch so ist. Wenn es keine Warnung vom Compiler gibt, dass >>i<< schon existiert, müsste das schon so sein, ich weiß es hier aber auch nicht genau. Ich programmiere zwar schon 28 Jahre, die ersten 10 bis 20 fast täglich als Hauptbeschäftigung, aber C/C++ und Java zusammengenommen davon nur wenige Monate. In C/C++ habe ich eigentlich nie was gemacht, nur immer versuchsweise, mal ein paar Tage oder eine Woche, hin und wieder. Jetzt für Arduino ist, dass ich es mal intensiver verwende. Deshalb, wenn ich den Code so überfliege, springen mir die Sachen dann nicht sofort ins Auge, weil ich mehr nach "int i=...;" Ausschau halte und da ist es mir nicht aufgefallen.

MfG

Zitat:

---

Streng genommen sollte für die Schleife eine neue Variable erzeugt werden, die nur innerhalb der Schleife gültig ist.

---

warum sollte das der Fall sein, wenn ich nur 1 int Zähler-Variable für ein paar Dutzend Einsatzzwecke nacheinander brauche?

(dass du das mit meinem int16_t verstanden und hier nur übersehen hast, war mir schon klar, ich wollte nur begründen, warum ich grundsätzlich (fast immer) die stdint-Variablentypen int16_t (und daneben auch weitgehend int8_t, int32_t, selten sogar auch int64_t etc.) verwende. )