geekcomputers / geekcomputers/Python
Term with mixed case NOT Captured
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 35.4k
- Fork
- 12.9k
- Merge trung bình
- 2 giờ 37 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
If you have term = "Sin(20) + cOs(30)", term.replace(function, withmath) on LINE 50 above would not work except the whole term has been converted to lower case. Hence input validation is required.
Possible Solution:
Add term = term.lower() prior to the loop on LINE 47. Hence, this will remove the need for having .lower() on LINE 48's conditional statement.
@jwadel @Victory-Uchenna
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với calculator.py quanh các dòng 44-50, đặc biệt là vòng lặp xử lý các term và điều kiện thay thế. Chạy calculator với các tên hàm có cách viết hoa và thường lẫn nhau, chẳng hạn như Sin(20) + cOs(30), và xác nhận rằng đầu vào được chuẩn hóa trước khi thay thế và cho ra kết quả mong đợi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- cli
- Loại issue
- Lỗi
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 58/100