Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
Hixon10
6 / 6 / 0
Регистрация: 17.10.2011
Сообщений: 137
07.12.2011, 19:07     Второй раз функция отрабатывает неверно [С++] #1
Добрый день. Есть функция char * fun(char *str);, которая выполняет несколько действий над этой строкой. Если вызывать функцию один раз, то она правильно отрабатывает. Однако, если вызывать функцию два раза подряд с разными строками, то иногда она возвращает неверный результат. С чем это может быть связано? Может быть, с тем, что в функции используются некие другие чаровские указатели? Хотя они удаляются в конце программы.
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
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
#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
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
07.12.2011, 21:20     Второй раз функция отрабатывает неверно [С++] #2
это трындец, а не функция.
1. настройте свой редкатор так, чтобы вместо табов вставлялись пробелы. код будет нормально отображаться везде. вы же видите, как криво он тут отображается.
2. рассмотрите вариант использования std:: string вместо динамических массивов символов.
3. разбейте свою толстую ф-ию на маленькие функции.
4. Воспользуйтесь дебагером для нахождения проблемы.

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

С чего вы взяли что если ее вызвать второй раз, то она некорректно работает? Если судить по коду, то она вызывается каждый раз с разными входящими строками. Если так, то это ф-ия кривая и она корректно работает лишь с определенным набором входных параметров.
Если запустить функцию один раз с вторым аргументом - результат верный, если с первым - результат верный, а если запустить их подряд, то второй результат уже неправильный.
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
07.12.2011, 21:51     Второй раз функция отрабатывает неверно [С++] #4
собрал. тьмя ворнингов. ну да ладно. у меня ни разу не отработала нормально.
вот почитайте вот этот топик для справки. вы пытаетесь модифицировать константу и результат этой операции неопределен. возможно это лишь одна из причин ваших проблем

http://www.cyberforum.ru/cpp-beginne...ead399873.html

похоже у вас там все очень плохо. если исправить литералы на массивы, все равно программа крашится даже на одном вызове. где-то корраптится память, где-то что-то вы не туда пытаетесь записать. посмотрите на пункт 2, который я в первом сообщении написал.
Yandex
Объявления
07.12.2011, 21:51     Второй раз функция отрабатывает неверно [С++]
Ответ Создать тему
Опции темы

Текущее время: 10:06. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru