Incidencia #18740

Check return codes everywhere

Abrir Fecha: 2009-09-17 01:58 Última actualización: 2009-10-06 22:29

Informador:
Propietario:
(Ninguno)
Estado:
Open
Componente:
(Ninguno)
Hito:
(Ninguno)
Prioridad:
3
Gravedad:
5 - Medium
Resolución:
Postponed
Fichero:
Ninguno
Vote
Score: 0
No votes
0.0% (0/0)
0.0% (0/0)

Details

Some checks for return codes are missing.

Examples:
Would you like to add more error handling for return values from "WriteFile" like in the function "CommentLogToFile" and from "GetModuleFileName" in the function "GetUILanguageFile"?

Ticket History (3/7 Histories)

2009-09-17 01:58 Updated by: elfring
  • New Ticket "Check return codes everywhere" created
2009-09-26 11:56 Updated by: (del#1144)
Comentario

How do you think a user hope which behavior?

(1) check the error and pop up MessageBox(modal), interrupt user operation.
(2) check the error and not notify to user.

2009-09-26 18:56 Updated by: elfring
Comentario

Situations might happen where users can not be notified about the unexpected "thing" by a message box.

Usual software design options are described in the article "Exception Handling Alternatives" by Detlef Vollmann.

I suggest to avoid unchecked function calls. Would you like to detect every error situation as early as possible?

2009-09-29 12:18 Updated by: (del#1144)
Comentario

I read your suggestion as "detect every error at every function call."

It is in the right. And it takes labor and time to check them.

2009-09-29 19:05 Updated by: elfring
Comentario

You do not need to make assumptions about error situations. Please check all error codes. Otherwise, it might not be noticed if anything unexpected went wrong.

Would you like to reduce the efforts for error code checking by an exception class hierarchy?

2009-10-06 22:28 Updated by: (del#1144)
  • Tipo Update from Bugs to Feature Requests
  • Prioridad Update from 5 - Medium to 3
2009-10-06 22:29 Updated by: (del#1144)
  • Resolución Update from Ninguno to Postponed

Attachment File List

No attachments

Editar

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Entrar