Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.74/34: Рейтинг темы: голосов - 34, средняя оценка - 4.74
0 / 0 / 0
Регистрация: 04.12.2015
Сообщений: 12

Помогите разобрать чужой код

10.12.2015, 20:55. Показов 6454. Ответов 18
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
нам дали задание понять, что собственно написано в коде.
Может кто подскажет как легче читать чужой код.
Вот тот самый кусок кода, который мне надо разобрать. Помогите новичку!
C++
1
2
3
4
5
6
7
8
9
int F7(int c[], int n)
{ int i,j,m,s;
for (s=0, i=0; i < n-1; i++)
    {
    for (j=i+1, m=0; j<n; j++)
        if (c[i]==c[j]) m++;
    if (m > s) s = m;
    }
return s; }
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
10.12.2015, 20:55
Ответы с готовыми решениями:

учусь разбирать чужой код
что делает эта функция? static unsigned long foo (char *w) { int c; unsigned long u = 0; while (*w != 0) { if...

Как научиться эффективно читать чужой код С++
Поделитесь пожалуйста опытом или просто мыслями о том, как всё таки начать разбираться в не своём коде? Может у кого-то есть какие-то...

Новичок. Прошу "бывалых" разобрать код подетально, код писал сам (рабочий)
Этот код моя первая самостоятельная работа. Писал его опираясь на уроки из интернета. Однако там плохо описан механизм действия каждой...

18
 Аватар для Mesteriis
599 / 237 / 69
Регистрация: 08.08.2015
Сообщений: 1,637
10.12.2015, 20:59
Dreqgan, это самописный метод, который принимает в себя массив интов и переменную типа инт, производит действия, и возвращает значения типа инт!
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
10.12.2015, 21:06
Максимальное количество повторов?
0
 Аватар для Mesteriis
599 / 237 / 69
Регистрация: 08.08.2015
Сообщений: 1,637
10.12.2015, 21:07
Цитата Сообщение от nmcf Посмотреть сообщение
Максимальное количество повторов?
хз, без тегов даже всматриваться не хочу, сижу с мобилы просто
0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
10.12.2015, 21:17
Dreqgan, за такое форматирование кода надо бить. Ногами. Собственно, как и за сам код. Так автору кода и скажите.
Вот Вам код в более пристойном виде (в плане форматирования):
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
int F7(int c[], int n)
{
    int i, j, m, s;
    for (s=0, i=0; i < n-1; i++)
    {
        for (j=i+1, m=0; j<n; j++)
        {
            if (c[i]==c[j])
                m++;
            if (m > s)
                s = m;
        }
    }
    return s;
}
Это код самописной (самодельной) функции по имени F7, которая возвращает целое значение (тот самый int в начале кода) и принимает в качестве аргументов указатель на начало массива c целых чисел (первый параметр) и размер массива n (второй параметр). Функция (по задумке) считает количество одинаковых элементов в массиве. На деле же, выполняет она это довольно хреново. Пруф:
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
#include <iostream>
 
int F7(int c[], int n)
{
    int i, j, m, s;
    for (s=0, i=0; i < n-1; i++)
    {
        for (j=i+1, m=0; j<n; j++)
        {
            if (c[i]==c[j])
                m++;
            if (m > s)
                s = m;
        }
    }
    return s;
}
 
int main()
{
    const int SOME_NUMBER = 4;
    const int ARR_SIZE = 8;
 
    int * arr = new int[ARR_SIZE];
    for (int i=0; i < ARR_SIZE; i++)
    {
        if (i%2)
            arr[i] = i + 1;
        else
            arr[i] = SOME_NUMBER;
    }
 
    for (int i=0; i < ARR_SIZE; i++)
        std::cout << arr[i] << " ";
 
    std::cout << "\n\nThe amount of equal elements is: " << F7(arr, ARR_SIZE);
 
    delete [] arr;
 
    return 0;
}
Миниатюры
Помогите разобрать чужой код  
0
4949 / 2289 / 287
Регистрация: 01.03.2013
Сообщений: 5,990
Записей в блоге: 32
11.12.2015, 00:00
gru74ik, форматирование конечно хреновое, ты прав, надо было однострочником. А считает она конечно неоптимально, но нормально - если представить, что возвращает она максимальное количество дублей. У тебя 5 четверок в массиве - 4 дубля
1
11.12.2015, 00:02

Не по теме:

Цитата Сообщение от gru74ik Посмотреть сообщение
Так автору кода и скажите.
А еще автору кода надо сказать про STL и C++11,C++14 :)

0
2688 / 2260 / 244
Регистрация: 03.07.2012
Сообщений: 8,231
Записей в блоге: 1
11.12.2015, 01:54
Цитата Сообщение от gru74ik Посмотреть сообщение
На деле же, выполняет она это довольно хреново.
А какой код не хреновый?
0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
11.12.2015, 09:25
Цитата Сообщение от _Ivana Посмотреть сообщение
А считает она конечно неоптимально, но нормально - если представить, что возвращает она максимальное количество дублей.
ОК, ногами отменяется. И вообще битьё. Подзатыльник за форматирование кода.

Добавлено через 2 часа 47 минут
Цитата Сообщение от zer0mail Посмотреть сообщение
А какой код не хреновый?
По пунктам?
  1. С нормальными именами функций.
  2. С нормальными именами параметров функций.
  3. С нормальными именами переменных.
  4. С нормальным форматированием кода (форматирование кода в примере от ТС - уёжищное).
  5. Без велосипедов, с использованием STL и текущего стандарта.
  6. С использованием const, где это необходимо.

Например, так:
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
#include <iostream>
#include <set>
 
int count_equal_elements(int arr[], int arr_size)
{
    std::set<int> temp(arr, arr + arr_size);
    return ( arr_size - temp.size() ) + 1 ;
}
 
int main()
{
    const int SOME_NUMBER = 4;
    const int ARR_SIZE = 8;
 
    int * arr = new int[ARR_SIZE];
    for (int i=0; i < ARR_SIZE; i++)
    {
        if (i%2)
            arr[i] = i + 1;
        else
            arr[i] = SOME_NUMBER;
    }
 
    for (int i=0; i < ARR_SIZE; i++)
        std::cout << arr[i] << " ";
 
    std::cout << "\n\nThe amount of equal elements is: "
              << count_equal_elements(arr, ARR_SIZE);
 
    delete [] arr;
 
    return 0;
}
0
11.12.2015, 09:35

Не по теме:

Цитата Сообщение от gru74ik Посмотреть сообщение
По пунктам
Я как специально под эти пункты последнего кота писал Найти количество всевозможных маршрутов от города до города :) Привожу не холивара ради а просто совпало удачно так. Хотя const я не против понасовать почти везде :)

