Click here to Skip to main content
15,893,266 members
Home / Discussions / C#
   

C#

 
GeneralRe: Web Service question (C#) Pin
leppie18-Sep-02 6:46
leppie18-Sep-02 6:46 
GeneralRe: Web Service question (C#) Pin
User 988518-Sep-02 6:54
User 988518-Sep-02 6:54 
GeneralRe: Web Service question (C#) Pin
leppie18-Sep-02 7:12
leppie18-Sep-02 7:12 
GeneralRe: Web Service question (C#) Pin
User 988518-Sep-02 6:56
User 988518-Sep-02 6:56 
GeneralRe: Web Service question (C#) Pin
leppie18-Sep-02 7:17
leppie18-Sep-02 7:17 
GeneralRe: Web Service question (C#) Pin
User 988518-Sep-02 7:39
User 988518-Sep-02 7:39 
GeneralRe: Web Service question (C#) Pin
User 988518-Sep-02 5:36
User 988518-Sep-02 5:36 
GeneralRe: Web Service question (C#) Pin
leppie18-Sep-02 6:29
leppie18-Sep-02 6:29 
GeneralRe: Web Service question (C#) Pin
Ray Cassick18-Sep-02 5:12
Ray Cassick18-Sep-02 5:12 
GeneralRe: Web Service question (C#) Pin
User 988518-Sep-02 5:27
User 988518-Sep-02 5:27 
QuestionIncrementing build numbers? Pin
Zinj18-Sep-02 4:29
sussZinj18-Sep-02 4:29 
AnswerRe: Incrementing build numbers? Pin
leppie18-Sep-02 5:09
leppie18-Sep-02 5:09 
GeneralDateTime.AddMinutes Pin
sharon18-Sep-02 2:11
sharon18-Sep-02 2:11 
GeneralRe: DateTime.AddMinutes Pin
Paul Riley18-Sep-02 2:15
Paul Riley18-Sep-02 2:15 
GeneralRe: DateTime.AddMinutes Pin
sharon18-Sep-02 2:17
sharon18-Sep-02 2:17 
GeneralActiveX and c# Pin
ORi x18-Sep-02 0:58
ORi x18-Sep-02 0:58 
GeneralRe: ActiveX and c# Pin
Stephane Rodriguez.18-Sep-02 2:21
Stephane Rodriguez.18-Sep-02 2:21 
GeneralRe: ActiveX and c# Pin
ORi x18-Sep-02 3:29
ORi x18-Sep-02 3:29 
GeneralRe: ActiveX and c# Pin
Stephane Rodriguez.18-Sep-02 4:03
Stephane Rodriguez.18-Sep-02 4:03 
GeneralBeep Pin
Mazdak18-Sep-02 0:02
Mazdak18-Sep-02 0:02 
GeneralRe: Beep Pin
Stephane Rodriguez.18-Sep-02 1:46
Stephane Rodriguez.18-Sep-02 1:46 
QuestionStatic Analysis tools? Pin
Martin D17-Sep-02 23:51
Martin D17-Sep-02 23:51 
AnswerRe: Static Analysis tools? Pin
Stephane Rodriguez.18-Sep-02 1:25
Stephane Rodriguez.18-Sep-02 1:25 
GeneralRe: Static Analysis tools? Pin
leppie18-Sep-02 4:15
leppie18-Sep-02 4:15 
GeneralRe: Static Analysis tools? Pin
leppie18-Sep-02 6:24
leppie18-Sep-02 6:24 
OK, so I got a @#$#load of errors again, though most of these "errors" are totally biased design "rules" (really only considerations).

EG all priority 1:

MemberNameKeywordCollision:
The member name 'Stop' conflicts in whole or part with reserved keyword 'Stop'. //yeah right, maybe with your code, but not with mine Laugh | :laugh:

EventHandlerNamesEndInEventHandler:
Rename 'BASSProgessHandler' to end in 'EventHandler'. //why make the name even longer, it wont effect the code.

EventFirstParameterIsNamedSender:
Change the first parameter of 'Progress' to an object type named 'sender'. // hell, i cast enough, why do i need to cast even more?

MemberOnlyShortAcronymsAreAllCaps:
Correct capitalization of member name 'get_ChannelEAXMix'. // again, who cares? EAX looks better to me than Eax (that looks more like some assembler function, maybe FXCop will complain about it too Laugh | :laugh:

ExceptionsRequireMultipleConstructors:
Add a constructor with a string parameter to custom exception 'BASSException'. Add a constructor with a string parameter and exception parameter to custom exception 'BASSException'. Add a protected constructor with a SerializationInfo object parameter and a StreamingContext object parameter to custom exception 'BASSException'. // again why? if you dont need it, why clutter your code with useless code?

TypeNameKeywordCollision:
The type name 'Error' conflicts in whole or part with reserved keyword 'Error'. // reserved????

MemberNamesDontUseUnderscores:
Remove the underscore from 'BASS_OK'. //if its not OK, why even make it legal to use?

ParameterNamesUseCamelCasing:
Instead of parameter name 'Length', use 'length'. // i cant help it if i hit my caps lock by mistake.

TypesDontExposePublicInstanceFields:
Make 'fWetDryMix' private or protected and provide a public property to access it. // Properties dont allways work when u need to "chain" a few, also the extra processing (repeated get/sets) makes it inefficient.

NamespaceNamesUsePascalCasing:
Instead of namespace 'nBASS', use 'NBASS'. //no, nBASS looks way cooler Smile | :)

and now for the 2 (yes only 2) usefull ones (priority 1):

AssembliesShouldBeStrongNamed:
Sign 'nBASS' with a strong name key. // not neccesary, but recommended.

MethodShouldKeepAliveIntPtrIfFinalized:
Add a GC.KeepAlive(this) to the method 'Void Stop()'. //i have no idea what they mean...does look like it could break my code though.

Any comments welcome Wink | ;)

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.