総合演習：品質チェック実践

var orders = [];
var TAX = 0.1;

function createOrder(productId, qty, customer) {
  console.log("creating order for:", customer.name);

  var order = {
    id: Math.random().toString(36).substr(2, 9),
    productId: productId,
    quantity: qty,
    customer: customer,
    status: "created",
    createdAt: new Date()
  };

  orders.push(order);
  return order;
}

function getOrder(id) {
  for (var i = 0; i < orders.length; i++) {
    if (orders[i].id == id) {
      return orders[i];
    }
  }
}

function cancel(id) {
  var order = getOrder(id);
  order.status = "cancelled";
  return order;
}

function calc(id, unitPrice) {
  var order = getOrder(id);
  var price = unitPrice * order.quantity;
  var tax = price * 0.1;
  console.log("calculated tax:", tax);
  return price + tax;
}

function getOrdersByCustomer(customerId) {
  var result = [];
  for (var i = 0; i < orders.length; i++) {
    if (orders[i].customer.id == customerId) {
      result.push(orders[i]);
    }
  }
  return result;
}

// TODO: 後で実装
function updateOrder(id, data) {
}

## 発見した問題点

| # | 問題点 | カテゴリ | 重要度 |
|---|--------|---------|--------|
| 1 | | | |
| 2 | | | |
| 3 | | | |
（続く...）

## 注文管理システム 品質チェックリスト

### コーディング規約
- [ ]
- [ ]
- [ ]

### 命名規則
- [ ]
- [ ]

### エラーハンドリング
- [ ]
- [ ]
- [ ]

### 仕様準拠
- [ ]
- [ ]
- [ ]

### コード品質
- [ ]
- [ ]
- [ ]

### セキュリティ
- [ ]
- [ ]

## 注文管理システム 品質チェックリスト

### コーディング規約
- [ ] var を使わず const/let を使用している
- [ ] 比較演算子は === を使用している
- [ ] console.log を削除している
- [ ] インデントが統一されている

### 命名規則
- [ ] 関数名は動詞で始まり、処理内容がわかる
- [ ] 変数名から内容が推測できる
- [ ] 略語を使いすぎていない

### エラーハンドリング
- [ ] 存在しない注文IDを指定した場合のエラー処理がある
- [ ] 外部呼び出しに try-catch がある
- [ ] エラーメッセージが具体的である

### 仕様準拠
- [ ] 数量が1-100の範囲内かチェックしている
- [ ] キャンセルは24時間以内のみ可能にしている
- [ ] 税率10%が正しく適用されている
- [ ] 必須項目がすべて含まれている

### コード品質
- [ ] マジックナンバーを定数化している
- [ ] 重複コードがない
- [ ] TODO コメントを放置していない
- [ ] 関数が50行以下である

### セキュリティ
- [ ] 顧客情報が適切に保護されている
- [ ] 入力値のバリデーションがある

## テストケース

| ID | テスト名 | 前提条件 | 手順 | 期待結果 |
|----|---------|---------|------|---------|
| TC-001 | | | | |
| TC-002 | | | | |
| TC-003 | | | | |
| TC-004 | | | | |
| TC-005 | | | | |

## バグ報告書 #1

### タイトル

### 環境情報

### 再現手順

### 期待結果

### 実際結果

### 重要度

### 備考

## バグ報告書 #1

### タイトル
[注文キャンセル] 存在しない注文IDを指定するとTypeErrorが発生する

### 環境情報
- テスト環境: ローカル開発環境
- Node.js: v18.x
- テスト日時: 2025-04-15 14:30

### 再現手順
1. orders配列が空の状態で、存在しない注文ID（例: "invalid-id"）を使用
2. cancel("invalid-id") を呼び出す

### 期待結果
適切なエラーメッセージ（例: "Order not found: invalid-id"）が返される

### 実際結果
「TypeError: Cannot read property 'status' of undefined」が発生
アプリケーションがクラッシュする

### 重要度
高（アプリケーションがクラッシュするため）

### 備考
- getOrder関数がundefinedを返し、cancel関数でnullチェックがないことが原因
- getOrder関数でnullを返すか、cancel関数でnullチェックを追加する必要がある

## バグ報告書 #2

### タイトル
[注文作成] 数量が100を超えても注文が作成できてしまう

### 環境情報
- テスト環境: ローカル開発環境
- Node.js: v18.x
- テスト日時: 2025-04-15 14:45

### 再現手順
1. createOrder("P001", 101, {id: "C001", name: "田中"}) を呼び出す

### 期待結果
「数量は1以上100以下で指定してください」というエラーが発生し、注文は作成されない

### 実際結果
qty=101 の注文が正常に作成され、orders配列に追加された

### 重要度
中（仕様違反。大量注文による在庫問題や請求問題が発生する可能性）

### 備考
- 仕様では「数量は1以上100以下」と記載されている
- createOrder関数に入力バリデーションが実装されていない
- qty <= 0 のケースも同様にバリデーションが必要