プログラミング

プログラミングの「ハードコーディング」なぜダメ？

ハードコーディングが悪い理由と改善方法を解説。保守性、再利用性、テスト性の問題から、適切な設定管理とリファクタリング方法まで詳しく紹介します

Learning Next 運営
約 31 分で読めます
ハードコーディングプログラミング保守性設定管理リファクタリング

みなさん、コードの中に直接値を書き込んで、後で「変更が大変だった」という経験はありませんか？

例えば、APIのURLやデータベースの接続情報、設定値などを直接コードに書いてしまうことがありますよね。

これが「ハードコーディング」と呼ばれる問題のあるプログラミング手法です。 この記事では、なぜハードコーディングがダメなのか、そしてどう改善すべきかを具体例とともに詳しく解説します。

ハードコーディングとは？

基本的な概念

ハードコーディングとは、プログラムの中に設定値や定数を直接書き込むことです。

簡単に言うと、「後で変更する可能性がある値を、コードの中に直接書いてしまう」ことです。

ハードコーディングの例

悪い例

def connect_to_database():
    # データベース接続情報を直接書いている
    connection = mysql.connector.connect(
        host="192.168.1.100",
        user="admin",
        password="secret123",
        database="production_db"
    )
    return connection


def send_notification(message):
    # APIのURLを直接書いている
    api_url = "https://api.example.com/v1/notifications"
    
    # タイムアウト時間を直接書いている
    response = requests.post(api_url, data=message, timeout=30)
    return response


def calculate_tax(price):
    # 税率を直接書いている
    tax_rate = 0.10
    return price * tax_rate

この例では、IP アドレス、パスワード、URL、税率などが直接コードに書かれています。

良い例との比較

良い例

import os
from config import TAX_RATE, API_TIMEOUT


def connect_to_database():
    # 環境変数から取得
    connection = mysql.connector.connect(
        host=os.getenv("DB_HOST"),
        user=os.getenv("DB_USER"),
        password=os.getenv("DB_PASSWORD"),
        database=os.getenv("DB_NAME")
    )
    return connection


def send_notification(message):
    # 設定ファイルから取得
    api_url = os.getenv("NOTIFICATION_API_URL")
    
    response = requests.post(api_url, data=message, timeout=API_TIMEOUT)
    return response


def calculate_tax(price):
    # 設定ファイルで管理
    return price * TAX_RATE

この例では、設定値が外部から取得されるようになっています。

ハードコーディングの問題点

保守性の問題

変更が困難

問題の例

public class OrderService {
    public void processOrder(Order order) {
        // 送料を直接書いている
        double shippingCost = 500.0;
        
        // 割引率を直接書いている
        double discountRate = 0.15;
        
        // 最大注文数を直接書いている
        int maxOrderQuantity = 100;
        
        // 処理ロジック...
    }
}

問題点

  • 送料変更時にコードを修正する必要
  • 複数箇所に同じ値があると修正漏れのリスク
  • デプロイが必要で変更コストが高い

影響範囲の把握困難

問題の例

# ファイル1
def get_user_data():
    max_users = 1000  # 最大ユーザー数
    # 処理...


# ファイル2  
def validate_registration():
    max_users = 1000  # 同じ値だが別の場所
    # 処理...


# ファイル3
def generate_report():
    user_limit = 1000  # また同じ値
    # 処理...

同じ意味の値が複数箇所に書かれていると、変更時に見落としが発生しやすくなります。

再利用性の問題

環境別の対応困難

問題の例

// 開発環境用の設定
const API_BASE_URL = "http://localhost:3000/api";
const DATABASE_URL = "localhost:5432/dev_db";
const DEBUG_MODE = true;


// 本番環境では？？？

環境によって設定が異なる場合、ハードコーディングでは対応が困難です。

複数のプロジェクトでの利用困難

問題の例

class EmailService:
    def send_email(self, to, subject, body):
        # SMTPサーバー情報を直接書いている
        smtp_server = "smtp.company.com"
        smtp_port = 587
        username = "noreply@company.com"
        password = "email_password"
        
        # 送信処理...

このクラスを他のプロジェクトで使う場合、毎回コードを修正する必要があります。

テスト性の問題

テストが困難

問題の例

def fetch_user_profile(user_id):
    # 外部APIのURLを直接書いている
    api_url = f"https://api.production.com/users/{user_id}"
    
    response = requests.get(api_url)
    return response.json()


# テスト時の問題
def test_fetch_user_profile():
    # 本番APIにアクセスしてしまう！
    result = fetch_user_profile(123)
    # テストが本番データに依存してしまう

ハードコーディングされた値により、適切な単体テストが書けません。

モックの利用困難

改善例

class UserService:
    def __init__(self, api_base_url):
        self.api_base_url = api_base_url
    
    def fetch_user_profile(self, user_id):
        api_url = f"{self.api_base_url}/users/{user_id}"
        response = requests.get(api_url)
        return response.json()


# テスト時
def test_fetch_user_profile():
    # テスト用のURLを注入
    service = UserService("http://mock-api.test")
    result = service.fetch_user_profile(123)
    # テストが独立して実行できる

セキュリティの問題

機密情報の露出

危険な例

# GitHubに公開されるコード
def connect_to_payment_api():
    api_key = "sk_live_abcd1234567890"  # 本番APIキー！
    secret = "secret_xyz987654321"      # 秘密鍵！
    
    # 決済処理...

問題点

  • APIキーがソースコードに残る
  • バージョン管理システムに記録される
  • 第三者に見られるリスク
  • 不正利用の危険性

設定変更の困難さ

問題の例

public class SecurityConfig {
    // セキュリティ設定を直接書いている
    private static final int SESSION_TIMEOUT = 1800; // 30分
    private static final int MAX_LOGIN_ATTEMPTS = 3;
    private static final String ENCRYPTION_KEY = "hardcoded_key_123";
}

セキュリティ要件が変わった時に、コードの修正とデプロイが必要になります。

改善方法

設定ファイルの活用

基本的な設定ファイル

config.py

# アプリケーション設定
APP_NAME = "MyApplication"
VERSION = "1.0.0"
DEBUG = False


# データベース設定
DB_HOST = "localhost"
DB_PORT = 5432
DB_NAME = "myapp_db"


# API設定
API_TIMEOUT = 30
MAX_RETRY_COUNT = 3


# ビジネスロジック設定
TAX_RATE = 0.10
SHIPPING_COST = 500
MAX_ORDER_QUANTITY = 100

使用例

from config import TAX_RATE, SHIPPING_COST, MAX_ORDER_QUANTITY


def calculate_order_total(items, quantity):
    if quantity > MAX_ORDER_QUANTITY:
        raise ValueError("注文数が上限を超えています")
    
    subtotal = sum(item.price for item in items)
    tax = subtotal * TAX_RATE
    shipping = SHIPPING_COST
    
    return subtotal + tax + shipping

JSON設定ファイル

config.json

{
  "database": {
    "host": "localhost",
    "port": 5432,
    "name": "myapp_db",
    "pool_size": 10
  },
  "api": {
    "base_url": "https://api.example.com",
    "timeout": 30,
    "retry_count": 3
  },
  "business": {
    "tax_rate": 0.10,
    "shipping_cost": 500,
    "max_order_quantity": 100
  }
}

読み込み例

import json


class Config:
    def __init__(self, config_file):
        with open(config_file, 'r') as f:
            self.config = json.load(f)
    
    def get(self, key_path):
        keys = key_path.split('.')
        value = self.config
        
        for key in keys:
            value = value[key]
        
        return value


# 使用例
config = Config('config.json')
tax_rate = config.get('business.tax_rate')
db_host = config.get('database.host')

環境変数の活用

基本的な環境変数の使用

環境変数の設定

# .env ファイル
DB_HOST=localhost
DB_USER=myapp_user
DB_PASSWORD=secure_password
DB_NAME=myapp_production


API_BASE_URL=https://api.production.com
API_KEY=prod_api_key_12345
API_TIMEOUT=30


TAX_RATE=0.10
SHIPPING_COST=500

Python での読み込み

import os
from dotenv import load_dotenv


# .env ファイルを読み込み
load_dotenv()


class DatabaseConfig:
    HOST = os.getenv('DB_HOST', 'localhost')
    USER = os.getenv('DB_USER', 'default_user')
    PASSWORD = os.getenv('DB_PASSWORD')
    NAME = os.getenv('DB_NAME', 'default_db')


class APIConfig:
    BASE_URL = os.getenv('API_BASE_URL')
    API_KEY = os.getenv('API_KEY')
    TIMEOUT = int(os.getenv('API_TIMEOUT', '30'))


# 使用例
def connect_to_database():
    return psycopg2.connect(
        host=DatabaseConfig.HOST,
        user=DatabaseConfig.USER,
        password=DatabaseConfig.PASSWORD,
        database=DatabaseConfig.NAME
    )

環境別設定の管理

開発環境用

# .env.development
DB_HOST=localhost
DB_NAME=myapp_dev
DEBUG=True
API_BASE_URL=http://localhost:3000

本番環境用

# .env.production
DB_HOST=prod-db.example.com
DB_NAME=myapp_prod
DEBUG=False
API_BASE_URL=https://api.production.com

設定の切り替え

import os


# 環境に応じて設定ファイルを切り替え
ENV = os.getenv('ENVIRONMENT', 'development')
load_dotenv(f'.env.{ENV}')

定数の管理

定数クラスの作成

constants.py

class DatabaseConstants:
    DEFAULT_TIMEOUT = 30
    MAX_CONNECTIONS = 100
    RETRY_COUNT = 3


class BusinessConstants:
    TAX_RATE = 0.10
    SHIPPING_COST = 500
    MAX_ORDER_QUANTITY = 100
    MIN_ORDER_AMOUNT = 1000


class APIConstants:
    DEFAULT_TIMEOUT = 30
    MAX_RETRY_COUNT = 3
    RATE_LIMIT_PER_MINUTE = 1000


class ValidationConstants:
    MIN_PASSWORD_LENGTH = 8
    MAX_USERNAME_LENGTH = 50
    ALLOWED_FILE_EXTENSIONS = ['.jpg', '.png', '.pdf']

使用例

from constants import BusinessConstants, ValidationConstants


def validate_order(order):
    if order.quantity > BusinessConstants.MAX_ORDER_QUANTITY:
        raise ValueError("注文数が上限を超えています")
    
    if order.amount < BusinessConstants.MIN_ORDER_AMOUNT:
        raise ValueError("最小注文金額を下回っています")
    
    return True


def validate_password(password):
    if len(password) < ValidationConstants.MIN_PASSWORD_LENGTH:
        raise ValueError(f"パスワードは{ValidationConstants.MIN_PASSWORD_LENGTH}文字以上である必要があります")
    
    return True

依存性注入の活用

基本的な依存性注入

改善前

class OrderService:
    def process_order(self, order):
        # 設定を直接書いている
        tax_rate = 0.10
        shipping_cost = 500
        
        # 処理...

改善後

class OrderService:
    def __init__(self, tax_rate, shipping_cost):
        self.tax_rate = tax_rate
        self.shipping_cost = shipping_cost
    
    def process_order(self, order):
        # 注入された設定を使用
        tax = order.amount * self.tax_rate
        total = order.amount + tax + self.shipping_cost
        return total


# 使用例
from config import TAX_RATE, SHIPPING_COST


order_service = OrderService(TAX_RATE, SHIPPING_COST)

設定オブジェクトの注入

設定クラス

class OrderConfig:
    def __init__(self):
        self.tax_rate = float(os.getenv('TAX_RATE', '0.10'))
        self.shipping_cost = int(os.getenv('SHIPPING_COST', '500'))
        self.max_quantity = int(os.getenv('MAX_ORDER_QUANTITY', '100'))


class OrderService:
    def __init__(self, config: OrderConfig):
        self.config = config
    
    def process_order(self, order):
        if order.quantity > self.config.max_quantity:
            raise ValueError("注文数が上限を超えています")
        
        tax = order.amount * self.config.tax_rate
        total = order.amount + tax + self.config.shipping_cost
        return total

リファクタリングの手順

既存コードの改善

Step 1: ハードコーディング箇所の特定

検索すべきパターン

  • 数値リテラル（マジックナンバー）
  • 文字列リテラル（URL、パスワード等）
  • 設定値らしきもの
  • 環境依存の値

特定方法

# 数値の検索
grep -r "\b[0-9]\+\b" src/


# URLの検索  
grep -r "http[s]*://" src/


# IPアドレスの検索
grep -r "\b[0-9]\{1,3\}\.[0-9]\{1,3\}\.[0-9]\{1,3\}\.[0-9]\{1,3\}\b" src/


# パスワード的な文字列の検索
grep -ri "password\|secret\|key" src/

Step 2: 設定値の分類

分類例

## 環境依存の設定
- データベース接続情報
- API エンドポイント
- ファイルパス
- 認証情報


## ビジネスルールの設定
- 税率
- 手数料
- 制限値
- デフォルト値


## 技術的な設定
- タイムアウト値
- リトライ回数
- バッファサイズ
- ログレベル

Step 3: 段階的な修正

修正例

# Step 1: 元のコード
def calculate_fee(amount):
    return amount * 0.03  # 手数料率をハードコーディング


# Step 2: 定数として抽出
FEE_RATE = 0.03


def calculate_fee(amount):
    return amount * FEE_RATE


# Step 3: 設定ファイルに移動
# config.py
FEE_RATE = float(os.getenv('FEE_RATE', '0.03'))


# main.py
from config import FEE_RATE


def calculate_fee(amount):
    return amount * FEE_RATE


# Step 4: 依存性注入
class FeeCalculator:
    def __init__(self, fee_rate):
        self.fee_rate = fee_rate
    
    def calculate_fee(self, amount):
        return amount * self.fee_rate

テストの改善

テスタブルなコードへの変更

改善前

def send_email(to, subject, body):
    # SMTP設定をハードコーディング
    smtp_server = "smtp.company.com"
    smtp_port = 587
    
    # 送信処理...
    
# テストが困難

改善後

class EmailService:
    def __init__(self, smtp_server, smtp_port):
        self.smtp_server = smtp_server
        self.smtp_port = smtp_port
    
    def send_email(self, to, subject, body):
        # 送信処理...


# テスト用のモック
class MockEmailService(EmailService):
    def __init__(self):
        super().__init__("mock.smtp.com", 587)
        self.sent_emails = []
    
    def send_email(self, to, subject, body):
        self.sent_emails.append({
            'to': to,
            'subject': subject,
            'body': body
        })


# テスト
def test_email_sending():
    email_service = MockEmailService()
    email_service.send_email("test@example.com", "Test", "Content")
    
    assert len(email_service.sent_emails) == 1
    assert email_service.sent_emails[0]['to'] == "test@example.com"

ベストプラクティス

設定管理のガイドライン

設定の階層化

優先順位

  1. 環境変数: 最高優先度
  2. 設定ファイル: 中優先度
  3. デフォルト値: 最低優先度

実装例

def get_config_value(env_var, config_key, default_value):
    # 1. 環境変数を確認
    env_value = os.getenv(env_var)
    if env_value is not None:
        return env_value
    
    # 2. 設定ファイルを確認
    config_value = config.get(config_key)
    if config_value is not None:
        return config_value
    
    # 3. デフォルト値を使用
    return default_value


# 使用例
DB_HOST = get_config_value('DB_HOST', 'database.host', 'localhost')

設定の検証

設定値の妥当性チェック

