Skip to main content
replaced http://codereview.stackexchange.com/ with https://codereview.stackexchange.com/
Source Link

You can do negative checks first and return as suggested by JohanJohan. This will results in much readable and less number of lines in code

You can do negative checks first and return as suggested by Johan. This will results in much readable and less number of lines in code

You can do negative checks first and return as suggested by Johan. This will results in much readable and less number of lines in code

deleted 3 characters in body
Source Link
SiD
  • 597
  • 2
  • 8

You don't need these variables to be declared outside score.

You don't need these variables to be declared outside score.

You don't need these variables to be declared.

deleted 3 characters in body
Source Link
SiD
  • 597
  • 2
  • 8
  • Make use of foreach instead of for whenever possible
  • Use of switch instead of if-else ladder
  • Use variables to store and re-use. I have used currentRow for sheet.GetRow(i)
  • Make use of foreach instead of for whenever possible
  • Use of switch instead of if-else ladder
  • Use variables to store and re-use. I have used currentRow for sheet.GetRow(i)
  • Make use of foreach instead of for whenever possible
  • Use switch instead of if-else ladder
  • Use variables to store and re-use. I have used currentRow for sheet.GetRow(i)
added 90 characters in body
Source Link
SiD
  • 597
  • 2
  • 8
Loading
added 90 characters in body
Source Link
SiD
  • 597
  • 2
  • 8
Loading
added 269 characters in body
Source Link
SiD
  • 597
  • 2
  • 8
Loading
Source Link
SiD
  • 597
  • 2
  • 8
Loading