Подбор введённого пароля

@socweb · Регистрация: 11.08.2016

Author24 — интернет-сервис помощи студентам

Не могу понять почему не работает. Компилятор не выдаёт ошибок, но ничего не работает. Точнее работает только случай ввода 0000;. Проблема где-то в циклах, возможно в трансформации char->int.

C++

int main(int argc, char* argv[])
{
  char pas[5], log1[5]="0000";
  cin>>pas;
  cout << "WAITING"<<endl;
  atoi(pas);
  atoi(log1);
  if(strcmp(pas,log1)==0){cout << "PASSWORD: " << log1;}
  else
   for(int z=0;z<10;z++)
  {
     log1[0]=z;
 
  for(int y=0;y<10;y++)
  {
    log1[1]=y;
  for(int x=0;x<10;x++)
  {
     log1[2]=x;
  for(int i=0;i<10;i++)
        {
           log1[3]=i;
 
           if(strcmp(pas,log1)==0){cout << "PASSWORD: " << log1;}
 
        }
   }
   }
   }
        system("pause");
        return 0;
}

@nmcf · 11.08.2016, 22:40

Циклы должны быть на символах:

C++

1	for (char z = '0'; z <= '9'; ++z)

@Devenmort · 11.08.2016, 22:48

C++

int main(int argc, char* argv[])
{
  char pas[5], log1[5]="0000"; // Создаете неинициализированный массив pas[5], и инициализированный log1[5]
  cin>>pas; // Возможность вылететь за пределы массива
  cout << "WAITING"<<endl;
  atoi(pas); //эта функция не меняет входных данных, она возвращает int, следовательно здесь происходит не то, чего вы возможно хотели
  atoi(log1);
  if(strcmp(pas,log1)==0){cout << "PASSWORD: " << log1;} // В pas лежит совсем не то, чего вы ожидали, а может вы не того ожидали в log1
  else
   for(int z=0;z<10;z++)
  {
     log1[0]=z; //неявное приведение типа int в char. Получится совсем не то, чего вы ожидаете.
 
  for(int y=0;y<10;y++)
  {
    log1[1]=y;
  for(int x=0;x<10;x++)
  {
     log1[2]=x;
  for(int i=0;i<10;i++)
        {
           log1[3]=i;
 
           if(strcmp(pas,log1)==0){cout << "PASSWORD: " << log1;} 
 
        }
   }
   }
   }
        system("pause");
        return 0;

У вас идет серьезная путанница в том, что такое символ и что такое число.
Работает оно со случаем 0000, потому что первый strcmp отрабатывает правильно (просто потому что вам повезло с ошибками), а дальше идет множество ошибок, которые шансов уже не дают.

@socweb · 11.08.2016, 22:57 **[ТС]**

Заработало, однако если преобразовывать строки через atoi в int можно ли будет в цикле for оставить тип int?

Добавлено через 42 секунды
Добавлено через 2 минуты
Devenmort, однако достаточно оказалось только изменения типа переменных цикла на char;

@Devenmort · 11.08.2016, 23:05

Да, здесь этого достаточно, я просто указал все ошибки первого варианта кода, а не способ исправления. Даже если просто заменить на итерации по char, все равно остаются указанные мной ошибки в начале кода.
Раз решение уже дано, зачем повторяться?

atoi преобразует всю строку в одно число, а не набор цифр.
Цикл бы сменился на for (int i = 0; i<10000; i++)

да и пришлось бы менять изначальное char pas[5] = "0000" на int pas = 0

И это решение было бы лучше, не было бы никакого strcmp, было бы обычное сравнение чисел.

@socweb · 11.08.2016, 23:19 **[ТС]**

Получается заработало через char, но как сделать с int? Нужно привести charовские массивы к intовым? Как?

Добавлено через 12 минут
Да, с atoi наглупил сильно. Действительно ожидал совсем не того... Ну что ж, спасибо за объяснение.

Подбор введённого пароля

Решение

Решение

@socweb 0 / 0 / 0 Регистрация: 11.08.2016 Сообщений: 6
	11.08.2016, 22:57 [ТС]	4
	Заработало, однако если преобразовывать строки через atoi в int можно ли будет в цикле for оставить тип int? Добавлено через 42 секунды Добавлено через 2 минуты Devenmort, однако достаточно оказалось только изменения типа переменных цикла на char; 0

@Devenmort 9 / 9 / 9 Регистрация: 04.08.2016 Сообщений: 24
	11.08.2016, 23:05	5
	Сообщение было отмечено socweb как решение Решение Да, здесь этого достаточно, я просто указал все ошибки первого варианта кода, а не способ исправления. Даже если просто заменить на итерации по char, все равно остаются указанные мной ошибки в начале кода. Раз решение уже дано, зачем повторяться? atoi преобразует всю строку в одно число, а не набор цифр. Цикл бы сменился на `for (int i = 0; i<10000; i++)` да и пришлось бы менять изначальное `char pas[5] = "0000"` на `int pas = 0` И это решение было бы лучше, не было бы никакого strcmp, было бы обычное сравнение чисел. 1

@socweb 0 / 0 / 0 Регистрация: 11.08.2016 Сообщений: 6
	11.08.2016, 23:19 [ТС]	6
	Получается заработало через char, но как сделать с int? Нужно привести charовские массивы к intовым? Как? Добавлено через 12 минут Да, с atoi наглупил сильно. Действительно ожидал совсем не того... Ну что ж, спасибо за объяснение. 0