class ConfigValidator:
    @staticmethod
    def validate_port(port):
        if not isinstance(port, int) or port < 1 or port > 65535:
            raise ValueError(f"無効なポート番号: {port}")
    
    @staticmethod
    def validate_url(url):
        if not url or not url.startswith(('http://', 'https://')):
            raise ValueError(f"無効なURL: {url}")
    
    @staticmethod
    def validate_rate(rate):
        if not isinstance(rate, (int, float)) or rate < 0 or rate > 1:
            raise ValueError(f"無効な率: {rate}")


# 設定読み込み時に検証
class DatabaseConfig:
    def __init__(self):
        self.host = os.getenv('DB_HOST', 'localhost')
        self.port = int(os.getenv('DB_PORT', '5432'))
        
        # 検証
        ConfigValidator.validate_port(self.port)

セキュリティ対策

機密情報の保護

暗号化された設定ファイル

import base64
import json
from cryptography.fernet import Fernet


class SecureConfig:
    def __init__(self, key):
        self.fernet = Fernet(key)
    
    def encrypt_config(self, config_dict, output_file):
        config_json = json.dumps(config_dict)
        encrypted_data = self.fernet.encrypt(config_json.encode())
        
        with open(output_file, 'wb') as f:
            f.write(encrypted_data)
    
    def decrypt_config(self, input_file):
        with open(input_file, 'rb') as f:
            encrypted_data = f.read()
        
        decrypted_data = self.fernet.decrypt(encrypted_data)
        return json.loads(decrypted_data.decode())


# 使用例
key = Fernet.generate_key()
secure_config = SecureConfig(key)


# 機密設定の暗号化
sensitive_config = {
    "api_key": "secret_api_key",
    "db_password": "secret_password"
}
secure_config.encrypt_config(sensitive_config, "config.enc")

環境別のアクセス制御

環境別の設定分離

import os


class EnvironmentConfig:
    def __init__(self):
        self.env = os.getenv('ENVIRONMENT', 'development')
        self.load_config()
    
    def load_config(self):
        if self.env == 'production':
            self.load_production_config()
        elif self.env == 'staging':
            self.load_staging_config()
        else:
            self.load_development_config()
    
    def load_production_config(self):
        # 本番環境用の厳密な設定
        self.debug = False
        self.log_level = 'WARNING'
        self.allowed_hosts = ['api.production.com']
    
    def load_development_config(self):
        # 開発環境用の緩い設定
        self.debug = True
        self.log_level = 'DEBUG'
        self.allowed_hosts = ['*']

まとめ

ハードコーディングは、保守性、再利用性、テスト性を著しく損なう問題のあるプログラミング手法です。

重要なポイント

  • 設定の外部化: 値をコードから分離する
  • 環境変数の活用: 環境に応じた設定の切り替え
  • 依存性注入: テスタブルで柔軟なコード設計
  • 段階的改善: 既存コードの計画的なリファクタリング
  • セキュリティ対策: 機密情報の適切な管理

初心者の方は、まず小さな設定値から外部化を始めて、徐々に全体の設計を改善していくことをおすすめします。

適切な設定管理により、保守しやすく、テストしやすい、高品質なコードを書くことができるでしょう。 ぜひハードコーディングを避けて、より良いプログラムを作ってみてください。

関連記事

アグリテックエンジニア - 農業×技術の可能性

農業とテクノロジーを融合させるアグリテックエンジニアの仕事内容、必要なスキル、キャリアパスを解説。IoT、AI、ドローンなどの最新技術で農業の課題を解決する新しい職種の魅力をお伝えします。

2025年7月5日
10分

【初心者向け】プログラミングの「GraphQL」vs「REST」

GraphQLとRESTの違いを初心者にもわかりやすく解説。API設計の基本から実際の使い分け方まで、具体例を交えて詳しく説明します

2025年7月5日
18分

【初心者向け】プログラミングの「非同期通信」なぜ難しい？

プログラミング初心者が非同期通信で躓く理由を分析。同期処理との違い、コールバック地獄、Promise、async/awaitまで段階的に解説し、理解しやすい学習方法を紹介します。

2025年7月5日
16分