Компилятор QT работает неадекватно ?!

@Corvit · Регистрация: 14.10.2013

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

Доброго времени суток!
Задача простая. Есть класс String предназначенный для хранения строк. Недостаток этого класса в том, что он не контролирует выход за пределы поля str. Для искоренения этого недостатка создан класс Pstring. Должна выполняться проверка размера вводимой строки, перед записью этой строки в поле str. Но в строке 36 компилятор переходит к выполнению конструктора без параметров класса предшественника ( стр. 15), вместо того чтобы вызвать конструктор с одним параметром ( стр. 18). Хотелось бы понять причину

C++

#include <iostream>
using namespace std;
#include <string.h>          
#include <string>
///////////////////////////////////////////////////////////
class String
{
  protected:
    enum { SZ = 80 };        // размер массива
    char str [ SZ ];         // массив для хранения строки
  public:
    // конструктор без параметров
    String ( )
      { str [ 0 ] = '\x0'; }
    // конструктор с одним параметром
    String ( char s [ ] )
      { strcpy ( str, s ); } // сохранение строки в массив
    // показ строки
    void display ( ) const
      { cout << str; }
    // преобразование к стандартному типу
    operator char* ( )
      { return str; }
};
 
// дополнен контролем выхода за пределы массива str
class Pstring: public String
{
public:
    Pstring(): String() {}
    Pstring(char s [ ] ) : String (s)
    {
        if (  strlen( s ) < SZ - 1)
// в этой строке компилятор переходит к вызову конструктора без параметров ???
            String ( s );
        else
        {
            s [ SZ - 1 ] = '\0';
            strcpy ( str ,  s );
        }
    }
};
 
///////////////////////////////////////////////////////////
int main ( )
{
  Pstring s1;                 
 
  char xstr [ ] = "Some string "; 
 
  s1 = xstr;                
 
  s1.display ( );           
 
  Pstring s2 = "Another shtring!"; 
 
  cout << static_cast<char*>( s2 ); 
  cout << endl;
 
  return 0;
}

@ForEveR · 30.10.2013, 13:06

Corvit, Потому что так нельзя по стандарту.
Что делает строчка String(s)? Вызывает конструктор класса String с параметром s, однако объект уже создан в списке инициализации, следовательно для него не будет вызван конструктор.

@castaway · 30.10.2013, 13:07

Вообще то в этой строке (32-й):

C++

1	Pstring(char s [ ] ) : String (s)

@Corvit · 30.10.2013, 13:52 **[ТС]**

castaway

Вообще то в этой строке (32-й):

C++

1	Pstring(char s [ ] ) : String (s)

Опечатался - на самом деле так:

C++

1	Pstring(char s [ ] )

ForEveR

Что делает строчка String(s)? Вызывает конструктор класса String с параметром s, однако объект уже создан в списке инициализации, следовательно для него не будет вызван конструктор.

Ну тогда по идее должен выдать ошибку, а он вместо этого вызывает конструктор без параметров: 52->33->34->36->15->43. Почему он вообще заходит в конструктор без параметров, хотя согласно сигнатуре должен вызвать конструктор с одним параметром

@castaway · 30.10.2013, 13:59

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

Опечатался

Что значит "опечатался"? Покажи код, на котором можно строить предположения.

@Corvit · 30.10.2013, 14:04 **[ТС]**

C++

#include <iostream>
using namespace std;
#include <string.h>          
#include <string>
///////////////////////////////////////////////////////////
class String
{
  protected:
    enum { SZ = 80 };        // размер массива
    char str [ SZ ];         // массив для хранения строки
  public:
    // конструктор без параметров
    String ( )
      { str [ 0 ] = '\x0'; }
    // конструктор с одним параметром
    String ( char s [ ] )
      { strcpy ( str, s ); } // сохранение строки в массив
    // показ строки
    void display ( ) const
      { cout << str; }
    // преобразование к стандартному типу
    operator char* ( )
      { return str; }
};
 
// дополнен контролем выхода за пределы массива str
class Pstring: public String
{
public:
    Pstring(): String() {}
    Pstring(char s [ ] )
    {
        if (  strlen( s ) < SZ - 1)
// в этой строке компилятор переходит к вызову конструктора без параметров ???
            String ( s );
        else
        {
            s [ SZ - 1 ] = '\0';
            strcpy ( str ,  s );
        }
    }
};
 
///////////////////////////////////////////////////////////
int main ( )
{
  Pstring s1;                 
 
  char xstr [ ] = "Some string "; 
 
  s1 = xstr;                
 
  s1.display ( );           
 
  Pstring s2 = "Another shtring!"; 
 
  cout << static_cast<char*>( s2 ); 
  cout << endl;
 
  return 0;
}

@ForEveR · 30.10.2013, 14:07

Corvit, Если код такой:

C++

1	Pstring(char s [ ] )

то тут очевидно будет вызван конструктор без параметров базового класса. Вызова конструктора, находящегося в функции естественно не будет.

@Corvit · 30.10.2013, 14:09 **[ТС]**

то тут очевидно будет вызван конструктор без параметров базового класса.

Так и есть, но в строке

C++

1	String ( s );

опять вызывается конструктор без параметров базового класса

@ForEveR · 30.10.2013, 14:31

Про то что вызова конструктор в функции не будет - это я конечно погорячился. Вызов будет, но он создаст временный объект, который тут же уничтожится. Почему вызывается конструктор без параметров пробую понять.
Насколько я понимаю, компилятор оптимизирует и имеет на это полное право, потому как результат вызова данного конструктора никак не используется. Оптимизация глушится либо

C++

1	String tmp(s);

либо

C++

1	(void)String(s);

@Corvit · 30.10.2013, 14:49 **[ТС]**

Спасибо за разъяснение

@castaway · 30.10.2013, 14:51

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

Так и есть, но в строке

Откуда знаешь что вызов происходит именно в этой строке?
Самый первый конструктор без параметров происходит в этой (48-й строке последнего кода) строке: Pstring s1;

@Corvit · 01.11.2013, 09:57 **[ТС]**

Переписал немного код

C++

#include <iostream>
using namespace std;
#include <string.h>
#include <string>
///////////////////////////////////////////////////////////
class String
{
  protected:
    enum { SZ = 80 };        // размер массива
    char str [ SZ ];         // массив для хранения строки
  public:
    // конструктор без параметров
    String ( )
      { str [ 0 ] = '\x0'; }
    // конструктор с одним параметром
    String ( char s [ ] )
      { strcpy ( str, s ); } // сохранение строки в массив
    // показ строки
    void display ( ) const
      { cout << str; }
    // преобразование к стандартному типу
    operator char* ( )
      { return str; }
};
 
// дополнен контролем выхода за пределы массива str
class Pstring: public String
{
public:
    Pstring(char s [ ] )
    {
        if (  strlen( s ) > SZ - 1)
        {
            for ( int j = 0; j < SZ - 1; j++)
                str [ j ] = s [ j ];
            str [ SZ - 1] = '\0';
        }
        else
        {
            String(s);
        }
    }
};
 
///////////////////////////////////////////////////////////
int main ( )
{
  Pstring s1 = " Some very very big string, which is maybe, or likely for sure"
          " Will exceed size of SZ";
 
  cout <<"\ns1 = "; s1.display();
 
  Pstring s2 = " just a little string";
  cout <<"\ns2 = "; s2.display();
 
  return 0;
}

Инициализация первой переменной происходит без неожиданностей, а вот с определением второй переменной происходит какая-то ахинея.
castaway

Откуда знаешь что вызов происходит именно в этой строке?

Ставлю точки останова на 53-ую строку. Отладчик выполняет программу в следующем порядке: 53->31->14->32->40->14->42.
ForEveR

Насколько я понимаю, компилятор оптимизирует и имеет на это полное право, потому как результат вызова данного конструктора никак не используется. Оптимизация глушится

Почему в таком случае он вообще выполняет по второму кругу конструктор без параметров родительского класса??
Получается, что единственный способ использовать конструктор:

C++

1	Pstring(char s [ ] ) : String ( s )

И если нужно внести малейшие изменения то полностью переписывать конструктор класса наследника, вообще не обращаясь ни к каким конструкторам класса родителя?

Добавлено через 18 часов 10 минут
Люди

ValeryS · 01.11.2013, 10:10

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

C++

if ( strlen( s ) > SZ - 1)
 {
  for ( int j = 0; j < SZ - 1; j++)
     str [ j ] = s [ j ];
  str [ SZ - 1] = '\0';
 }

долго, вот побыстрее

C++

if ( strlen( s ) > SZ - 1)
 {
  memcpy(str,s,SZ);
  str [ SZ - 1] = '\0';
 }

@Corvit · 01.11.2013, 10:12 **[ТС]**

долго, вот побыстрее

Вопрос концептуальный, а не оптимизационный

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	30.10.2013, 13:06	2
	Corvit, Потому что так нельзя по стандарту. Что делает строчка String(s)? Вызывает конструктор класса String с параметром s, однако объект уже создан в списке инициализации, следовательно для него не будет вызван конструктор. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	30.10.2013, 13:59	5
	Сообщение от Corvit Опечатался Что значит "опечатался"? Покажи код, на котором можно строить предположения. 0

@Corvit 0 / 0 / 0 Регистрация: 14.10.2013 Сообщений: 13
	30.10.2013, 14:49 [ТС]	10
	Спасибо за разъяснение 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	30.10.2013, 14:51	11
	Сообщение от Corvit Так и есть, но в строке Откуда знаешь что вызов происходит именно в этой строке? Самый первый конструктор без параметров происходит в этой (48-й строке последнего кода) строке: Pstring s1; 0

@Corvit 0 / 0 / 0 Регистрация: 14.10.2013 Сообщений: 13
	01.11.2013, 10:12 [ТС]	14
	долго, вот побыстрее Вопрос концептуальный, а не оптимизационный 0