[c++] semplice controllo con If che non va

[AYAMY]

Ciao a tutti...
Ho un programmino che mi genera dei numeri secondo un certo algoritmo...
Ora io vorrei fare in modo che sono i numeri che sono compresi tra 2 estremi (min e max) siano accettati...
Per fare questo ho usato un ciclo for con dentro un if.... ma non va...secondo voi dove sta l'inghippo?
NB: Z.next mi restituisce un numero secondo un certo algoritmo
poi io associo questo numero alla variabile "dentrocode" e con questa faccio il controllo.... ma non va...mi escono anche valori che non sono compresi tra min e max????
mi date una mano please?
Grazie

for (int i = 0; i < 99; ++i)
        {
          dentrocode=Z.Next();
          if ( dentrocode >= min and dentrocode <= max )
             {
              vector[k] = dentrocode; 
              k++;
              }        
              }
     for (int i = 0; i < 99; ++i)
         { 
          cout << "Vettore Valori" << endl;
          cout << vector[i] << endl;
          }

[Ziosilvio]

Per caso, i numeri fuori dall'intervallo, sono solo tra gli elementi di vector con i>=k?
Perché, se è così, il problema è solo che tu visualizzi tutti gli elementi del vettore, e non solo quelli in cui hai messo davvero un numero: e in generale, la memoria allocata è "sporca" e può contenere qualsiasi valore.

[reptile9985]

l'indice i del primo ciclo for come mai non viene usato dentro al pezzo di codice?? e k dove l'hai inizializzato??
perchè scrivi and e non &&?? come vedi qualcosa da sistemare c'è

[AYAMY]

innanzitutto grazie x l'aiuto...
K l'ho inizializzato in alto nella parte del programma che non ho postato... adesso ho anche sostituito and con && ma continua a non andare...

Per caso, i numeri fuori dall'intervallo, sono solo tra gli elementi di vector con i>=k?

forse non ho capito.. quello che mi stai dicendo.....
se con il ciclo

for (int i = 0; i < 99; ++i)
        {
          pippo << Z.Next() << "\n";
          dentrocode=Z.Next();
          if ( dentrocode >= min && dentrocode <= max )
             {
              vector[k] = dentrocode; 
              k++;
              }        
              }

faccio prima il controllo sul valore del numero generato.. e poi lo metto nel vettore chiamato vector[k]... in questo vettore ci dovrebbero essere solo valori compresi tra min e max... giusto?
Ad esempio.. Z.next() mi da come valore 79... min=81 , max = 99 l'IF quindi dovrebbe scartare 79 visto che è minore di min ..e poi continuare l'esecuzione e rieseguire il ciclo for in cui mi viene generato un altro numero da Z.next() sul quale riandare a fare il test di controllo...
E' giusto l'algoritmo? oppure ho sbagliato ..e non funziona come ho scritto?


PS: ecco tutto il codice

#include <iostream>
#include <fstream>
#include "newran03/newran.h"

int main()
{
ofstream pippo;
pippo.open("zuzzurellone.txt", ios::out);
cout << "Prego Inserise L'Angolo (cioe' la Media della Gaussiana)" << endl;
cout << "La Varianza verrà calcolata come il 10% della Media" << endl;
float media;
cin >> media;
float sigma;
int k = 0;
sigma = (0.1*media);
float min = media-3*sigma;
float max = media+3*sigma;
float dentrocode;
float vector[100];
srand(time(0));
int mostra = rand()%100;
char s[]="test file";
MotherOfAll urng;
Random::Set(urng);
Normal N;
SumRandom Z = media + sigma * N;
for (int i = 0; i < 99; ++i)
{
pippo << Z.Next() << "\n";
dentrocode=Z.Next();
if ( dentrocode >= min && dentrocode <= max )
{
vector[k] = dentrocode;
k++;
}
}
for (int i = 0; i < 99; ++i)
{
cout << "Vettore Valori" << endl;
cout << vector[i] << endl;
}
//std::cout << std::endl;
cout << "Test eseguito trovi i risultati in zuzzurellone.txt" << endl;
cin.get();
cin.get();
}

PS: Si appoggia a delle librerie esterne che si possono trovare qui

http://www.robertnz.net/ftp/newran03.zip

GRAZIE ANCORA

[reptile9985]

il secondo for dovrebbe fare il controllo su k, perchè nel vettore hai inserito k elementi nell'intervallo desiderato:

for (int i = 0; i < k; ++i)
{
cout << "Vettore Valori" << endl;
cout << vector[i] << endl;
}

il resto del tuo codice sembra ok, a meno che l'errore non sia nell'uso della libreria esterna...

[AYAMY]

ho messo K ma continua a non andare..
come è possibile?
Ma che cavolo di errore c'è nell'IF... mi sembra che sia tutto apposto?
Secondo Voi l'IF funziona nel modo che ho postato poco sopra.. escludendo i valori esterni a min e max?

[yorkeiser]

L'if mi pare giusto, non credo ti dia problemi quello.

Non ho idea di cosa sia quel SumRandom Z, ma se viene gestito come una lista/collection, nelle seguenti istruzioni

pippo << Z.Next() << "\n";
dentrocode=Z.Next();

la scorri in avanti 2 volte.

Potresti aver bisogno di fare una cosa del tipo

float temp=Z.Next();
pippo << temp << "\n";
dentrocode=temp;

[AYAMY]

Non ci credo....
Non è possibile.. non va nemmeno così!!!
Ma perchè eppure è di una semplicità imbarazzante... ma non trovo il problema...
che cavolo..

Ragà.. chiedo il vostro Aiuto...

[yorkeiser]

Con qualche semplice modifica al tuo programma ti mostro che l'if è giusto (non sono riuscito a linkare le tue librerie, mancano dei .lib e non so in alternativa quali .cpp includere). In qualsiasi caso ti riporto il codice, che non cambia nulla dal tuo punto di vista.

#include <iostream.h>
#include <time.h>
#include <math.h>
#include <stdlib.h>
int main()
{
cout << "Prego Inserise L'Angolo (cioe' la Media della Gaussiana)" << endl;
cout << "La Varianza verrà calcolata come il 10% della Media" << endl;
float media;
cin >> media;
float sigma;
int k = 0;
sigma = (0.1*media);
float min = media-3*sigma;
float max = media+3*sigma;
float dentrocode;
float vector[100];
srand(time(0));
int mostra = rand()%100;
char s[]="test file";
for (int i = 0; i < 99; ++i)
{
dentrocode=(float)(rand()%100);
if ( dentrocode >= min && dentrocode <= max )
{
vector[k] = dentrocode;
k++;
}
}
for (i = 0; i < 99; ++i)
{
cout << "Vettore Valori" << " Min:" << min << " Max:" << max << endl;
cout << vector[i] << endl;
}
cin.get();
cin.get();
}

Il problema è che stampi TUTTO l'array (ovvero da 0 a 99) e non solo per i k indici che hai filtrato per vector (ovvero i valori realmente compresi tra min e max).


Di seguito ti riporto l'output del programma:

La Varianza verrÓ calcolata come
56
Vettore Valori Min:39.2 Max:72.8
65
Vettore Valori Min:39.2 Max:72.8
58
Vettore Valori Min:39.2 Max:72.8
58
Vettore Valori Min:39.2 Max:72.8
66
Vettore Valori Min:39.2 Max:72.8
62
Vettore Valori Min:39.2 Max:72.8
70
Vettore Valori Min:39.2 Max:72.8
43
Vettore Valori Min:39.2 Max:72.8
55
Vettore Valori Min:39.2 Max:72.8
66
Vettore Valori Min:39.2 Max:72.8
49
Vettore Valori Min:39.2 Max:72.8
68
Vettore Valori Min:39.2 Max:72.8

Ovvero fino a un certo indice l'output è giusto, poi ovviamente negli indici finali (dal valore massimo di k fino a 99) sfori da min e max perchè questi valori non dovrebbero far parte di vector. Il linkare o meno le librerie che hai postato non dovrebbe cambiar nulla

[AYAMY]

... adesso il mio programma è così

#include <iostream>
#include <fstream>
#include "newran03/newran.h"

int main()
{
 ofstream pippo;
    pippo.open("zuzzurellone.txt",  ios::out);
    cout << "Prego Inserise L'Angolo (cioe' la Media della Gaussiana)" << endl;
    cout << "La Varianza verrà calcolata come il 10% della Media" << endl;
    float media;
    cin >> media;
    float sigma;
    int k = 0;
    sigma = (0.1*media);
    float min = media-3*sigma;
    float max = media+3*sigma;
    float dentrocode;
    float vector[100];
    srand(time(0));
    int mostra = rand()%100;
    char s[]="test file";
    MotherOfAll urng;
	Random::Set(urng);
	Normal N;
	SumRandom Z = media + sigma * N;
	for (int i = 0; i < 99; ++i)
        {
          float temp =Z.Next();
          pippo << temp  << "\n";
          dentrocode=temp;
          if ( dentrocode >= min && dentrocode <= max )
             {
              vector[k] = dentrocode; 
              k++;
              }        
              }
     for (int i = 0; i < k; ++i)
         { 
          cout << "Vettore Valori" << endl;
          cout << vector[i] << endl;
          }                            
	//std::cout << std::endl;
	cout << "Test eseguito trovi i risultati in zuzzurellone.txt" << endl; 
    cin.get();
    cin.get();
}

quindi nella stampa dei valori del vettore ci si ferma a k ma escono comunque valori che sono esterni agli estremi....

Sto iniziando a sflippare...
Concettualmente dovrebbe andare...
Il codice mi sembra giusto e una volta lanciato non ne vuole sapere di funzionare nel modo giusto....

Qualche altra idea su dove possa stare il problema?