Top > C++ > バグの少ないプログラムを書くためのメモ

ここに書いてあることは「バグの少ないプログラムを書く」ためのものであって,
「実行速度が速いプログラムを書く」ものではありません。

目次

非POD型のメンバ変数は,可能な限りポインタにしない Edit

delete忘れを防ぐため,
可能な限りポインタを所持しない。

すべてを展開すべてを収束
  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
 
-
|
|
|
|
-
!
|
!
 
 
 
-
|
|
|
|
|
|
-
!
!
 
 
 
-
|
|
|
|
|
|
|
|
|
-
!
|
|
-
|
|
|
!
|
!
 
class Angle
{
public:
  float rad;
 
  Angle():rad(0)
  {
  }
 
}
 
// ポインタ所持しない版
class Angle3
{
public:
  Angle x;
  Angle y;
  Angle z;
 
  Angle3()
  {
  }
}
 
// ポインタ所持する版(ポインタを公開してるので,あまりよくない例ですが;)
class Angle3_
{
public:
  Angle* x;
  Angle* y;
  Angle* z;
 
  Angle3():
    x( new Angle() ), 
    y( new Angle() ),
    z( new Angle() )
  {
  }
 
  ~Angle3()
  {
     delete x;
     delete y;
     delete z;
  }
 
}

初期化忘れを防ぐため,なるべく非POD型にする Edit

初期化忘れの例

すべてを展開すべてを収束
  1
  2
  3
  4
  5
  6
  7
  8
  9
 10
 11
 12
 13
 14
 15
 16
 17
 18
 19
 20
 21
 22
 23
 
 
 
-
|
|
|
|
|
-
|
!
!
 
 
-
|
|
|
|
|
!
 
#include <iostream>
 
struct Vector3
{
  float x;
  float y;
  float z;
 
  void print()
  {
    std::cout << x << y << z << std::endl;
  }
};
 
void func
{
  Vector3 vec;
  vec.x = 1;
  vec.y = 0;
  vec.print();
  // vec.zの初期化を忘れてる!
}

もし,Vector3が非Pod型だったら。

すべてを展開すべてを収束
  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
 
 
 
-
|
|
|
|
|
|
-
!
|
|
-
|
!
!
 
 
-
|
|
|
|
!
 
#include <iostream>
 
class Vector3
{
public:
  float x;
  float y;
  float z;
 
  Vector3():x(0),y(0),z(0)
  {
  }
 
  void print()
  {
    std::cout << x << y << z << std::endl;
  }
};
 
void func
{
  Vector3 vec;   // vec.zはここで初期化されている。
  vec.x = 1;
  vec.y = 0;
  vec.print();
}

リロード   新規 下位ページ作成 編集 凍結 差分 添付 コピー 名前変更   ホーム 一覧 検索 最終更新 バックアップ リンク元   ヘルプ   最終更新のRSS
Last-modified: Tue, 11 Jul 2006 22:05:27 JST (6521d)