That should do it

[Jaime]

I did a code review yesterday. The jist of the bug being fixed was that an application that polls a database and updates a screen with work to do occasionally gets a deadlock error and crashes.



Original code:

List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
  UpdateUI(newData);


New code after co-worker fixed it:

try {
  List<stuff> newData = GetDataFromDB();
  List<stuff> currentData = GetCurrentDataFromUI();
  if(!IsSameStuff(newData, currentData)
    UpdateUI(newData);
} catch (Exception ex) {}

Don't people even try any more? He took two weeks to submit this fix.

[Lorne Kates]

@Jaime said:

Don't people even try any more?

 

  Ummm....

 

[b]try[/b] {
  List<stuff> newData = GetDataFromDB();
  List<stuff> currentData = GetCurrentDataFromUI();
  if(!IsSameStuff(newData, currentData)
    UpdateUI(newData);
} catch (Exception ex) {}

 

What's your problem?

[those_who_survived_t]

try... lol

[TheCPUWizard]

@those who survived the plague said:

try... lol

"There is no try, there is only do" - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

[RichP]

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only do" - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

 

 

You win the Internets today!

 

 

[zelmak]

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only do" - Yoda

<FONT face="Courier New"><FONT size=5>Do</FONT> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</FONT>

<pedantic dickweed>The line is "Do... or do not. There is no try."</pedantic dickweed>

[dohpaz42]

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only doDo or do not, there is no try." - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

FTFY.

[Ibix]

@dohpaz42 said:

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only doDo or do not, there is no try." - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

F TFYF, I have.

Yoda'd that for you.

[db2]

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

[Someone_You_Know]

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

[Xyro]

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

[blakeyrat]

Is Star One named after the Federation space station in Blake's 7? Because... that would be awesome.

[Xyro]

@blakeyrat said:

Is Star One named after the Federation space station in Blake's 7? Because... that would be awesome.

Well of course it is, because Star One is awesome. PS, check out their live recording of Intergalactic Space Crusaders. It's one of my favorites ever.

[Power_Troll]

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only do" - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

Yoda is weak as shit. He got totally owned by the emperor and spent the rest of his life in a swamp while Luke and Han are killing themselves for the cause, dude won't even pick up a saber. Embarrassing as hell.

[derula]

@Someone You Know said:

There is no discord, there is harmony.

PTFY.

[blakeyrat]

@derula said:

@Someone You Know said:

Silence the discord!

PTFY.

SS2TFY

[DaveK1]

@Ibix said:

@dohpaz42 said:

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only doDo or do not, there is no try." - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData)
{ UpdateUI(newData); break;
} while (deadlock);</font>

F TFYF, I have.

Yoda'd tThat for you Yoda'd, I have.

TFYYIH.

[Watson1]

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

 

[Xyro]

@Watson said:

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

Black is white and white is black

The fact's a lie and the lie's a fact

You are the dead and the dead are alive

Two plus two equals five

[DaveK1]

@Xyro said:

@Watson said:

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

Black is white and white is black

The fact's a lie and the lie's a fact

You are the dead and the dead are alive

Two plus two equals five

JUMP IN THE URINAL AND STAND ON YOUR HEAD.

I'M THE ONE THAT'S ALIVE.  YOU'RE ALL DEAD.

 

[dhromed]

@DaveK said:

@Xyro said:

@Watson said:

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

Black is white and white is black

The fact's a lie and the lie's a fact

You are the dead and the dead are alive

Two plus two equals five

JUMP IN THE URINAL AND STAND ON YOUR HEAD.

I'M THE ONE THAT'S ALIVE.  YOU'RE ALL DEAD.

 

 

There is no spoon.

 

[immibis]

@Ibix said:

@dohpaz42 said:

@TheCPUWizard said:

@those who survived the plague said:

try... lol

"There is no try, there is only doDo or do not, there is no try." - Yoda

<font face="Courier New"><font size="5">Do</font> {
List<stuff> newData = GetDataFromDB();
List<stuff> currentData = GetCurrentDataFromUI();
if(!IsSameStuff(newData, currentData))
{ UpdateUI(newData); break;}
} while (deadlock);</font>

F TFYF, I have.

Yoda'd that for you.

Nobody even noticed the missing bracket and brace? You kids and your fancy compilers that check errors for you. Back in my day, if you ran a program with a missing character, your printer would catch fire and explode.

[derula]

@dhromed said:

@DaveK said:

@Xyro said:

@Watson said:

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

Black is white and white is black

The fact's a lie and the lie's a fact

You are the dead and the dead are alive

Two plus two equals five

JUMP IN THE URINAL AND STAND ON YOUR HEAD.

I'M THE ONE THAT'S ALIVE.  YOU'RE ALL DEAD.

 

There is no spoon.

Put that cookie down!

[Guest]

@derula said:

@dhromed said:

@DaveK said:

@Xyro said:

@Watson said:

@Xyro said:

@Someone You Know said:

@db2 said:

@TheCPUWizard said:

"There is no try, there is only do" - Yoda

There is no Dana. There is only Zuul.

 

There is no emotion, there is peace.
There is no ignorance, there is knowledge.
There is no passion, there is serenity.
There is no chaos, there is harmony.
There is no death, there is the Force.

Peace means war, rich means poor, and perfect, flawed.

Less means more, five means four, and hate, adore.

Love means law, true means false, and break, restore.

Dusk means dawn, here means gone, unbend, deform.

Die means born, admire means scorn, to know, ignore.

Won means lost, calm means storm, forget, recall.

Stand means fall, run means crawl, attack, withdraw.

Weak means strong, right means wrong, diverge, conform.

War is Peace

Freedom is Slavery

Ignorance is Strength

Black is white and white is black

The fact's a lie and the lie's a fact

You are the dead and the dead are alive

Two plus two equals five

JUMP IN THE URINAL AND STAND ON YOUR HEAD.

I'M THE ONE THAT'S ALIVE.  YOU'RE ALL DEAD.

 

There is no spoon.

Put that cookie down!

 

                               COOKIE!!

 

[boomzilla]

@El_Heffe said:

@derula said:

Put that cookie down!

                               COOKIE!!

Who Told You You Could Eat my Cookies

[blakeyrat]

@derula said:

Put that cookie down!

That is the worst movie to star Sinbad, and that's really saying something. He was in Good Burger.