Skip to content

safe API の入力検証を共通化するヘルパーを追加#15

Merged
T3pp31 merged 1 commit intomainfrom
codex/add-validate_safe_inputs-function
Apr 17, 2026
Merged

safe API の入力検証を共通化するヘルパーを追加#15
T3pp31 merged 1 commit intomainfrom
codex/add-validate_safe_inputs-function

Conversation

@T3pp31
Copy link
Copy Markdown
Owner

@T3pp31 T3pp31 commented Apr 17, 2026

Motivation

  • encrypt_safedecrypt_safe に同一の入力検証ロジックが存在しており、重複とエラーメッセージ散逸を防ぐために共通化する必要があった。

Description

  • encrypt_safe / decrypt_safe の検証処理を private 関数 validate_safe_inputs に集約して重複を削減した。
  • InvalidShift のエラーメッセージ生成を invalid_shift_error に切り出し、エラー文言を一元化した。
  • decrypt_safe は検証後に既存のオーバーフロー回避ロジック(i32 へ拡張して正規化)を維持するように整理した。
  • 変更ファイル: src/caesar_cipher.rs

Testing

  • 自動テストを cargo test で実行し、全テストが成功しました(すべてのテストパス)。

Codex Task

@T3pp31 T3pp31 merged commit f21e486 into main Apr 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant