Duurrrr...

[geekchick]

I just spent 15 minutes trying to figure out why my code just wouldn't make a connection to the SMTP server.

my $confirmation = Net::SMTP->new(Host => $smtpserver,
                                  Hello => $domain,
                                  Timeout => 30,
                                  Debug => 1);
die "Couldn't connect to server" unless $smtp;

*facepalm* Yeah, a little bit braindead today I think.

[Edit: And hey, look! My domain registration expired. In my defense, I have never seen a single notification letter or email about it. All set for the next two years though.]

Comments

[marnanel.livejournal.com]

use strict is your friend :)

[geekchick]

And cut-and-paste is not. ;)

[quasigeostrophy.livejournal.com]

I hate it when that happens. :-)

[geekchick]

Me too. =)

("But it worked before!")

[gsh]

For those who aren't web programmers, can you tell me what's wrong with that? Is is that you set variable confirmation and test variable smtp?

[geekchick]

Got it in one.

[nminusone.livejournal.com]

That's what you get for bothering to check return codes. Just assume code always works!

[geekchick]

But code intermittently working is why I'm rewriting it to use Net::SMTP.

[nminusone.livejournal.com]

Code intermittently working = ship it! Anything more reliable has a negative impact on job security. ;)

[geekchick]

Heh. True. Having it not-work consistently is very bad for job security. =)

[thunderbunny3.livejournal.com]

Gods, I do things like that all the time. *hugs*

[geekchick]

Today, I count myself lucky to be able to spell SMTP. =)

[hopeforyou.livejournal.com]

I can't tell you how many times I've had something staring me right in the face, only to not actually SEE it.

My sympathies.

[geekchick]

I find it really hard to go back through my own code to find errors, especially if it's something I've been working on for a while. Normally it's even less egregious than this, maybe just a simple typo somewhere else, and those get really hard to find after staring at my editor for a couple of hours.

[stitchinthyme.livejournal.com]

That's why I usually write something like:

my $confirmation = Net::SMTP->new(blahblahblah) || die "couldn't connect to server";

Less readable, but harder to mess up. :-)

[geekchick]

Thing is, I normally do that too. =) I have no idea why I randomly decided to switch formats for that particular script.