|
No idea why he was "spying" on a client.
We only found this out after he left and we took over the account.
private MailMessage BuildMailMessage(...)
{
...
mailMessage.BCC.Add("guys@address.com");
..
}
.
|
|
|
|
|
Some of us have done that at my job to temporarily monitor production emails to be sure everything is working as expected.
|
|
|
|
|
I can understand why a dev would want to monitor things to ensure that everything is working properly. I also understand why a customer would not like the idea of being spied upon.
Just because the code works, it doesn't mean that it is good code.
|
|
|
|
|
Potayto, potawto.
Another thing we do is retain any emails sent from our servers. I suppose that could be considered spying too.
|
|
|
|
|
AspDotNetDev wrote: Some of us have done that at my job to temporarily monitor production emails to
be sure everything is working as expected.
Yeah, that would be my initial reaction too. Just making sure the body/to/ccs/etc are working right. Of course, I'm coming from my world, where we have several automatically-generated emails. Although I would typically write this info to a log instead of copying myself/someone else.
A little disappointing that dev didn't configure the 'emailChecker' address so it could be easily de-configured.
|
|
|
|
|
Somewhere (the daily WTF?) I saw <a href="mailto:....bcc=guys@address.com.."> link. Seems to be not so uncommon.
|
|
|
|
|
Thinking the best, he wanted to be sure that the system was running perfectly, thinking the worst, he may be working for someone else... However, if the account was a corporate account (and not his personal email account), most likely he was monitoring the system.
|
|
|
|
|
I found this in one of my old codes (it was a program to check whether a square was a magic square[^] or not)
private void button1_Click(object sender, EventArgs e)
{
int lefttop = Convert.ToInt16(_1.Text);
int centertop = Convert.ToInt16(_2.Text);
int righttop = Convert.ToInt16(_3.Text);
int leftcenter = Convert.ToInt16(_4.Text);
int centercenter = Convert.ToInt16(_5.Text);
int rightcenter = Convert.ToInt16(_6.Text);
int leftbottom = Convert.ToInt16(_7.Text);
int centerbottom = Convert.ToInt16(_8.Text);
int rightbottom = Convert.ToInt16(_9.Text);
int row1 = lefttop + centertop + righttop;
int row2 = leftcenter + centercenter + rightcenter;
int row3 = leftbottom + centerbottom + rightbottom;
int col1 = lefttop + leftcenter + leftbottom;
int col2 = centertop + centercenter + centerbottom;
int col3 = righttop + rightcenter + rightbottom;
int dia1 = lefttop + centercenter + rightbottom;
int dia2 = leftbottom + centercenter + righttop;
if (row1 == row2)
{
if (row2 == row3)
{
if (row3 == col1)
{
if (col1 == col2)
{
if (col2 == col3)
{
if (col3 == dia1)
{
if (dia1 == dia2)
{
MessageBox.Show("Your magic square is correct!");
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
else
{
MessageBox.Show("Wrong! Application will restart!");
this.Close();
Process restart = new Process();
restart.StartInfo.FileName = @"path name here";
restart.Start();
}
}
|
|
|
|
|
lol
The first step in the acquisition of wisdom is silence, the second is listening, the third memory, the forth, practice and the fifth is teaching others!
|
|
|
|
|
Well...at least you indented it correctly?
But that's terrible! You still use Visual Studio default names for your controls?
The universe is composed of electrons, neutrons, protons and......morons. (ThePhantomUpvoter)
|
|
|
|
|
Ah... the glorious joy of "WTF"
(yes|no|maybe)*
|
|
|
|
|
Looks like you get payed by lines of code
|
|
|
|
|
I don´t believe in if anymore
|
|
|
|
|
Quote: I don´t believe in if anymore It's not "if", it's "when." You have to think positive.
There are only 10 types of people in the world, those who understand binary and those who don't.
|
|
|
|
|
You're right!
But, don`t just disregard this code, even Metallica says "nothing ELSE matters"
So, try coding ELSE without the IF.
|
|
|
|
|
How about SOMETHING ELSE?
"It's true that hard work never killed anyone. But I figure, why take the chance." - Ronald Reagan
That's what machines are for.
Got a problem?
Sleep on it.
|
|
|
|
|
Wonderfull! Even to the point of paraphrasing the old BASIC error message "?Redo from start" - and so that we really can see it, duplicate it several times.
Nice
|
|
|
|
|
At least it's easy to see what its doing if nothing else!
|
|
|
|
|
Aaaah!, it makes me remember some of my early programs...
|
|
|
|
|
OMG .. reminds me of some old code base I had to maintain some 4 yrs back ... Once we tried counting the cyclometric complexity of the code and got a headOverfolowException
On a more serious note there are many situations when coders care little about how others see it ... They just fall into a loop of get-it-done-yesterday (by one of the zillion managers who think they are God) and the result is just infront of us
|
|
|
|
|
A right/wrong flag will make the code simpler. But I think you should place all the values in 2 arrays and use memcmp (or similar functions)
|
|
|
|
|
There're lots of ways to optimise this code (without a right/wrong flags, there are more elegant solutions) but I don't think it's thread author's intention to get a code review...
How can we code so ugly when we're fresh eh?
It's so funny to review our old codes.
These days I found a code of mine written in C# and using a goto, WTF? *palmface*
|
|
|
|
|
Link to the article
For More detail explanation(PDF)
Counting arrays from 0 simplifies the computation of the memory address of each element.
If an array is stored at a given position in memory (it’s called the address) the position of each element can be computed as
element(n) = address + n * size_of_the_element
element(n) = address + (n-1) * size_of_the_element
modified 22-Apr-13 13:45pm.
|
|
|
|
|
kburman6 wrote: Not a huge difference but it adds an unnecessary subtraction for each access.
And because unnecessary calcs are slowing down the computers' speed it was just logical to do it as simple as possible. Developer's course at my school, lesson 1 (or two, however; It was at the very beginning when they explained it to us).
|
|
|
|
|
Marco Bertschi wrote: lesson 1 0
FTFY!
|
|
|
|