The Weird and The Wonderful

ricmil4227-Apr-09 11:32

ricmil42

27-Apr-09 11:32

Brady Kelly wrote:
Switch Statement from Hell

You were NOT exaggerating!

Spunky Coder27-Apr-09 19:41

Spunky Coder

27-Apr-09 19:41

Thank God! There are only 8 levels!!

"Don't worry if it doesn't work right. If everything did, you'd be out of a job." (Mosher's Law of Software Engineering)

Super Lloyd29-Apr-09 17:38

Super Lloyd

29-Apr-09 17:38

What about the other case statements?
I'm curious now!! Wink | ;)

A train station is where the train stops. A bus station is where the bus stops. On my desk, I have a work station....
_________________________________________________________
My programs never have bugs, they just develop random features.

Brady Kelly29-Apr-09 21:57

Brady Kelly

29-Apr-09 21:57

OK, that was the last one, here are the first two.. You should be able to infer the rest.

case "LEFT JOIN Extensions ON CallRecords.Ext_ID = Extensions.Ext_ID LEFT JOIN Level1 ON Extensions.LevUp_ID = Level1.Lev_ID LEFT JOIN CAT ON CallRecords.Cat_ID = CAT.Cat_ID ":
    strSQL = strSQL + "Sites.SiteName, Sites.ReportCentre, Level1.LevelCode, Level1.LevelName, ";
    strSQLGROUPandORDER = "GROUP BY Sites.SiteName, Sites.ReportCentre, Level1.LevelCode, Level1.LevelName ORDER BY Sites.SiteName, Sites.ReportCentre, Level1.LevelCode, Level1.LevelName;";
    break;

case "LEFT JOIN Extensions ON CallRecords.Ext_ID = Extensions.Ext_ID LEFT JOIN Level1 ON Extensions.LevUp_ID = Level1.Lev_ID LEFT JOIN Level2 ON Level1.LevUp_ID = Level2.Lev_ID LEFT JOIN CAT ON CallRecords.Cat_ID = CAT.Cat_ID ":
    strSQL = strSQL + "Sites.SiteName, Sites.ReportCentre, Level2.LevelCode, Level2.LevelName, ";
    strSQLGROUPandORDER = "GROUP BY Sites.SiteName, Sites.ReportCentre, Level2.LevelCode, Level2.LevelName ORDER BY Sites.SiteName, Sites.ReportCentre, Level2.LevelCode, Level2.LevelName;";
    break;

You really gotta try harder to keep up with everyone that's not on the short bus with you.
- John Simmons / outlaw programmer.

Super Lloyd29-Apr-09 22:47

Super Lloyd

29-Apr-09 22:47

I see, mmm...

Really the original programmer should have foreseen the problem and use Emit to dynamically generate/compile a method for an ever increasing number of case! Cool | :cool:

OriginalGriff30-Apr-09 1:19

OriginalGriff

30-Apr-09 1:19

Super Lloyd wrote:
A train station is where the train stops. A bus station is where the bus stops. On my desk, I have a work station....

I want to steal your signature!

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones

How to check less then / greater then

Super Lloyd30-Apr-09 1:39

Super Lloyd

30-Apr-09 1:39

Ho no... I already stole it from someone else, you can't steal it from me!!! Roll eyes | :rolleyes:

ParagPatel22-Apr-09 22:50

ParagPatel

22-Apr-09 22:50

if(guess == random)
{
    printf("\nExcellent! You guessed the number!\nWould you like to play again?(y/n)");
    scanf("%c",&play);
    if(play == 'n')
    {
         game = 0;
    }
}

else
{
    if(guess < random && guess != random)
    {
        printf("\nToo low, try again!");
    }

    if(guess > random && guess != random)
    {
        printf("\nToo high, try again!");
    }
}

Reference[^]

Parag Patel
Sr. Software Eng, Varaha Systems

StM0n22-Apr-09 23:05

StM0n

22-Apr-09 23:05

if(guess < random && guess != random)
{
    printf("\nToo low, try again!");
}

if(guess > random && guess != random)
{
    printf("\nToo high, try again!");
}

nice...

my guess would be: never trust a computer...

(yes|no|maybe)*

Jeroen De Dauw23-Apr-09 0:27

Jeroen De Dauw

23-Apr-09 0:27

It's just to easy to imagine this person is now bragging about his program and trying to impress others with his coding skills - lol.

My little forums: http://code.bn2vs.com

70 72 6F 67 72 61 6D 6D 69 6E 67 20 34 20 6C 69 66 65!

Michael Bookatz23-Apr-09 1:01

Michael Bookatz

23-Apr-09 1:01

was he irish by any chance... you got to be sure after all?

Nagy Vilmos23-Apr-09 1:38

Nagy Vilmos

23-Apr-09 1:38

if(guess == random)
{
    [...]
}
else
{
    if(guess < random &&
       guess != random)
    {
        [...]
    }
    else
    {
	if(guess > random &&
           guess != random &&
           !(guess < random &&
             guess != random))
        {
            [...]
        }
    }
}

Fixed it!!

Panic, Chaos, Destruction.
My work here is done.

Nemanja Trifunovic23-Apr-09 7:57

Nemanja Trifunovic

23-Apr-09 7:57

That's called Defense in Depth[^]

Programming Blog

utf8-cpp

Dan Neely23-Apr-09 8:17

Dan Neely

23-Apr-09 8:17

Interesting article, but it only has backward links. Is there a list that links to all 11 parts available somewhere?

It is a truth universally acknowledged that a zombie in possession of brains must be in want of more brains.
-- Pride and Prejudice and Zombies

SalarSoft26-Apr-09 1:45

SalarSoft

26-Apr-09 1:45

This one is better to be sure Unsure | :~

if(guess == random && !(guess != random))

www.softprojects.org

Simone Serponi27-Apr-09 3:54

Simone Serponi

27-Apr-09 3:54

if(guess == random && !(guess != random))

Can we call it an example of "psychotic" programming? Laugh | :laugh:

Bye By(t)e Wink | ;-)

