Как преобразовать код для n-ого количества строк?

@Glebordan · Регистрация: 07.04.2014

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

Всем доброго дня, у меня есть проблема с кодом, не получается вот что, сделать так чтобы частоты символом считались для всего текста, а не для строки отдельно. Есть код:

C++

#include "stdafx.h"
#include <iostream>
#include <string.h>
#include <stdio.h>
#include <Windows.h>
#include <stdlib.h>
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "RUS");
    int i, j, n, len = 0, m, p = 0, y;
 
    char text[1000] = { 0 };
    char find[1000] = { 0 };
    char *sub = NULL;
    cout << "Введите кол-во строк:";
    cin >> m;
    j = 0;
    for (y = 0; y <= m; y++)
    {
        cin.getline(text, 999);
        len = strlen(text);
        p += len;
 
        for (i = 0; text[i] != '\0'; i++)
        {
            if (!strchr(find, text[i]))
            {
                find[j] = text[i];
                j = j + 1;
            }
        }
 
 
        for (n = j = 0; find[j] != '\0'; j++)
        {
            if (sub = strchr(text, find[j]))
            {
                cout << "Символ:" << find[j] << " ";
                for (n = 1; sub != NULL; n++)
                {
                    if (sub + 1)
                        sub = strchr(sub + 1, find[j]);
                }
                cout << "Встречается раз:" << (n - 1) << " "
                    << "Его частота в строке:" << (100.0*(n - 1)) / p << "%" << endl;
            }
        }p = 0;
    }
    cout << "Общее количество элементов:" << p;
 
    system("pause");
    return 0;
}

@jurok_85 · 09.07.2014, 15:16

C++

#include <iostream>
#include <string.h>
#include <stdio.h>
 
#include <stdlib.h>
using namespace std;
 
int main()
{
 
    int i, j, n, len = 0, m, p = 0, y;
 
 
    char find[1000] = { 0 };
    char *sub = NULL;
    cout << "Vvedite kolichestvo strok:";
    cin >> m;
    char text[m][1000] ;
    j = 0;
    for (y = 0; y <= m; y++)
    {
        cin.getline(text[y], 999);
        len = strlen(text[y]);
        p += len;
 
        for (i = 0; text[y][i] != '\0'; i++)
        {
            if (!strchr(find, text[y][i]))
            {
                find[j] = text[y][i];
                j = j + 1;
            }
        }
    }
bool flag;
 
       for (j = 0; find[j] != '\0'; j++)
        {
            flag = false;
            n = 0;
            for (y = 0; y <= m; y++)
            {
                if ((sub = strchr(text[y], find[j])))
                {
                   flag = true;
                    while(sub != NULL)
                    {
                        n++;
                        sub = strchr(sub + 1, find[j]);
                    }
 
 
                }
            }
            if (flag)
            {
 
                cout << "Simvol:" << find[j] << " ";
                cout << "Vstrechaetsa:" << (n ) << " "
                    << "Ego chastota v stroke:" << (100.0*(n )) / p << "%" << endl;
            }
        }
 
    cout << "obwee kolichestvo simvolov : " << p;
 
    system("pause");
    return 0;
}

@Glebordan · 09.07.2014, 16:27 **[ТС]**

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

@jurok_85 · 09.07.2014, 16:35

как не решил?!

Сообщение от Glebordan

сделать так чтобы частоты символом считались для всего текста, а не для строки отдельно

ну программа считает частоты символов для всего текста?

@Glebordan · 09.07.2014, 16:40 **[ТС]**

Ты ведь создаёшь динамический массив, там нужно освобождать память. Просто если ты хотел на это намекнуть, то спасибо

Добавлено через 1 минуту
Чувак ты молодец, прости что бочку на тебя катил... там надо было чуть-чуть подправить

C++

#include "stdafx.h"
#include <iostream>
#include <string.h>
#include <stdio.h>
#include <Windows.h>
#include <stdlib.h>
using namespace std;
 
