xxxx

Author: nowakfli

ai-track-docs/perf-baseline.md

@@ -0,0 +1,40 @@
+# Performance Baseline
+
+## Function
+
+- Module: `samples/book-app-project/books.py`
+- Target: `BookCollection.list_books()`
+- Measurement approach: standalone micro-benchmark in `samples/book-app-project/benchmark_list_books.py`
+
+## Command
+
+```bash
+cd samples/book-app-project
+python benchmark_list_books.py
+```
+
+## Baseline Results
+
+Run 1
+
+- Iterations: 10000
+- Book count: 1000
+- Mean: 0.122 us
+- Median: 0.100 us
+- Min: 0.000 us
+- Max: 2.100 us
+
+Run 2
+
+- Iterations: 10000
+- Book count: 1000
+- Mean: 0.098 us
+- Median: 0.100 us
+- Min: 0.000 us
+- Max: 3.700 us
+
+## Variance Notes
+
+- The median stayed at 0.100 us across both runs, which matches the simple in-memory return path.
+- The mean moved from 0.122 us to 0.098 us and the max moved from 2.100 us to 3.700 us, which is consistent with small scheduler or interpreter noise.
+- Re-run the benchmark from a quiet shell if you want to compare future changes against a cleaner baseline.

samples/book-app-project/benchmark_list_books.py

@@ -0,0 +1,38 @@
+import statistics
+import time
+
+from books import Book, BookCollection
+
+
+def run_benchmark(iterations: int = 10000, book_count: int = 1000) -> dict[str, float]:
+    collection = BookCollection()
+    collection.books = [
+        Book(title=f"Book {index}", author="Benchmark Author", year=2000 + (index % 20))
+        for index in range(book_count)
+    ]
+
+    samples = []
+    for _ in range(iterations):
+        start = time.perf_counter()
+        collection.list_books()
+        samples.append((time.perf_counter() - start) * 1_000_000)
+
+    return {
+        "iterations": iterations,
+        "book_count": book_count,
+        "mean_us": statistics.mean(samples),
+        "median_us": statistics.median(samples),
+        "min_us": min(samples),
+        "max_us": max(samples),
+    }
+
+
+if __name__ == "__main__":
+    results = run_benchmark()
+    print("Benchmark: list_books")
+    print(f"Iterations: {results['iterations']}")
+    print(f"Book count: {results['book_count']}")
+    print(f"Mean: {results['mean_us']:.3f} us")
+    print(f"Median: {results['median_us']:.3f} us")
+    print(f"Min: {results['min_us']:.3f} us")
+    print(f"Max: {results['max_us']:.3f} us")

samples/book-app-project/book_app.py

@@ -33,8 +33,17 @@ def handle_add():
     author = input("Author: ").strip()
     year_str = input("Year: ").strip()
 
+    # Input validation
+    if not title:
+        print("\nError: Title cannot be empty.\n")
+        return
+    if not author:
+        print("\nError: Author cannot be empty.\n")
+        return
     try:
         year = int(year_str) if year_str else 0
+        if year < 0 or year > 9999:
+            raise ValueError("Year must be between 0 and 9999.")
         collection.add_book(title, author, year)
         print("\nBook added successfully.\n")
     except ValueError as e:

samples/book-app-project/tests/test_books.py

@@ -68,3 +68,31 @@ def test_remove_book_invalid():
     collection = BookCollection()
     result = collection.remove_book("Nonexistent Book")
     assert result is False
+
+def test_handle_add_invalid_input(monkeypatch, capsys):
+    from book_app import handle_add
+
+    # Test empty title
+    monkeypatch.setattr('builtins.input', lambda _: "")
+    handle_add()
+    captured = capsys.readouterr()
+    assert "Error: Title cannot be empty." in captured.out
+
+    # Test empty author
+    monkeypatch.setattr('builtins.input', lambda prompt: "Title" if "Title" in prompt else "")
+    handle_add()
+    captured = capsys.readouterr()
+    assert "Error: Author cannot be empty." in captured.out
+
+    # Test invalid year
+    monkeypatch.setattr('builtins.input', lambda prompt: "Title" if "Title" in prompt else ("Author" if "Author" in prompt else "-1"))
+    handle_add()
+    captured = capsys.readouterr()
+    assert "Error: Year must be between 0 and 9999." in captured.out
+
+def test_list_books():
+    collection = BookCollection()
+    collection.add_book("Test Book", "Author", 2023)
+    books = collection.list_books()
+    assert len(books) == 1
+    assert books[0].title == "Test Book"