Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
DoctorRA
1 / 1 / 0
Регистрация: 11.12.2016
Сообщений: 21
1

Прокомментируйте пожалуйста код

02.10.2017, 20:42. Просмотров 211. Ответов 9
Метки нет (Все метки)

// Zadacha1.1.cpp: определяет точку входа для консольного приложения.
////Задано четырехзначное число N.
//Будут ли все цифры этого числа различны или среди них есть равные, какие и сколько равных.
// ConsoleApplication1.cpp: определяет точку входа для консольного приложения.
//

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <math.h>
#include <locale.h>
using namespace std;
 
bool proverka(int b, int c)
{
    char str;
    if (int(c / (pow(10.0, (b - 1)))) == 0 || int(c / (pow(10.0, (b - 1)))) > 9) //разряды числа
        return false;
    else
        return true;
 
}
 
void main()
{
    setlocale(LC_ALL, "rus"); //Подключение русского языка
    int  c, b, n = 0;  //используемые переменные
    cout << "Укажите цифрой, каким будет ваше число (двухзначное, трехзначное и т.д.) : " << endl;
    cin >> b;
    for (bool i = false; i == false; i = proverka(b, c))
    {
        cout << "Введите число : " << endl;
        cin >> c;
    }
    int a;
    a = 0;
    int mas[11] = { 0 };  //Хранение количества повторяющихся значений
    mas[10] = c;
    for (int t = pow(10.0, b); t >= 10; t = t / 10)
    {
        c = (c - a * t);
        a = int(c / (t / 10));
        for (int i = 0; i < 10; i++)
        {
            if (a == i)
            {
                mas[i]++;
                break;
 
            }
        }
 
    }
    for (int i = 0; i < 10; i++)
    {
        if (mas[i] > 1)
        {
            cout << "В заданном вами числе " << mas[10] << " цифра " << i << " встречается " << mas[i] << " раз." << endl;
        }
        else
            n++;
 
    }
    if (n == 10) cout << "В числе " << mas[10] << " не имеется повторяющихся цифр.";
    _getch();
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
02.10.2017, 20:42
Ответы с готовыми решениями:

Прокомментируйте пожалуйста код
Объясните пожалуйста как работает данный кусок кода: template &lt;typename T&gt;...

Прокомментируйте код пожалуйста
#include&lt;iostream.h&gt; #include&lt;stdio.h&gt; #include&lt;conio.h&gt;...

Прокомментируйте, пожалуйста, код программы.
Доброго дня всем! Перед Вами - 2 программы - 1 шифрует текст методом...

Прокомментируйте пожалуйста код header.h
header.h #include &lt;iostream&gt; #include &lt;fstream&gt; #include &lt;string&gt;...

Прокомментируйте пожалуйста код (класс String)
Добрый день. У меня есть пользовательский класс String. Есть какие-то замечания...

9
lArtl
223 / 102 / 47
Регистрация: 09.10.2014
Сообщений: 581
Завершенные тесты: 3
02.10.2017, 20:48 2
Готово
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
//#include <iostream>
//#include <conio.h>
//#include <math.h>
//#include <locale.h>
//using namespace std;
//
//bool proverka(int b, int c)
//{
//  char str;
//  if (int(c / (pow(10.0, (b - 1)))) == 0 || int(c / (pow(10.0, (b - 1)))) > 9) //разряды числа
//      return false;
//  else
//      return true;
//
//}
//
//void main()
//{
//  setlocale(LC_ALL, "rus"); //Подключение русского языка
//  int c, b, n = 0; //используемые переменные
//  cout << "Укажите цифрой, каким будет ваше число (двухзначное, трехзначное и т.д.) : " << endl;
//  cin >> b;
//  for (bool i = false; i == false; i = proverka(b, c))
//  {
//      cout << "Введите число : " << endl;
//      cin >> c;
//  }
//  int a;
//  a = 0;
//  int mas[11] = { 0 }; //Хранение количества повторяющихся значений
//  mas[10] = c;
//  for (int t = pow(10.0, b); t >= 10; t = t / 10)
//  {
//      c = (c - a * t);
//      a = int(c / (t / 10));
//      for (int i = 0; i < 10; i++)
//      {
//          if (a == i)
//          {
//              mas[i]++;
//              break;
//
//          }
//      }
//
//  }
//  for (int i = 0; i < 10; i++)
//  {
//      if (mas[i] > 1)
//      {
//          cout << "В заданном вами числе " << mas[10] << " цифра " << i << " встречается " << mas[i] << " раз." << endl;
//      }
//      else
//          n++;
//
//  }
//  if (n == 10) cout << "В числе " << mas[10] << " не имеется повторяющихся цифр.";
//  _getch();
//}
3
DoctorRA
1 / 1 / 0
Регистрация: 11.12.2016
Сообщений: 21
02.10.2017, 20:51  [ТС] 3
Это, конечно, восхитительный ответ
Но помогите закомментировать оставшиеся строки
0
zss
Модератор
Эксперт С++
7394 / 6790 / 4295
Регистрация: 18.12.2011
Сообщений: 17,932
Завершенные тесты: 1
02.10.2017, 22:05 4
DoctorRA, Вам намекают, что слово "закомментировать" означает превратить фрагмент программы в комментарий. А для Вашей просьбы подходит слово "прокомментировать".
0
DoctorRA
1 / 1 / 0
Регистрация: 11.12.2016
Сообщений: 21
02.10.2017, 22:08  [ТС] 5
Вот черт! Спасибо, я не заметил

Добавлено через 24 секунды
не поможете прокомментировать код?
0
COKPOWEHEU
1047 / 729 / 172
Регистрация: 09.09.2017
Сообщений: 3,223
02.10.2017, 22:12 6
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <math.h>
#include <locale.h>
using namespace std;
 
/*название ужасно! Оно транслитом (что само по себе ужасно) и оно не отражает цели функции (что трижды ужасно). Основной аргумент идет вторым, что непривычно. Переменные не имеют осмысленных названий (что тоже трижды ужасно)
В общем, она проверяет, состоит ли число c из b десятичных разрядов
bool test_dec_size(int num, int digits)//проверка размера десятичного числа, num-число, digits - количество разрядов
*/ 
bool proverka(int b, int c)
{
    char str;
    /*int(x) - целая часть x
      pow(10.0, (b-1) ) - получение 10^(b-1). Например, b=3 -> pow(...)=100
      int(c / pow(...)) - "отрезание" (b-1) младших десятичных разрядов.
        Например, c=1234.56, b=2 -> pow(...)=10 -> (c/pow())=123.456 -> int(c/pow)=123
      сравнение с нулем выполняется если c<10^(b-1), то есть в c менее (b-1) десятичных цифр
      вторая проверка выполняется если в с более b цифр (при делении получилось число больше либо равно 10)
      в целом, проверка весьма неочевидна и неоптимальна. И что-то мне подсказывает, что сам подход стоит поменять
    */
    if (int(c / (pow(10.0, (b - 1)))) == 0 || int(c / (pow(10.0, (b - 1)))) > 9) //разряды числа
        return false;
    else
        return true;
 
}
 
void main()
{
    setlocale(LC_ALL, "rus"); //Подключение русского языка
    int  c, b, n = 0;  //используемые переменные
//вывод строки и перевода строки
    cout << "Укажите цифрой, каким будет ваше число (двухзначное, трехзначное и т.д.) : " << endl;
//ввод чего-то в переменную b с периферийного устройства рандомного ввода*
//проверки на корректность, разумеется, нет... кто бы сомневался
    cin >> b;
//достаточно непривычное использование for(). По смыслу лучше подошел бы while()
//на каждой итерации оно присваивает переменной i (крайне неудачное имя!) результат функции proverka()
//если на очередной итерации значение этой переменной оказалось true (по смыслу - если пользователь ввел число с нужным количеством цифр) то цикл прерывается.
    for (bool i = false; i == false; i = proverka(b, c))
    {
        cout << "Введите число : " << endl;
        cin >> c;//внутри каждой итерации запрашивается новое число
    }
    int a;
    a = 0;
    int mas[11] = { 0 };  //Хранение количества повторяющихся значений
    mas[10] = c;
//перебор чисел от 10^b до 1 для разложения на десятичные цифры. Сделано через жопу, логичнее было перебирать с младшего разряда
//для этого достаточно
//digit = x%10;
//x /= 10;
    for (int t = pow(10.0, b); t >= 10; t = t / 10)
    {
        c = (c - a * t); //"отрезаем" от числа старшую цифру
        a = int(c / (t / 10)); //получение старшего десятичного разряда (цифры)
//черезжопный способ сделать mas[a]++; - увеличить счетчик конкретной цифры (что название mas[] не подходит для задачи - говорить излишне)
        for (int i = 0; i < 10; i++)
        {
            if (a == i)
            {
                mas[i]++;
                break;
 
            }
        }
 
    }
//перебор содержимого массива
    for (int i = 0; i < 10; i++)
    {
        if (mas[i] > 1)
        {
//вывод строки, числа, строки, числа, строки, числа, строки и конца строки (может где-то обсчитался)
            cout << "В заданном вами числе " << mas[10] << " цифра " << i << " встречается " << mas[i] << " раз." << endl;
        }
        else
            n++; //счетчик нулевых элементов массива
 
    }
    if (n == 10) cout << "В числе " << mas[10] << " не имеется повторяющихся цифр.";
    _getch();
}
Суммарно - этот код пример как НЕ НАДО ПИСАТЬ.
2
DoctorRA
1 / 1 / 0
Регистрация: 11.12.2016
Сообщений: 21
03.10.2017, 11:11  [ТС] 7
Есть альтернатива этому коду?
0
COKPOWEHEU
1047 / 729 / 172
Регистрация: 09.09.2017
Сообщений: 3,223
03.10.2017, 11:52 8
C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
#include <stdio.h>
 
//степень 10. Наверняка можно сделать быстрее, но для задачи сойдет
int pow10(int n){
  int i, res=1;
  for(i=0;i<n;i++)res*=10;
  return res;
}
 
int main(){
  int N=0;
  int digs=4;
  int p10, i, digit;
  int num[10]={0};
  
  printf("Input number of decimal digits>");
  scanf("%i", &digs);
  
  p10 = pow10(digs-1); //вычисляется всего 1 раз, а не при каждом вводе
  printf("Input number>");
  scanf("%i",&N);
  while( N<p10 || N>=(p10*10) ){
    printf("Wrong number; try again>");
    scanf("%i",&N);
  }
  
  //стандартное разбиение на десятичные цифры
  for(i=0;i<digs;i++){
    digit = N%10;
    N /= 10;
    num[digit]++; //добавляем в массив очередную цифру
  }
  
  //можно было бы совместить определение наличия повторений с выводом, но из соображений модульности и наглядности оставлю так
  for(i=0;i<10;i++){
    if(num[i]>1)break;
  }
  if(num[i]<=1){
    printf("different\n");
    return 0; //возвращаемое значение: 0-повторений не найдено
  }else{
    for(i=0;i<10;i++)
      if(num[i]>1)printf("%i : %i times\n", i, num[i]);
    return 1; //1 - повторения найдены. Не то чтобы это было нужно для интерактивной программы, но ведь и хуже не будет
  }
}
И почему мне кажется, что даже с более логичным и прокомментированным кодом вы разбираться не будете...
P.S. Да, я пишу на С++ в Си-стиле. Он намного проще.
0
MrGluck
Модератор
Эксперт CЭксперт С++
8101 / 4952 / 1436
Регистрация: 29.11.2010
Сообщений: 13,437
03.10.2017, 12:23 9
Цитата Сообщение от COKPOWEHEU Посмотреть сообщение
Да, я пишу на С++ в Си-стиле. Он намного проще.
Вы пишите на С (однострочные комментарии добавили в С99).
0
COKPOWEHEU
1047 / 729 / 172
Регистрация: 09.09.2017
Сообщений: 3,223
03.10.2017, 13:05 10
Современные компиляторы С++ такое соберут.
Однострочные комментарии слишком удобны чтобы от них отказываться ради совместимости со старыми компиляторами.
0
03.10.2017, 13:05
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
03.10.2017, 13:05

Прокомментируйте, пожалуйста, код. Список - очередь
//Список вида &quot;очередь&quot; #include &lt;iostream&gt; using namespace std; ...

Написал тетрис. Прокомментируйте код пожалуйста )
Программа полностью готова. Все как в тетрисе работает. Немного мигает, не знаю...

Объясните код или прокомментируйте его, пожалуйста
Змейка закручена, но я не понимаю как, объясните пожалуйста:wall: #include...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
10
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru