Второй раз функция отрабатывает неверно [С++]

@Hixon10 · Регистрация: 17.10.2011

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

Добрый день. Есть функция char * fun(char *str);, которая выполняет несколько действий над этой строкой. Если вызывать функцию один раз, то она правильно отрабатывает. Однако, если вызывать функцию два раза подряд с разными строками, то иногда она возвращает неверный результат. С чем это может быть связано? Может быть, с тем, что в функции используются некие другие чаровские указатели? Хотя они удаляются в конце программы.

C++

#include <iostream.h>
#include <conio.h>
#include <sstream>
#pragma hdrstop
 
using namespace std;
 
 
 
char* DelChar(char* str, size_t n)
{
  int len = strlen(str)-n;
  if(len>0)
    memmove(str+n, str+n+1, len);
 
  return str;
}
 
 
char *getOctCharacter(char *str)
{    
    if(strcmp(str, "111")== 0) return "7";
    else if(strcmp(str, "110")== 0) return "6";
    else if(strcmp(str, "101")== 0) return "5";
    else if(strcmp(str, "100")== 0) return "4";
    else if(strcmp(str, "011")== 0) return "3";
    else if(strcmp(str, "010")== 0) return "2";
    else if(strcmp(str, "001")== 0) return "1";
    else if(strcmp(str, "000")== 0) return "0";
    else if(strcmp(str, "11")== 0) return "3";
    else if(strcmp(str, "10")== 0) return "2";
    else if(strcmp(str, "01")== 0) return "1";
    else if(strcmp(str, "00")== 0) return "0";
    else if(strcmp(str, "1")== 0) return "1";
    else if(strcmp(str, "0")== 0) return "0";
}
 
char * perevesti(char *rowresult)
{
    char * endresult = "";
        char * dd = new char [80];
        strcpy(dd, "");
        char *k = new char [80];
        strcpy(k, "");
        int sign = 0;
        int sign2 = 0;
        int a=0;
        int m = 0, n4 = 0;
        int minus = 0;
        int null = 0;
        double result = 0;
        char *result2 = new char [16];
        strcpy(result2, "");
 
        if (rowresult[0] == '-' ) {
            sign = 0;
            minus = 1;
            rowresult = DelChar(rowresult, 0);
        }
 
        char *rowresult2 = new char [17];
        for (int i=0; i<17; i++ ) rowresult2[i] = '0';
 
        for( char *d3 = strtok(rowresult, "."); d3; d3 = strtok(NULL, ".") )  {
    if (a == 0) {
                m = 0;
                for (int i=8-strlen(d3); i<8; i++) {
                        rowresult2[i] = d3[m++];
                }
                a = a + 1;
                rowresult2[8] = '.';
        } else if (a == 1) {
                m = 0;
                for (int i = 9; i<9+strlen(d3); i++) {
                        rowresult2[i] = d3[m++];
                }
        }
        }
 
 
        a=0;
 
        for( char *d = strtok(rowresult2, "."); d; d = strtok(NULL, ".") )  {
    if (a == 0) {
 
 
                null = 0;
 
 
                for (int e=1; e<8; e++) {
                        if (d[e] == '0') null++;
                }
 
 
 
 
                if (null == 7) strcpy(d, "000");
                result2 = strcat(result2,d);
 
        a = a + 1;
 
    } else if (a == 1) {
 
                n4 = 0;
                for (int f=0; f<8; f++) {
                        if (d[f] == '0') n4++;
                }
 
                result2 = strcat(result2,d);
     }
      }
 
                if (null < 7) {
                        stringstream ss2 (stringstream::in | stringstream::out);
                        ss2<<"0"<<result2<<"0";
                        ss2>>result2;
                }
 
 
        a=0;
 
        strcpy(endresult, "");
 
    for(int i = 0; i < strlen(result2); i = i+3)
    {
                strcpy(dd, "");
                strcpy(k, "");
                for (int j=0; j<3; j++) {
                     if (result2[i+j] == '0')  strcpy(k, "0");
                     else if (result2[i+j] == '1') strcpy(k, "1");
                     dd = strcat (dd, k);
                     strcpy(k, "");
                }
                endresult =  strcat(endresult, getOctCharacter(dd));
                if ( (a == 2) && (null != 7) ) {
                    endresult =  strcat(endresult, ".");
                }
                                if (a == 0 && null==7) endresult =  strcat(endresult, ".");
                a = a + 1;
    }
 
   int k1=0;
   for (int i=0; i<strlen(endresult); i++) {
        if (endresult[i] == '0') k1++;
        else break;
   }
 
   if (endresult[1] == '.') k1--;
 
   for (int i=0; i<k1; i++) {
        endresult = DelChar(endresult, 0);
   }
 
   if (endresult[strlen(endresult)-1] == '0' && endresult[strlen(endresult)-2] == '0' && endresult[strlen(endresult)-3] == '0' && endresult[strlen(endresult)-4] == '.') {
        endresult = DelChar(endresult, strlen(endresult)-1);
        endresult = DelChar(endresult, strlen(endresult)-1);
        endresult = DelChar(endresult, strlen(endresult)-1);
        endresult = DelChar(endresult, strlen(endresult)-1);
   }
 
 
        if(minus == 1) {
                stringstream ss (stringstream::in | stringstream::out);
                                ss<<"-"<<endresult;
                ss>>endresult;
        }
 
 
    delete dd;
    delete k;
    delete result2;
    delete rowresult2;
 
 
 
    char *itog = new char [strlen(endresult) + 1];
    strcpy(itog, endresult);
    delete endresult;
 
 
    return itog;
}
 
int main()
{
    char * rowresult = "0.101001";
    cout << rowresult << " in hex is " << perevesti(rowresult) << endl;
 
char * rowresult2 = "0.11001";
    cout << rowresult2 << " in hex is " << perevesti(rowresult2) << endl;
 
    getch();
}

В данном примере функция второй раз работает уже неправильно.

@DU · 07.12.2011, 21:20

это трындец, а не функция.
1. настройте свой редкатор так, чтобы вместо табов вставлялись пробелы. код будет нормально отображаться везде. вы же видите, как криво он тут отображается.
2. рассмотрите вариант использования std:: string вместо динамических массивов символов.
3. разбейте свою толстую ф-ию на маленькие функции.
4. Воспользуйтесь дебагером для нахождения проблемы.

С чего вы взяли что если ее вызвать второй раз, то она некорректно работает? Если судить по коду, то она вызывается каждый раз с разными входящими строками. Если так, то это ф-ия кривая и она корректно работает лишь с определенным набором входных параметров.

@Hixon10 · 07.12.2011, 21:24 **[ТС]**

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

это трындец, а не функция.
1. настройте свой редкатор так, чтобы вместо табов вставлялись пробелы. код будет нормально отображаться везде. вы же видите, как криво он тут отображается.
2. рассмотрите вариант использования std:: string вместо динамических массивов символов.
3. разбейте свою толстую ф-ию на маленькие функции.
4. Воспользуйтесь дебагером для нахождения проблемы.

С чего вы взяли что если ее вызвать второй раз, то она некорректно работает? Если судить по коду, то она вызывается каждый раз с разными входящими строками. Если так, то это ф-ия кривая и она корректно работает лишь с определенным набором входных параметров.

Если запустить функцию один раз с вторым аргументом - результат верный, если с первым - результат верный, а если запустить их подряд, то второй результат уже неправильный.

@DU · 07.12.2011, 21:51

собрал. тьмя ворнингов. ну да ладно. у меня ни разу не отработала нормально.
вот почитайте вот этот топик для справки. вы пытаетесь модифицировать константу и результат этой операции неопределен. возможно это лишь одна из причин ваших проблем

https://www.cyberforum.ru/cpp-... 99873.html

похоже у вас там все очень плохо. если исправить литералы на массивы, все равно программа крашится даже на одном вызове. где-то корраптится память, где-то что-то вы не туда пытаетесь записать. посмотрите на пункт 2, который я в первом сообщении написал.

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	07.12.2011, 21:20	2
	это трындец, а не функция. 1. настройте свой редкатор так, чтобы вместо табов вставлялись пробелы. код будет нормально отображаться везде. вы же видите, как криво он тут отображается. 2. рассмотрите вариант использования std:: string вместо динамических массивов символов. 3. разбейте свою толстую ф-ию на маленькие функции. 4. Воспользуйтесь дебагером для нахождения проблемы. С чего вы взяли что если ее вызвать второй раз, то она некорректно работает? Если судить по коду, то она вызывается каждый раз с разными входящими строками. Если так, то это ф-ия кривая и она корректно работает лишь с определенным набором входных параметров. 1

@Hixon10 6 / 6 / 0 Регистрация: 17.10.2011 Сообщений: 153
	07.12.2011, 21:24 [ТС]	3
	Сообщение от DU это трындец, а не функция. 1. настройте свой редкатор так, чтобы вместо табов вставлялись пробелы. код будет нормально отображаться везде. вы же видите, как криво он тут отображается. 2. рассмотрите вариант использования std:: string вместо динамических массивов символов. 3. разбейте свою толстую ф-ию на маленькие функции. 4. Воспользуйтесь дебагером для нахождения проблемы. С чего вы взяли что если ее вызвать второй раз, то она некорректно работает? Если судить по коду, то она вызывается каждый раз с разными входящими строками. Если так, то это ф-ия кривая и она корректно работает лишь с определенным набором входных параметров. Если запустить функцию один раз с вторым аргументом - результат верный, если с первым - результат верный, а если запустить их подряд, то второй результат уже неправильный. 0

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	07.12.2011, 21:51	4
	собрал. тьмя ворнингов. ну да ладно. у меня ни разу не отработала нормально. вот почитайте вот этот топик для справки. вы пытаетесь модифицировать константу и результат этой операции неопределен. возможно это лишь одна из причин ваших проблем https://www.cyberforum.ru/cpp-... 99873.html похоже у вас там все очень плохо. если исправить литералы на массивы, все равно программа крашится даже на одном вызове. где-то корраптится память, где-то что-то вы не туда пытаетесь записать. посмотрите на пункт 2, который я в первом сообщении написал. 0