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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Hixon10
6 / 6 / 0
Регистрация: 17.10.2011
Сообщений: 153
#1

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

07.12.2011, 19:07. Просмотров 417. Ответов 3
Метки нет (Все метки)

Добрый день. Есть функция 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();
}
В данном примере функция второй раз работает уже неправильно.
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.12.2011, 19:07
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Второй раз функция отрабатывает неверно [С++] (C++):

Функция sin.getline не вызывается во второй раз - C++
Возникли проблемы с функцией sin.getline(). Имеется заголовочный файл: #ifndef GOLF #define GOLF const int Len = 40; struct...

Один раз, - нет! Второй, - да! - C++
Почему в 77-ой строке не осуществляется вызов оператора индексирования, а в 88-ой, - осуществляется (135-145)? #include &quot;stdafx.h&quot; ...

Не хочет считывать второй раз строку - C++
Здравствуйте, пытаюсь работать с базой данных (в виде *.txt). Мои действия: 1) создаем базу данных, название вводим без &quot;.txt&quot; (1) 2)...

Named Pipe клиент не подключается второй раз - C++
Нужно сделать клиент-сервер процессы, клиент запускается, шлет на сервер привет мир и падает себе. Потом снова запускается, шлет привет,...

Сколько раз второй прямоугольник можно вписать в первый. - C++
Описать функцию, которая принимает высоту и ширину прямоугольника. а так же высоту и ширину второго прямоугольника. Возвращает сколько раз...

Ошибка в коде(программа с вектором выполняется только если вызвать её второй раз) - C++
Сабж: void vInit() { vInputing(); // Ввод вектора vector_Size = names.size(); cout &lt;&lt; &quot;Ваш вектор names: &quot;...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
DU
1483 / 1059 / 45
Регистрация: 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
1483 / 1059 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
07.12.2011, 21:51 #4
собрал. тьмя ворнингов. ну да ладно. у меня ни разу не отработала нормально.
вот почитайте вот этот топик для справки. вы пытаетесь модифицировать константу и результат этой операции неопределен. возможно это лишь одна из причин ваших проблем

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

похоже у вас там все очень плохо. если исправить литералы на массивы, все равно программа крашится даже на одном вызове. где-то корраптится память, где-то что-то вы не туда пытаетесь записать. посмотрите на пункт 2, который я в первом сообщении написал.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.12.2011, 21:51
Привет! Вот еще темы с ответами:

Повторяющиеся числа из первого массива занести во второй массив только один раз - C++
#include &lt;stdio.h&gt; #include &lt;iostream&gt; #include &lt;cstdlib&gt; using namespace std; int main(int argc, char* argv) { int...

Функция с двумя аргументами. Второй - по умолчанию = первому - C++
Привет! Есть какой - то способ задать в объявление функции второй аргумент, равный по умолчанию первому? :rofl: void...

Почему после передачи в функцию массива, выводятся случайные числа, если использовать *(uk++) во второй раз? - C++
Интересует такой вопрос: почему после передачи в функцию массива выводятся случайные числа из оперативной памяти, если использовать *(uk++)...

Функция не вызывается 2й раз - C++
Смысл - функция служит для записи числа в массив (посимвольно) int* read(int mass) { int i=0; while(cin&gt;&gt;mass) { i++;...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
07.12.2011, 21:51
Ответ Создать тему
Опции темы

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