0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
11.12.2015, 10:00
Блин, про const сказал, а сам забыл. Подзатыльник мне тоже. Вот:

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
#include <iostream>
#include <set>
 
int count_equal_elements(const int arr[], const int arr_size)
{
    std::set<int> temp(arr, arr + arr_size);
    return arr_size - temp.size() + 1 ;
}
 
int main()
{
    const int SOME_NUMBER = 4;
    const int ARR_SIZE = 8;
 
    int * arr = new int[ARR_SIZE];
    for (int i=0; i < ARR_SIZE; i++)
    {
        if (i%2)
            arr[i] = i;
        else
            arr[i] = SOME_NUMBER;
    }
 
    for (int i=0; i < ARR_SIZE; i++)
        std::cout << arr[i] << " ";
 
    std::cout << "\n\nThe amount of equal elements is: "
              << count_equal_elements(arr, ARR_SIZE);
 
    delete [] arr;
 
    return 0;
}
0
 Аватар для Mesteriis
599 / 237 / 69
Регистрация: 08.08.2015
Сообщений: 1,637
11.12.2015, 10:27
gru74ik, слушай, вопрос не по теме конечно, но всеже, какой смыслал передавать в метод переменную типа const? Видел по сути ей до фонаря, или я не прав?
0
2688 / 2260 / 244
Регистрация: 03.07.2012
Сообщений: 8,231
Записей в блоге: 1
11.12.2015, 10:51
Цитата Сообщение от gru74ik Посмотреть сообщение
По пунктам?
С нормальными именами функций.
С нормальными именами параметров функций.
С нормальными именами переменных.
С нормальным форматированием кода (форматирование кода в примере от ТС - уёжищное).
Без велосипедов, с использованием STL и текущего стандарта.
С использованием const, где это необходимо.
Имхо, не хватает одного пункта, самого главного:
7. Код должен правильно выполнять поставленную задачу.

Ваш пример отработает неверно на данных: 1,1,2,2,3,3,3
2
11.12.2015, 10:54

Не по теме:

zer0mail, к черту правильность, мы эстеты )))

0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
11.12.2015, 11:04
Цитата Сообщение от Mesteriis Посмотреть сообщение
какой смыслал передавать в метод переменную типа const?
Есть такая штука "граммар const". Это он.

Цитата Сообщение от zer0mail Посмотреть сообщение
Ваш пример отработает неверно на данных: 1,1,2,2,3,3,3
Хых, а и правда. Благодарю за наблюдательность. Ща поправлю.
0
 Аватар для Kuzia domovenok
4268 / 3327 / 926
Регистрация: 25.03.2012
Сообщений: 12,532
Записей в блоге: 1
11.12.2015, 13:14
Цитата Сообщение от Mesteriis Посмотреть сообщение
zer0mail, к черту правильность, мы эстеты )))
при чём тут эстеты/неэстеты? Тебе нравится смотреть на быдлокод кого-нибудь вроде _Ivana? Он специально прикидывается непонимающим, а на деле он намеренно обфусцирует любой код, который выдаёт. Он ненавидит новичков и хочет над ними издеваться.
0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
12.12.2015, 13:43
Цитата Сообщение от gru74ik Посмотреть сообщение
Ща поправлю.
Мда, быстро не получается.
0
0 / 0 / 0
Регистрация: 04.12.2015
Сообщений: 12
12.12.2015, 15:23  [ТС]
Спасибо за мнение)
0
Dimension
595 / 463 / 223
Регистрация: 08.04.2014
Сообщений: 1,710
12.12.2015, 15:52
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
#include <bits/stdc++.h>
using namespace std;
int f(int *a, int s) {
    sort(a, a + s);
    int ans=0, t = 0;
    for (int i = 1;i < s;i++) {
        while (i < s && a[i] == a[i - 1])
            t++,i++;
        ans = max(ans, t);
        t = 0;
    }
    return ans+1;
}
int main() {
    srand(time(0));
    const int s = 10;
    int a[s];
    for (int i = 0;i < s;i++)
        a[i] = rand() % 15, cout << a[i] << " ";
    cout << endl;
    cout << f(a, s);
    cin.get(), cin.get();
    return 0;
}
Добавлено через 7 минут
либо так ,если время важнее памяти и числа не очень большие и не отрицательные
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
#include <bits/stdc++.h>
using namespace std;
int f(int *a, int s,int m) {
    int ans = 0;
    vector<int> counter(m + 1);
    for (int i = 0;i < s;i++) {
        counter[a[i]]++;
        ans = max(ans, counter[a[i]]);
    }
    return ans;
}
int main() {
    srand(time(0));
    const int s = 10;
    int a[s],MaxN=0;
    for (int i = 0;i < s;i++) {
        a[i] = rand() % 15;
        cout << a[i] << " ";
        MaxN = max(MaxN, a[i]);
    }
    cout << endl;
    cout << f(a, s,MaxN);
    cin.get(), cin.get();
    return 0;
}
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
12.12.2015, 15:52
Помогаю со студенческими работами здесь

Разобрать код
В общем писал код для выявление слов в которых есть буква 'K', и вывод их на экран, все работает, но не проходит когда пишу два подобных...

Разобрать код с++
Всем доброго времени суток. Дело такое нужно разобрать код явных ошибок нету но при компиляции вижуалка вылетает с ошибкой вроде в хидере...

Разобрать код на с++
Всем доброго времени суток. Не подскажите в чём у меня ошибка. В общем задание такое я должен создать в готовой программе класс и в нём...

Разобрать код
Помогите пожулйста розобрать код програми struct node { int item; node* next; { item = x; next = t; } }; typadef node *link; ...

Разобрать код
Помогите разобраться с кодом!Можите рассписать где что находится пожалуйста! #include &quot;stdafx.h&quot; #include&lt;iostream&gt; ...


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

Или воспользуйтесь поиском по форуму:
19
Ответ Создать тему
Новые блоги и статьи
Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly
8Observer8 30.01.2026
Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .
Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров
8Observer8 29.01.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT
bedvit 28.01.2026
Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 28.01.2026
Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image
8Observer8 27.01.2026
Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .
Влияние грибов на сукцессию
anaschu 26.01.2026
Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .
Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android
8Observer8 26.01.2026
Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .
Установка Android SDK, NDK, JDK, CMake и т.д.
8Observer8 25.01.2026
Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru