Переделать код

[Faton 11]

Добрый вечер! Нужно было числа от 0 до 32 распределить в случайном порядке. Для этого был создан генератор случайных неповторяющихся чисел. Как можно переделать этот код, не используя команду "goto loop"?

#include <stdafx.h>
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <time.h>
#include <string>
#include <sstream>
#include <cmath>
#include <fstream>

using namespace std;

const int N=34;
int mas[N];
string mas2[N];
string mas3[N];
int a;

int main ()
{ 
 srand ( time(NULL) );
      for(int i=0;i<N;++i)
       {
          loop: a=rand()%33;
	            for(int l=0;l<i;l++)
                 {
                    if(mas[l]==a)
                     {
                       goto loop;
                     }
                    mas[i]=a;
                 }
	  }
system ("pause");
}

[Stilet]

break вместо goto loop;

[Faton 11]

тогда сравнение происходит только с нулевым элементом массива и числа повторяются

[rrrFer]

код-то вообще плохой

for(int l=0;l<i;l++)
                 {
                    if(mas[l]==a)
                     {
                       goto loop;
                     }
                    mas[i]=a; //1
                 }

присваивание 1 выполнится столько раз, сколько в массиве элементов, не совпадающих с "а" стоит перед первым элементом, совпадающим с "а". Должно выполняться только 1 раз.

[sVasilich]

Одного break здесь мало, т.к. goto loop здесь являются ещ одним циклом. Например так (можно и лучше написать):

for(int i=0;i<N;++i){
    bool q = true;
        while(q){
            q=false;
            a=rand()%33;
            for(int l=0;l<i;l++)
                if(mas[l]==a){
                    q=true;
                    break;				
                }		
	    if(!q)
                mas[i]=a; 
        }
 }

[rrrFer]

#include <iostream>
#include <stdlib.h>
#include <time.h>

int main() { 
  
  const int N=34;
  int mas[N];

  srand(time(NULL));
  for (int l, i = 0; i < N;) {
    mas[i] = rand() % N;
    for (l = 0; l < i; l++)
      if (mas[l] == mas[i])
	break;
    i += l == i;
  }
  
  for (int i = 0; i < N; ++i)
    std::cout << mas[i] << " ";
  
  return 0;
}

твой код чуть поправил
-----------
не проверял, возможны опечатки

Одного break здесь мало,

достаточно

тока это говнокод, алгоритм может очень долго работать. Лучше сгенерировать ряд чисел по порядку, а потом перемешать

[Faton 11]

Всем спасибо за ответы, с этими кодами все генерируется как надо!

Лучше сгенерировать ряд чисел по порядку, а потом перемешать

да, так действительно лучше =)