Revit電材積算アドオンプログラム設計(Gemini版)

GeminiにDeep Seekが提示したコードの判定をしてもらった。 【以下はGeminiの回答】 ご提示のソースコードにはいくつかの潜在的な問題点と改善点があります。以下に修正点とソースコード全体を示します。 **修正点と改善点:** 1. **Google Lens APIの非存在:** `analyze_with_google_lens` 関数内で Google Lens API を呼び出す処理がコメントアウトされています。実際には Google Lens の公式 API は公開されていません。代替手段として、OCR (Optical Character Recognition) ライブラリ (例: `pytesseract`) を使用して画像からテキストを抽出し、そのテキストに基づいて製品情報を推測する実装になっています。これは現実的なアプローチですが、精度は画像の品質やテキストの配置に依存します。`pytesseract` を使用するには別途インストールが必要です (`pip install pytesseract Pillow`). また、Tesseract OCR エンジンもインストールし、パスを設定する必要があります。 2. **スクレイピングの脆弱性:** `scrape_material_data` 関数で行っているスクレイピングは、Webサイトの構造が変更されると動作しなくなる可能性があります。また、robots.txt に従うべきであり、過度なアクセスはサーバーに負荷をかける可能性があります。より堅牢なデータ収集のためには、API が提供されている場合は API の利用を検討するか、特定のWebサイトに特化したスクレイピングロジックを実装する必要があります。 3. **仮想的な Revit API:** `revit_api` モジュールは仮想的なものであり、実際の Revit API との連携機能はありません。Revit API を操作するには、`pyrevit` などの Python ラッパーを使用する必要があります。このコードでは、仮想的な関数 (`revit_api.create_family`, `revit_api.link_estimation_data`, `revit_api.run_estimation`) が呼び出されているため、実際に Revit を操作することはできません。 4. **エラー処理の改善:** 例外処理 (`try-except`) は行われていますが、より具体的な例外を捕捉し、ユーザーに分かりやすいエラーメッセージを表示することが望ましい場合があります。 5. **UIの改善:** UI の配置や情報の表示方法など、ユーザーエクスペリエンス向上のための改善の余地があります。 6. **依存関係の明示:** 必要なライブラリ (`requests`, `bs4`, `pyautogui`, `time`, `os`, `PIL`, `pyperclip`, `json`, `pytesseract`) は import されていますが、ユーザーが実行する前にこれらのライブラリをインストールする必要があることを明示的に伝えることが推奨されます。 **修正後のソースコード:** ```python import tkinter as tk from tkinter import ttk, filedialog, messagebox import requests from bs4 import BeautifulSoup import pyautogui import time import os from PIL import Image import pyperclip import json # Tesseract OCR を使用する場合に必要なライブラリ try: from pytesseract import image_to_string except ImportError: print("Error: pytesseract がインストールされていません。インストールしてください: pip install pytesseract Pillow") image_to_string = None # 仮想的な Revit API ラッパー (実際には pyrevit などを使用) class RevitAPIWrapper: def create_family(self, params): print(f"Revitファミリを作成 (仮想): {params}") return "dummy_family_id" def link_estimation_data(self, family_id, data): print(f"積算データをファミリに紐付け (仮想): ID={family_id}, Data={data}") def run_estimation(self, family_id): print(f"Revitで積算を実行 (仮想): ID={family_id}") return {'total_quantity': 10, 'total_cost': 15000.00} revit_api = RevitAPIWrapper() class ElectricalMaterialEstimator: def __init__(self, root): self.root = root self.root.title("電材積算アドオン") self.root.geometry("800x600") # 変数初期化 self.image_path = tk.StringVar() self.material_data = {} self.family_data = {} # GUIコンポーネント作成 self.create_widgets() def create_widgets(self): # タブコントロール tab_control = ttk.Notebook(self.root) # タブ作成 tab1 = ttk.Frame(tab_control) tab2 = ttk.Frame(tab_control) tab3 = ttk.Frame(tab_control) tab4 = ttk.Frame(tab_control) tab_control.add(tab1, text="画像読み取り") tab_control.add(tab2, text="データ収集") tab_control.add(tab3, text="ファミリ作成") tab_control.add(tab4, text="積算設定") tab_control.pack(expand=1, fill="both") # タブ1: 画像読み取り self.create_tab1(tab1) # タブ2: データ収集 self.create_tab2(tab2) # タブ3: ファミリ作成 self.create_tab3(tab3) # タブ4: 積算設定 self.create_tab4(tab4) def create_tab1(self, tab): frame = ttk.LabelFrame(tab, text="Google Lensで電材画像を読み取る") frame.pack(padx=10, pady=10, fill="both", expand=True) ttk.Label(frame, text="電材画像を選択:").grid(row=0, column=0, padx=5, pady=5) ttk.Entry(frame, textvariable=self.image_path, width=50).grid(row=0, column=1, padx=5, pady=5) ttk.Button(frame, text="参照...", command=self.browse_image).grid(row=0, column=2, padx=5, pady=5) ttk.Button(frame, text="Google Lensで画像解析", command=self.analyze_with_google_lens).grid(row=1, column=0, columnspan=3, pady=10) self.result_text1 = tk.Text(frame, height=15, width=80) self.result_text1.grid(row=2, column=0, columnspan=3, padx=5, pady=5) scrollbar = ttk.Scrollbar(frame, command=self.result_text1.yview) scrollbar.grid(row=2, column=3, sticky='ns') self.result_text1.config(yscrollcommand=scrollbar.set) def create_tab2(self, tab): frame = ttk.LabelFrame(tab, text="電材データをスクレイピング") frame.pack(padx=10, pady=10, fill="both", expand=True) ttk.Label(frame, text="製品名/型番:").grid(row=0, column=0, padx=5, pady=5) self.product_entry = ttk.Entry(frame, width=50) self.product_entry.grid(row=0, column=1, padx=5, pady=5) ttk.Button(frame, text="メーカーサイトからデータ収集", command=self.scrape_material_data).grid(row=1, column=0, columnspan=2, pady=10) self.result_text2 = tk.Text(frame, height=15, width=80) self.result_text2.grid(row=2, column=0, columnspan=2, padx=5, pady=5) scrollbar = ttk.Scrollbar(frame, command=self.result_text2.yview) scrollbar.grid(row=2, column=2, sticky='ns') self.result_text2.config(yscrollcommand=scrollbar.set) def create_tab3(self, tab): frame = ttk.LabelFrame(tab, text="Revitファミリ作成") frame.pack(padx=10, pady=10, fill="both", expand=True) ttk.Label(frame, text="ファミリタイプ:").grid(row=0, column=0, padx=5, pady=5) self.family_type = ttk.Combobox(frame, values=["照明器具", "スイッチ", "コンセント", "配電盤", "その他"]) self.family_type.grid(row=0, column=1, padx=5, pady=5) ttk.Button(frame, text="ファミリを自動生成", command=self.create_revit_family).grid(row=1, column=0, columnspan=2, pady=10) self.result_text3 = tk.Text(frame, height=15, width=80) self.result_text3.grid(row=2, column=0, columnspan=2, padx=5, pady=5) scrollbar = ttk.Scrollbar(frame, command=self.result_text3.yview) scrollbar.grid(row=2, column=2, sticky='ns') self.result_text3.config(yscrollcommand=scrollbar.set) def create_tab4(self, tab): frame = ttk.LabelFrame(tab, text="積算データ紐付け") frame.pack(padx=10, pady=10, fill="both", expand=True) ttk.Label(frame, text="単価設定:").grid(row=0, column=0, padx=5, pady=5) self.unit_price = ttk.Entry(frame) self.unit_price.grid(row=0, column=1, padx=5, pady=5) ttk.Label(frame, text="労務費:").grid(row=1, column=0, padx=5, pady=5) self.labor_cost = ttk.Entry(frame) self.labor_cost.grid(row=1, column=1, padx=5, pady=5) ttk.Button(frame, text="積算データを紐付け", command=self.link_estimation_data).grid(row=2, column=0, columnspan=2, pady=10) self.result_text4 = tk.Text(frame, height=15, width=80) self.result_text4.grid(row=3, column=0, columnspan=2, padx=5, pady=5) scrollbar = ttk.Scrollbar(frame, command=self.result_text4.yview) scrollbar.grid(row=3, column=2, sticky='ns') self.result_text4.config(yscrollcommand=scrollbar.set) ttk.Button(frame, text="Revitで積算実行", command=self.run_estimation).grid(row=4, column=0, columnspan=2, pady=10) def browse_image(self): file_path = filedialog.askopenfilename( title="電材画像を選択", filetypes=(("画像ファイル", "*.jpg *.jpeg *.png"), ("すべてのファイル", "*.*")) ) if file_path: self.image_path.set(file_path) self.result_text1.insert(tk.END, f"選択された画像: {file_path}\n") def analyze_with_google_lens(self): if not self.image_path.get(): messagebox.showerror("エラー", "画像を選択してください") return if image_to_string is None: messagebox.showerror("エラー", "pytesseract がインストールされていません。") return try: self.result_text1.insert(tk.END, "画像からテキストを抽出中...\n") self.root.update() try: img = Image.open(self.image_path.get()) extracted_text = image_to_string(img, lang='jpn') self.result_text1.insert(tk.END, "抽出されたテキスト:\n") self.result_text1.insert(tk.END, extracted_text + "\n") # 製品名/型番を推測 product_info = self.extract_product_info(extracted_text) self.product_entry.delete(0, tk.END) self.product_entry.insert(0, product_info) self.result_text1.insert(tk.END, f"推測される製品情報: {product_info}\n") except FileNotFoundError: messagebox.showerror("エラー", "指定された画像ファイルが見つかりません。") except Exception as e: messagebox.showerror("エラー", f"画像処理中にエラーが発生しました: {str(e)}") except Exception as e: messagebox.showerror("エラー", f"画像解析中にエラーが発生しました: {str(e)}") def extract_product_info(self, text): # 簡単な製品情報抽出ロジック lines = text.split('\n') for line in lines: if '型番' in line or 'モデル' in line or '品番' in line: return line.strip() return lines[0].strip() if lines else "不明な製品" def scrape_material_data(self): product_query = self.product_entry.get() if not product_query: messagebox.showerror("エラー", "製品名/型番を入力してください") return try: self.result_text2.insert(tk.END, f"製品 '{product_query}' のデータ収集を開始...\n") self.root.update() # 主要電材メーカーのサイトを検索 manufacturers = ["panasonic", "toshiba", "mitsubishi", "hitachi", "fujitsu"] found = False for maker in manufacturers: url = f"https://www.{maker}.co.jp/search?q={product_query}" self.result_text2.insert(tk.END, f"{maker}サイトを検索中...\n") try: headers = {'User-Agent': 'Mozilla/5.0'} response = requests.get(url, headers=headers, timeout=10) response.raise_for_status() # HTTPエラーが発生した場合に例外を発生させる soup = BeautifulSoup(response.text, 'html.parser') # 製品ページへのリンクを探す（簡易版） product_link = None for a in soup.find_all('a', href=True): if 'product' in a['href'].lower() or 'catalog' in a['href'].lower(): product_link = a['href'] if a['href'].startswith('http') else f"https://www.{maker}.co.jp{a['href']}" break if product_link: self.result_text2.insert(tk.END, f"製品ページを発見: {product_link}\n") product_data = self.scrape_product_page(product_link) if product_data: self.material_data = product_data self.display_material_data(product_data) found = True break else: self.result_text2.insert(tk.END, f"{product_link} から製品データを取得できませんでした。\n") except requests.exceptions.RequestException as e: self.result_text2.insert(tk.END, f"{maker}サイトへの接続エラー: {str(e)}\n") except Exception as e: self.result_text2.insert(tk.END, f"{maker}サイト検索中に不明なエラー: {str(e)}\n") if not found: self.result_text2.insert(tk.END, "該当する製品情報が見つかりませんでした\n") except Exception as e: messagebox.showerror("エラー", f"データ収集中にエラーが発生しました: {str(e)}") def scrape_product_page(self, url): try: headers = {'User-Agent': 'Mozilla/5.0'} response = requests.get(url, headers=headers, timeout=10) response.raise_for_status() soup = BeautifulSoup(response.text, 'html.parser') # 簡易的なデータ抽出（実際にはサイトごとにカスタマイズが必要） product_data = { 'name': self.get_text_or_default(soup.find('h1'), '不明な製品'), 'model': self.get_text_or_default(soup.find('span', class_='model'), '不明な型番'), 'specifications': {}, 'dimensions': {}, 'image_url': self.get_first_image(soup) } # 仕様表からデータを抽出 (より具体的なセレクタが必要) spec_table = soup.find('table') # クラス名がないため、最初の table を取得 (注意が必要) if spec_table: for row in spec_table.find_all('tr'): cols = row.find_all('td') if len(cols) >= 2: key = cols[0].get_text(strip=True) value = cols[1].get_text(strip=True) if '寸法' in key or 'サイズ' in key: product_data['dimensions'][key] = value else: product_data['specifications'] ```