int main()
{
 
    int i, j, n, len = 0, m, p = 0, y;
 
 
    char find[1000] = { 0 };
    char *sub = NULL;
    cout << "Vvedite kolichestvo strok:";
    cin >> m;
    char text[10][1000];
    j = 0;
    for (y = 0; y <= m; y++)
    {
        cin.getline(text[y], 999);
        len = strlen(text[y]);
        p += len;
 
        for (i = 0; text[y][i] != '\0'; i++)
        {
            if (!strchr(find, text[y][i]))
            {
                find[j] = text[y][i];
                j = j + 1;
            }
        }
    }
    bool flag;
 
    for (j = 0; find[j] != '\0'; j++)
    {
        flag = false;
        n = 0;
        for (y = 0; y <= m; y++)
        {
            if ((sub = strchr(text[y], find[j])))
            {
                flag = true;
                while (sub != NULL)
                {
                    n++;
                    sub = strchr(sub + 1, find[j]);
                }
 
 
            }
        }
        if (flag)
        {
 
            cout << "Simvol:" << find[j] << " ";
            cout << "Vstrechaetsa:" << (n) << " "
                << "Ego chastota v stroke:" << (100.0*(n)) / p << "%" << endl;
        }
    }
 
    cout << "obwee kolichestvo simvolov : " << p;
 
    system("pause");
    return 0;
}

@jurok_85 · 09.07.2014, 16:45

ты опять неправильно понял

C++

1 2	cout << "Vvedite kolichestvo strok:"; cin >> m;

здесь ты задаешь размер массива
то есть сколько строк у тебя будет в тексте
если не знаешь скока строк то лучше std::vector вместо массива использовать

@Glebordan · 09.07.2014, 16:48 **[ТС]**

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

Да и я на сколько помню лекции и практики это динамический массив и для него надо место освободить

Т.к. компилятор заранее не знат сколько места надо под массив

Если я что путаю то поправь меня

@Kerry_Jr · 09.07.2014, 17:04

Glebordan, если поможет

C++

#include <iostream>
#include <windows.h>
#include <cstring>
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    int symbol[256] = {0};
    char text[1000];
    int count = 0, n;
    std::cout << "Введите количество строк: ";
    std::cin >> n;
    std::cin.ignore();
    for (int i = 0; i < n; i++)
    {
        std::cin.getline(text, 999);
        for (int j = 0; j < strlen(text); j++)
            symbol[(unsigned char)text[j]]++;
    }
    for (int i = 0; i < 256; i++)
    {
        if (symbol[i] != 0)
            std::cout << "Символ '" << (unsigned char)i << "' встречается в тексте " << symbol[i] << " раз" << std::endl; 
    }
    
    return 0;
}

@Glebordan · 09.07.2014, 17:12 **[ТС]**

Нет, больше не надо, предыдущий вариант работает, если ввести констану

Но и за твой ответ спасибо

@Glebordan 0 / 0 / 0 Регистрация: 07.04.2014 Сообщений: 37
	09.07.2014, 16:27 [ТС]	3
	Это конечно приятно, но так и я могу. Ты ведь просто приписал строковый индекс к массиву и подредактировал потом немного, но проблемы ты этим не решил 0

@jurok_85 365 / 321 / 219 Регистрация: 21.02.2013 Сообщений: 756
	09.07.2014, 16:35	4
	как не решил?! Сообщение от Glebordan сделать так чтобы частоты символом считались для всего текста, а не для строки отдельно ну программа считает частоты символов для всего текста? 0

@Glebordan 0 / 0 / 0 Регистрация: 07.04.2014 Сообщений: 37
	09.07.2014, 16:48 [ТС]	7
	Да ну... просто компилятор мне говорит, что надо освобождать память Да и я на сколько помню лекции и практики это динамический массив и для него надо место освободить Т.к. компилятор заранее не знат сколько места надо под массив Если я что путаю то поправь меня 0

@Glebordan 0 / 0 / 0 Регистрация: 07.04.2014 Сообщений: 37
	09.07.2014, 17:12 [ТС]	9
	Нет, больше не надо, предыдущий вариант работает, если ввести констану Но и за твой ответ спасибо 0

Как преобразовать код для n-ого количества строк?

Решение