Опять они ... классы

@gorgutz1234 · Регистрация: 02.03.2010

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

Привет всем!Мне очень нужна ваща помощь в исправлении ошибок в классе.
Вот сам класс

C++

#pragma argsused
#include <iostream.h>
#include <conio.h>
#include <string.h>
class _string
  { protected:
  char *str;
  int len;
  public:
  _string()
  {
   str=new char;    //constructor without parametrs
   str[0]='\0';
   len=0;
  }
 
  _string(char *st)
  {int i=0;
   while (st[i]!='\0')   //constructor with C string
   i++;
   len=i;
   str=new char[i];
   strcpy(str,st);
   }
 
  _string(char st)         //construcnor for symbol
  {   str=new char;
      str[0]=st;
      str[1]='\0';
      len=1;
   }
 
  _string(_string &str1)  //copy constructor
 {   str=new char;
     len=str1.len;
     strcpy(str,str1.str);
 }
 void show ()
 {
  cout<<str<<"   "<<len<<endl;
 }
 int dlina(char *st)
 {
 int i;
 while (st[i]!='\0')
 i++;
 len=i;
 return len;
  }
 char *clear(char *st)   //function clear
    {
    st[0]='\0';
     strcpy(str,st);
     return str;
     }
 };
 //~_string() { delete str; len=0;}  //destructor
 
 
 
  class bit_string: public _string    //child
  {
  public:
  bit_string()
  {                //constructor  without parametrs
  _string();
  }
 
  bit_string(char *st):_string(st)
  {
 
     int i=0;
 
     while (str[i]!='\0')
        {if ((str[i]!='0') && (st[i]!='1')&& (st[i]!='-')) {
                                             str[0]='\0';
                                             len=0;
                                             break;
                                            }
                                            i++;
         }
   }
 
   bit_string(char st):_string(st)         //construcnor for symbol
   {
   //_string(st)
   if  ((str[0]!='0') && (str[0]!='1')) {str[0]='\0';
                                len=0; }
   }
   bit_string(bit_string &str1):_string(str1)  //copy constructor
 {   str=new char;
     len=str1.len;
     strcpy(str,str1.str);
 }
  
  // ~string() { delete str; }; 
 
    char *negative_number ()
     { int *mas;
        mas=new int [10];                              //convert to negative number
        int i,l,dl,smart;
      if (str[0]=='-') {i=1;
                      while (str[i]!='\0') {
                                         if (str[i]=='0') str[i]='1';
                                         if (str[i]=='1') str[i]='0';
                                         i++;
                                         l+=i;
                                         }
                        i=0;
                        int a,b,c,s;
                        c=atoi(str);
                        while (b>0)
                        {
                        a=c % 10;
                        b=c / 10;
                        c=b;
                        mas[l]=a;
                        l=l-1;
                        }
                        smart=0;
                        s=0;
                        for (i=l;i>0;i--)
                        {
                        s=mas[i]+1;
                        if (s=1) {mas[i]=1; smart=0;}
                        if (s=2) {mas[i]=0; smart=1;}
                        if (s=3) {mas[i]=1; smart=1;}
                        if (smart==0) break;
                        }
                       }
        while (l>i)
     {itoa( mas[i],str,10);
     return str;
     };
 
 bit_string operator == (bit_string *str1)
   {
    if (strcmp(str,str1->str))
      return false;
     };
 bit_string operator = (bit_string *str1)
 {
 };
 bit_string operator + (bit_string *str1)
 { int mas1,mas2,a1,a2,b1,b2,s,sn=0,l1,l2,i,j,l;
   int *m1,*m2,*m;
   m1=new int [10];
   m2=new int [10];
   m=new int [10];
   mas1=atoi (str1->str);
   mas2=atoi (str);
    l1=strlen(str1->str);
    l2=strlen(str);
   l=l1+l2;
   sn=0;
   for (i=l;l-1<i;i++)
   {
   a1=mas1 % 10;       // mod
   a2=mas2 % 10;
   b1=mas1 / 10;       // div
   b2=mas2 / 10;
   mas1=b1;
   mas2=b2;
   m1[l1]=a1;
   m2[l2]=a2;
   l1--;
   l2--;
   }
   for (i=l;i>0;i--)
   {
   s=m2[l2]+m1[l1]+sn;
   if (s==0) {m[i]=0; sn=0;}
   if (s==1) {m[i]=1; sn=0;}
   if (s==2) {m[i]=0; sn=1;}
   if (s==3) {m[i]=1; sn=1;}
   l1--;
   l2--;
   }                   i=0;
   while (l>i)
     {itoa( m[i],str,10);
                                   i++;}
     return str;
 }
 };
 
int main(int argc, char* argv[])
{
 bit_string s1;
 s1.show();
 bit_string s2("-0101");
 s2.negative_number();
 s2.show();
 bit_string s3(s2);
 s3.show();
 bit_string s4('k');
 s4.show();
 getch();
 
 return 0;

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

@OstapBender · 31.05.2011, 17:47

C

    char *negative_number ()
     { int *mas;
        mas=new int [10];                              //convert to negative number
        int i,l,dl,smart;
      if (str[0]=='-') {i=1;
                      while (str[i]!='\0') {
                                         if (str[i]=='0') str[i]='1';
                                         if (str[i]=='1') str[i]='0';
                                         i++;
                                         l+=i;
                                         }
                        i=0;
                        int a,b,c,s;
                        c=atoi(str);
                        while (b>0)
                        {
                        a=c % 10;
                        b=c / 10;
                        c=b;
                        mas[l]=a;
                        l=l-1;
                        }
                        smart=0;
                        s=0;
                        for (i=l;i>0;i--)
                        {
                        s=mas[i]+1;
                        if (s=1) {mas[i]=1; smart=0;}
                        if (s=2) {mas[i]=0; smart=1;}
                        if (s=3) {mas[i]=1; smart=1;}
                        if (smart==0) break;
                        }
                       }
        while (l>i)
     {itoa( mas[i],str,10);
     return str;
     };

что это???

в сроке str содержится число и нужно его перевести в доп-код?

Добавлено через 14 минут

C

    char str[32+1]="10010100";
 
    puts(str);
 
    int val;
    char *p;
 
    val=strtol(str,&p,2);
 
    val=-val;
 
    itoa(val,str,2);
 
    puts(str);

возможно так

@gorgutz1234 · 01.06.2011, 08:49 **[ТС]**

str это переменная типа класса.я ее там с помощью дивов и модов переводил в массив

Добавлено через 2 минуты
и еще что еще за пут я просто новичок в с++ если есть возможность то поясняйте что пишите.И еще если возникнут вопросы по коду то я поясню что и как я там делал

Добавлено через 2 минуты
люди хелп!!!

Добавлено через 13 часов 25 минут
плз хоть кто-нибудь

@gorgutz1234 · 02.06.2011, 17:30 **[ТС]**

Вот вроде я тут пофиксил но помощь еще нужна.Плз кто нибудь посматрите у меня тут ошибка(и) в синтаксисе я незнаю что исправить чтоб заработало.вот переделаный код

C++

//---------------------------------------------------------------------------
 
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
#include <iostream.h>
#include <conio.h>
#include <string.h>
class _string
  { protected:
  char *str;
  int len;
  public:
  _string()
  {
   str=new char;    //constructor without parametrs
   str[0]='\0';
   len=0;
  }
 
  _string(char *st)
  {int i=0;
   while (st[i]!='\0')   //constructor with C string
   i++;
   len=i;
   str=new char[i];
   strcpy(str,st);
   }
 
  _string(char st)         //construcnor for symbol
  {   str=new char;
      str[0]=st;
      str[1]='\0';
      len=1;
   }
 
  _string(_string &str1)  //copy constructor
 {   str=new char;
     len=str1.len;
     strcpy(str,str1.str);
 }
 void show ()
 {
  cout<<str<<"   "<<len<<endl;
 }
 int dlina(char *st)
 {
 int i;
 while (st[i]!='\0')
 i++;
 len=i;
 return len;
  }
 char *clear(char *st)   //function clear
    {
    st[0]='\0';
     strcpy(str,st);
     return str;
     }
 };
~_string() { delete str; len=0;}  //destructor
 
 
 
  class bit_string: public _string    //child
  {
  public:
  bit_string()
  {                //constructor  without parametrs
  _string();
  }
 
  bit_string(char *st):_string(st)
  {
 
     int i=0;
 
     while (str[i]!='\0')
        {if ((str[i]!='0') && (st[i]!='1')&& (st[i]!='-')) {
                                             str[0]='\0';
                                             len=0;
                                             break;
                                            }
                                            i++;
         }
   }
 
   bit_string(char st):_string(st)         //construcnor for symbol
   {
   if  ((str[0]!='0') && (str[0]!='1')) {str[0]='\0';
                                len=0; }
   }
   bit_string(bit_string &str1):_string(str1)  //copy constructor
 {   str=new char;
     len=str1.len;
     strcpy(str,str1.str);
 }
  
~ bit_string() { } //destractor
 
    char *negative_number ()
     { int *m;
         m=new int [10];                              //convert to negative number
        int i,l,dl,smart,k=0;
      if (str[0]=='-') {i=1;  l=0; str[0]='0';
                      while (str[i]!='\0') {
                                         if (str[i]=='0') {str[i]='1';}
                                         else  {str[i]='0';}
                                         i++;
                                         l=i;
                                         }
                        int a,b,c,s;
                        c=atoi(str);
                        l--;
                        while (b!=0)
                        {
                        a=c % 10;    //mod
                        b=c / 10;    //div
                        c=b;
                        m[l]=a;
                        l--;
                        }
                        smart=1;
                        s=0;
                        l=i-1;
                        for (i=l;i>0;i--)
                        {
                        s=m[i]+1;
                        if (s==1)
                          {m[i]=1;
                           smart=0;}
                        if (s==2)
                           {m[i]=0;
                            smart=1;}
                        if (s==3)
                           {m[i]=1;
                            smart=1;}
                        if (smart==0) break;
                        }
                       }
        for (i=0;i<l;i++)
        {
        if (m[i]!=1)
        k++;
        else break;  
        }
        int c,a;
        c=0;
 
        while (l>=k)
        {
        c+=m[k];
        if (l==k) break;
        c*=10;
        k++;
        }
     itoa(c,str,10);
     i++;
 
     return str;
 
     };
 
 bit_string operator == (bit_string *str1)
   {
    if (strcmp(str,str1->str))
      return false;
     };
 
 
 bit_string operator + (bit_string *str1)
 { int mas1,mas2,a1,a2,b1,b2,s,sn=0,l1,l2,i,j,l,k;
   int m1[5],m2[5],m[5];
   mas1=atoi (str1->str);
   mas2=atoi (str);
    l1=strlen(str1->str);
    l2=strlen(str);
   l=l1+l2;
   sn=0;
   for (i=l;0<i;i++)
   {
   a1=mas1 % 10;       // mod
   a2=mas2 % 10;
   b1=mas1 / 10;       // div
   b2=mas2 / 10;
   mas1=b1;
   mas2=b2;
   m1[l1]=a1;
   m2[l2]=a2;
   l1--;
   l2--;
   }
   for (i=l;i>0;i--)
   {
   s=m2[l2]+m1[l1]+sn;
   if (s==0) {m[i]=0; sn=0;}
   if (s==1) {m[i]=1; sn=0;}
   if (s==2) {m[i]=0; sn=1;}
   if (s==3) {m[i]=1; sn=1;}
   l1--;
   l2--;
   }
    for (i=0;i<l;i++)
        {
        if (m[i]!=1)
        k++;
        else break;  
        }
        int c,a;
        c=0;
 
        while (l>=k)
        {
        c+=m[k];
        if (l==k) break;
        c*=10;
        k++;
        }
     itoa(c,str,10);
     i++;
     return str;
 }
 };
 
int main(int argc, char* argv[])
{
 bit_string s1;
 s1.show();
 bit_string s2("-01010010");
 s2.negative_number();
 s2.show();
 bit_string s3(s2);
 s3.show();
 bit_string s4('k');
 s4.show();
 bit_string s5("100");
 bit_string s6;
 bit_string s7("100");
 s6=s5+s7;
 s6.show();
 getch();
 
 return 0;
}
//-------------------------------------------------------------------------

@grizlik78 · 02.06.2011, 17:44

Посмотрел конструкторы только. Практически везде память выделяется неверно.
В первом конструкторе выделяется память под переменную, а надо под массив из 1 элемента, т.е.

C++

1	str=new char[1];

Во втором не выделяется память для хранения завершающего нуля, а функция strcpy этот ноль записывает, портя чужую память. Надо

C++

1	str=new char[i+1];

В третьем и четвёртом снова та же ошибка, что и в первом. Нужно создать массивы с размерами 2 и str1.len+1 соответственно.

Добавлено через 1 минуту
И в деструкторе, соответственно, надо удалять массив:

C++

1	~_string() { delete [] str; len=0;} //destructor

@gorgutz1234 · 02.06.2011, 17:54 **[ТС]**

я канечно незнаю но препод конструкторы смотрел сказал мол правильно.Хотя может он проглядел сейчас исправлю.

Добавлено через 4 минуты
тем боле я вызывал их и она работали.Кста-ти посматри уж ошиббки остальные.Очень буду признателен.

@grizlik78 · 02.06.2011, 18:11

Ну первый-то, в принципе, нормально работать будет, а три других память портят однозначно.

Добавлено через 17 минут
Деструктор вылез за объявление класса. Строки 63 и 64 поменять.
operator == в строке 167 должен возвращать bool.
Затем, в аргументах операторов == и + используется указатель. Это так надо? Мне кажется там нужна константная ссылка. Ну или не константная, хотя константная лучше.

@gorgutz1234 · 02.06.2011, 18:16 **[ТС]**

в них динамические должны быть(препод настаивает говорит типа это хороший стиль програмирования) у + ==.А насчет деструктора я растяпа)).Щас исправлю.А там еще были какие то ошибки.Вот с операторами + и(насчет равно неуверен).

@grizlik78 · 02.06.2011, 18:17

Кто динамический должен быть?

@gorgutz1234 · 02.06.2011, 18:18 **[ТС]**

"в строке 167 должен возвращать bool".я чтот непонял я же написал стрцмп и ретурн в нем.Что же еще я должен возвратить?

@grizlik78 · 02.06.2011, 18:22

вот заголовок:

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

C++

1	bit_string operator == (bit_string *str1)

первое слово говорит, что возвращать будем bit_string, а возвращаем bool
должно быть

C++

1	bool operator == (bit_string *str1)

Добавлено через 2 минуты
Кстати, а что если строки равны? Тогда вообще ничего не возвращается? А должно возвращаться true

@gorgutz1234 · 02.06.2011, 18:23 **[ТС]**

вот исправил ты это имел ввиду?

C++

bit_string operator == (bit_string *str1)
     {
    if (strcmp(str,str1->str))==0 {return true;}
      return false;
     };

хотя он ругается что я неправильно написал стрцмп.

@grizlik78 · 02.06.2011, 18:26

Вот так:

C++

bool operator == (bit_string *str1)
{
    if (strcmp(str,str1->str) == 0)
        return true;
    return false;
};

@gorgutz1234 · 02.06.2011, 18:30 **[ТС]**

там вот еще ошибка
[C++ Error] Unit1.cpp(243): E2093 'operator+' not implemented in type 'bit_string' for arguments of the same type

Добавлено через 33 секунды
эт когда я делал проверку своего оператора +

@grizlik78 · 02.06.2011, 18:34

Я не уверен, но подозреваю, что это как раз из-за указателя в аргументе вместо ссылки.

@gorgutz1234 · 02.06.2011, 18:46 **[ТС]**

тогда он ругается на [C++ Error] Unit1.cpp(178): E2288 Pointer to structure required on left side of -> or ->*

@grizlik78 · 02.06.2011, 18:47

Ну да, разумеется, вместо xxx->str надо будет использовать xxx.str

@gorgutz1234 · 02.06.2011, 18:57 **[ТС]**

ок все понятн.щас пойду исправлять свой алгоритм.спс

Добавлено через 3 минуты
Вот блин ток спрашивается вопрос зачем нас 1.5 года на паскале мучили после него в с++ перейти сложновато(по крайней мере для меня).

Добавлено через 3 минуты
а вот как перевести из строки по символьно в массив?

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	02.06.2011, 18:34	15
	Я не уверен, но подозреваю, что это как раз из-за указателя в аргументе вместо ссылки. 0

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	01.06.2011, 08:49 [ТС]	3
	str это переменная типа класса.я ее там с помощью дивов и модов переводил в массив Добавлено через 2 минуты и еще что еще за пут я просто новичок в с++ если есть возможность то поясняйте что пишите.И еще если возникнут вопросы по коду то я поясню что и как я там делал Добавлено через 2 минуты люди хелп!!! Добавлено через 13 часов 25 минут плз хоть кто-нибудь 0

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 17:54 [ТС]	6
	я канечно незнаю но препод конструкторы смотрел сказал мол правильно.Хотя может он проглядел сейчас исправлю. Добавлено через 4 минуты тем боле я вызывал их и она работали.Кста-ти посматри уж ошиббки остальные.Очень буду признателен. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	02.06.2011, 18:11	7
	Ну первый-то, в принципе, нормально работать будет, а три других память портят однозначно. Добавлено через 17 минут Деструктор вылез за объявление класса. Строки 63 и 64 поменять. operator == в строке 167 должен возвращать bool. Затем, в аргументах операторов == и + используется указатель. Это так надо? Мне кажется там нужна константная ссылка. Ну или не константная, хотя константная лучше. 1

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 18:16 [ТС]	8
	в них динамические должны быть(препод настаивает говорит типа это хороший стиль програмирования) у + ==.А насчет деструктора я растяпа)).Щас исправлю.А там еще были какие то ошибки.Вот с операторами + и(насчет равно неуверен). 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	02.06.2011, 18:17	9
	Кто динамический должен быть? 0

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 18:18 [ТС]	10
	"в строке 167 должен возвращать bool".я чтот непонял я же написал стрцмп и ретурн в нем.Что же еще я должен возвратить? 0

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 18:30 [ТС]	14
	там вот еще ошибка [C++ Error] Unit1.cpp(243): E2093 'operator+' not implemented in type 'bit_string' for arguments of the same type Добавлено через 33 секунды эт когда я делал проверку своего оператора + 0

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 18:46 [ТС]	16
	тогда он ругается на [C++ Error] Unit1.cpp(178): E2288 Pointer to structure required on left side of -> or ->* 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	02.06.2011, 18:47	17
	Ну да, разумеется, вместо xxx->str надо будет использовать xxx.str 1

@gorgutz1234 3 / 3 / 1 Регистрация: 02.03.2010 Сообщений: 75
	02.06.2011, 18:57 [ТС]	18
	ок все понятн.щас пойду исправлять свой алгоритм.спс Добавлено через 3 минуты Вот блин ток спрашивается вопрос зачем нас 1.5 года на паскале мучили после него в с++ перейти сложновато(по крайней мере для меня). Добавлено через 3 минуты а вот как перевести из строки по символьно в массив? 0