What a gem.

[Nagy Vilmos]

Julien Villers wrote:

BTW, shouldn't that be 'N/A'?

Na

---

Panic, Chaos, Destruction. My work here is done. Drink. Get drunk. Fall over - P O'H OK, I will win to day or my name isn't Ethel Crudacre! - DD Ethel Crudacre I cannot live by bread alone. Bacon and ketchup are needed as well. - Trollslayer Have a bit more patience with newbies. Of course some of them act dumb - they're often *students*, for heaven's sake - Terry Pratchett

[Lutoslaw]

This is a good example for an uncertainty principle. Such method of retreiving a "battery percent" may indeed change the BatteryPercent itself...

Greetings - Jacek

[0bx]

This is pure genius, if he gets paid by the line that is. If this method is ran while the battery is charging, is there a chance that it would return "NA" because the battery percentage changed at the correct moment? :)

Giraffes are not real.

[Fabio Franco]

Wow! This is only not worse than what I've seen, because the DB field length related to what I found is small:

public string GetStringToDatabase(int value)
{
if (value > 99999999)
return "0" + value.ToString();
else if (value > 9999999)
return "00" + value.ToString();
else if (value > 999999)
return "000" + value.ToString();
.
.
.
}

"To alcohol! The cause of, and solution to, all of life's problems" - Homer Simpson

[edmurphy99]

Maybe they got paid by the line

[BGArts]

Should be an philosopher, not a programmer. This is pure, unadulterated logic.

[Tomz_KV]

I am wondering if the statement continues to reach 0%. To return a result, the program will have to add a power source somewhere.

TOMZ_KV

[Lilith C]

Could have made a better case for switch/case statements. And if the device needed display "NA" it probably wouldn't have been running anyway.

I'm not a programmer but I play one at the office

[Lilith C]

Julien Villers wrote:

BTW, shouldn't that be 'N/A'?

Not when you have to scrape for every byte you can get.

I'm not a programmer but I play one at the office

[Paulo_JCG]

I like the ToString...

If SystemInformation.PowerStatus.BatteryLifePercent = 1.00f then ...

is such a PITA

Paulo Gomes Over and Out :D

[James Treworgy]

What a shortcut! A qualified programmer would have accounted for the fact that BatteryLifePercent is floating point, and may not be in increments of 1/100th. What if it's 0.975? Obviously this should have been written as:

If SystemInformation.PowerStatus.BatteryLifePercent <= 1 And
SystemInformation.PowerStatus.BatteryLifePercent > 0.99 then Return "99%"
ElseIf SystemInformation.PowerStatus.BatteryLifePercent <= 0.99 And
SystemInformation.PowerStatus.BatteryLifePercent > 0.98 then Return "98%"
...

:)

[Jasmine2501]

They are clearly just coding around a bad framework. There's a field called BatteryLifePercent, but it's not actually a percentage number. Obviously the framework is incorrect.

[Sander Rossel]

Option Strict is off or the Property would have an As clause! Correct would've been:

Public ReadOnly Property BatteryPercent As Percent
Get
If SystemInformation.PowerStatus.BatteryLifePercent.ToString = "1" Then
' No implicit conversions with Strict On!
Return Convert.ToPercentage("100%")
' Etc...

Now that just made a world of difference :)

It's an OO world.

public class Naerling : Lazy<Person>{
public void DoWork(){ throw new NotImplementedException(); }
}

[YSLGuru]

Could be wrong but it looks like a case of lack of knowledge, not knwoing/understanding that there is a better/more effecoent way to achieve teh same goal. I work in RDBMS (sepcifcally SQL Server) and I routinely see this same kind of bad design choice and usually its due to a lack of knowledge/eductaion on the product/system. For example when the develoepr needs to store a bolean value ( 0 and <>0 ) they use a NUMERIC data type which [by default) uses 9 bytes instead of using the smaller BIT data type.

[AspDotNetDev]

This is why degrees are useful. You may not be the best programmer in the world after graduating, but at least you should learn enough to avoid code like this. Having a get-r-done attitude is fine and all, but a get-r-done intelligently attitude is much better.

Somebody in an online forum wrote:

INTJs never really joke. They make a point. The joke is just a gift wrapper.

[weberrich]

Although, I know doesn't look all that good. There may be come consideration to optimization. Does the huge case statement cost less than a few function calls? Just a thought, and like to play devils advocate.

[Jorgen Andersson]

That's horrible. Everyone knows that you should use a varchar and store it as 'True' or 'False'.

Light moves faster than sound. That is why some people appear bright, until you hear them speak. List of common misconceptions

[firegryphon]

That is a joke right? Does this joke include enough to go all the way to 0%?

[Julien Villers]

You were joking right? I saw graduates that couldn't tell a variable from an array from a class from an object (etc...). Oh, and they weren't like barely getting their grades, in fact, they were top of the class (due to compensating programming with communication or economics, plus getting their projects done by others).

'As programmers go, I'm fairly social. Which still means I'm a borderline sociopath by normal standards.' Jeff Atwood 'I'm French! Why do you think I've got this outrrrrageous accent?' Monty Python and the Holy Grail

[Mark AJA]

Don't know what language this is, but I'm sure you could use the following to reduce the code length:

' ...
Return (SystemInformation.PowerStatus.BatteryLifePercent.ToString *100) & "%"
' . End of function.