Tôi đã làm việc trong khoảng một năm. Tôi chủ yếu làm việc trong giao diện GUI của chúng tôi, sử dụng các phương thức từ phụ trợ C, nhưng tôi thường không phải xử lý chúng ngoại trừ các giá trị trả về. GUI của chúng tôi được cấu trúc khá hợp lý, với những hạn chế của chúng tôi.
Tôi đã được giao nhiệm vụ thêm một chức năng vào phần dòng lệnh của chương trình. Hầu hết các chức năng này giống như 300 dòng dài và khó sử dụng. Tôi đang cố gắng thu thập các mẩu của chúng để có được thông tin báo động cụ thể và tôi gặp khó khăn trong việc tổ chức. Tôi biết rằng tôi đang làm cho thử nghiệm của mình phức tạp hơn bằng cách thực hiện nó trong một chức năng dài duy nhất.
Tôi có nên giữ mọi thứ trong một chức năng lớn theo kiểu của các chức năng hiện có hay tôi nên gói gọn các báo động trong các chức năng của chúng?
Tôi không chắc liệu nó có phù hợp để đi ngược lại các quy ước mã hóa hiện tại hay không, liệu tôi có nên cắn viên đạn và làm cho mã khó hiểu hơn một chút để tự viết.
Tóm lại, tôi đang so sánh
showAlarms(){
// tons of code
}
chống lại
showAlarms(){
alarm1();
alarm2();
return;
}
alarm1(){
...
printf(...);
return;
}
EDIT: Cảm ơn lời khuyên của mọi người, tôi quyết định rằng tôi sẽ thiết kế mã của mình, sau đó hỏi họ muốn gì, và nếu họ muốn tất cả trong một, tôi có thể cắt từ mã được bao thanh toán của mình và biến nó thành 1 chức năng lớn. Điều này sẽ cho phép tôi viết nó và kiểm tra nó dễ dàng hơn ngay cả khi họ muốn tất cả mã trong một định nghĩa duy nhất.
CẬP NHẬT: Họ cuối cùng đã hài lòng với mã được bao thanh toán và hơn một người đã cảm ơn tôi vì đã thiết lập tiền lệ này.