perels21-Jul-09 13:23

perels

21-Jul-09 13:23

This is my five cents Wink | ;-)

if(guess == random || guess != random)

What would he do if it were floats?

Simone Serponi27-Apr-09 3:50

Simone Serponi

27-Apr-09 3:50

well... maybe the coder wished to cover ALL cases... Laugh | :laugh:

Bye By(t)e Wink | ;-)

mav.northwind21-Apr-09 20:56

mav.northwind

21-Apr-09 20:56

Found in an article approved not too long ago...
And no, I didn't throw out case 5.

public int Volume
{
  get { return this.volume; }
  set
  {
    if (value < 0) { this.volume = 0; }
    else if (value > 10) { this.volume = 10; }
    else { this.volume = value; }

    //  Set it on Windows Media Player

    //  Fix up

    switch ( this.volume )
    {
      case 0: this.volume = 10; break;
      case 1: this.volume = 9; break;
      case 2: this.volume = 8; break;
      case 3: this.volume = 7; break;
      case 4: this.volume = 6; break;
      case 6: this.volume = 4; break;
      case 7: this.volume = 3; break;
      case 8: this.volume = 2; break;
      case 9: this.volume = 1; break;
      case 10: this.volume = 0; break;
    }
    
    this.volume = this.volume * -100;

    this.player.Volume = this.volume;
    //System.Windows.Forms.MessageBox.Show(this.player.Volume.ToString());
  }
}

Regards,
mav

--
Black holes are the places where God divided by 0...

OriginalGriff21-Apr-09 21:44

OriginalGriff

21-Apr-09 21:44

mav.northwind wrote:
no, I didn't throw out case 5.

Why would you? Anyone can see it isn't necessary and the original author has optimised it out for efficiency reasons. Laugh | :laugh:

Megidolaon26-Apr-09 21:45

Megidolaon

26-Apr-09 21:45

How about:

this.volume = 10 - this.volume;

Or is that too verbose?

mav.northwind wrote:
this.volume = this.volume * -100;

So, first prevent setting the volume to any value lower than 0 and than set it way below 0 every time?

mav.northwind wrote:
this.player.Volume = this.volume;

Good job causing an endless recursion...

Dead | X|

OriginalGriff26-Apr-09 22:01

OriginalGriff

26-Apr-09 22:01

I just love the effect of

this.volume++;

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones