So, this doesn’t work:

string username = "username";
string password = "password";
HttpWebRequest request = (HttpWebRequest)WebRequest.Create("url-here");
request.Credentials = new NetworkCredential(username, password);

To work around this you need to force the authentication to be sent. I found the details of how to do this in a blog post by Ian Dykes here:

http://devproj20.blogspot.com/2008/02/assigning-basic-authorization-http.html

But for fullness here’s the code:

string username = "username";
string password = "password";
byte[] authBytes = Encoding.UTF8.GetBytes(username + ":" + password.ToCharArray());
request.Headers["Authorization"] = "Basic " + Convert.ToBase64String(authBytes);

Permalink

| Leave a comment  »

Related posts:

1. How to make a cross domain web request with SilverLight 2
2. Kwwika-Superfeedr real-time demo available
3. Using Fiddler to trick Silverlight into allowing a crossdomain Web Request

Write a piece of code that prints all odd integer numbers between 1 and 99

This really isn’t a difficult question but it still requires some thought. When I’m posed with any question I like to break things down into their constituent parts.

Here’s the process I went through:

Okay, so I’ll define two variables for a start and end value and there’s going to have to be a loop.

int startValue = 1;
int endValue = 99;
for(int i = startValue;
     i <= endValue;
     i++)
{
   // work out if "i" is an odd number
}

Now, for the odd number detection. And… after a few umms and errrs … I’m going to have to mod 2 (%2) the current value of i to work out if the value is odd. More … umms and errs. Okay, I’ve finally worked out that if something mod 2 is not equal to 0 it’s clearly an odd number. This took me longer than it should have but never mind. Once I’ve detected if i is an odd number I’ll then put the odd number into a list for use later.

int startValue = 1;
int endValue = 99;
IList<int> oddValues = new List&lt;int&gt;();
for(int i = startValue;
     i &lt;= endValue;
     i++)
{
   if(i%2 != 0)
   {
      oddValues.Add(i);
   }
}

Those of you that are good at these little puzzles, or just think this is way too easy, might already be screaming at me about one of the following:

• Why are you using a IList, why don’t you just print the value?
• Odd numbers are always 2 apart so why aren’t you just increment i by 2 using i+=2?

I admit it, I missed the second point and that is a bit silly of me. However, what I have starting doing is separating the concerns of the piece of code. The code does two things; it detects the odd number and it prints the odd numbers. Those are two very distinct things. So, my code now looks like this:

int startValue = 1;
int endValue = 99;
IList<int> oddNumbers = new List&lt;int&gt;();
for(int i = startValue;
     i &lt;= endValue;
     i++)
{
   if(i%2 != 0)
   {
      oddNumbers.Add(i);
   }
}

string oddNumbersList = string.Join(",", oddNumbers.ToArray());
Console.WriteLine(oddNumbersList );

I’m now going to refactor this further so I’ll put the two different pieces of functionality into different methods. I’ll rename the i, startValue and endValue variables to be something a bit more useful; say numberToCheck, startNumber and endNumber. I’ll also create another helper for the odd number checking named IsOddNumber:

IList<int> GetOddNumbersBetween(int startNumber, int endNumber)
{
   IList oddValues = new List&lt;int&gt;();
   for(int numberToCheck = startNumber;
        numberToCheck &lt;= endNumber;
        numberToCheck++)
   {
      if(IsOddNumber(numberToCheck) == true)
      {
         oddValues.Add(numberToChecki);
      }
   }
   return oddValues;
}

bool IsOddNumber(int number)
{
   return (number % 2 == 1);
}

void PrintOddNumbersBetween(int startNumber, int endNumber)
{
   IList<int> oddNumbers = GetOddNumbersBetween(startNumber, endNumber);
   string oddNumbersList = string.Join(",", oddNumbers.ToArray());
   Console.WriteLine(oddNumbersList);
}

Let’s say I then notice the second point you’ve been screaming at me about (Odd numbers are always 2 apart so why aren’t you just increment i by 2 using i+=2) that I mentioned above? In practice I should notice this sort of thing either when I give the code a complete review, or one of my peers spots it. When I see this problem I decide to update the for loop, as noted, and I then see that I possibly don’t need the if(IsOddNumber(i) == true) statement. Although it would pain me to do this, since it’s a lovely little method, I would need to consider deleting it. But then it strikes me, I’m no longer just solving the “odd numbers between 1 and 99 problem” so I can’t just assume that the startNumber is going to be an odd number. I need to make sure that it’s an odd number so I’ll create another small utility method for that called EnsureOddNumber which will check if the value passed is an odd number, and if not return the next odd number (I’d like to rethink the name of this method).

IList<int> GetOddNumbersBetween(int startNumber, int endNumber)
{
   startNumber = EnsureOddNumber(startNumber);

   IList<int> oddValues = new List&lt;int&gt;();
   for(int numberToCheck = startNumber;
        numberToCheck &lt;= endNumber;
        numberToCheck+=2)
   {
      oddValues.Add(numberToCheck);
   }
   return oddValues;
}

int EnsureOddNumber(int number)
{
   if( IsOddNumber(startNumber) == false )
   {
      startNumber++;
   }
   return startNumber;
}

bool IsOddNumber(int number)
{
   return (number % 2 == 1);
}

void PrintOddNumbersBetween(int startNumber, int endNumber)
{
   IList<int> oddNumbers = GetOddNumbersBetween(startNumber, endNumber);
   string oddNumbersList = string.Join(",", oddNumbers.ToArray());
   Console.WriteLine(oddNumbersList);
}

Now, when I look at this code I get a warm feeling because I feel that it solves the problem, it’s well engineered, the concerns are separated and the code is completely self documenting.

There are a few comments that people may have here:

The question asked specifically to print odd values between 1 and 99 and you’ve done more than was required.

Although my answer does satisfy the original question have I over engineered things? The question does specifically ask us to print odd values between 1 and 99 so maybe I should have created a function that just satisfied that requirement.

void PrintOddNumbersBetween1And99()
{
   for(int i = 1;
        i <= 99;
        i+=2)
   {
      Console.WriteLine(i + " ");
   }
}

And I’d have to admit that this very short piece of code exactly answers the question. But I’d also argue that there is very little chance of this code being reused. Don’t get me wrong, you definitely shouldn’t over engineer things but there should be some scope for code reuse.

For such a simple problem you’ve over engineered this.

or

There’s a better solutions that that… it’s not efficient

Is creating four methods over engieering? Does my code require any comments to provide documentation? There may well be a more performant solution to this, but that’s not my point. My point is the way of approaching a question: the thought processes involved in understanding the problem and breaking it down to separate concerns, making it easy to read, reusable and self documenting. If the code that worked out the odd numbers is not efficient it could easily be changed in one place without impacting the interface, the other methods within the class, or the overall functionality.

Some people may jump to the simplest solution but I think the way i’ve described approaching and solving the problem demonstrates good practice. If I’m completely honest I would normally approach the development of something such as this by writing a test case first since I practice TDD but that can wait for another blog post.

Related posts:

1. Problem solving lessons relearnt
2. Basic Authentication against the Superfeedr HTTP PubSubHubbub API using a .NET HttpWebRequest

Here’s the code. It’s obviously part of a full solution containing a Web Application and a SilverLight Application project. I’ve also uploaded it.

Useful Information:

• You can not do a cross domain request over HTTPS.
• You can do a cross domain request over HTTP.
• To be able to make the cross domain request the server that you are making the request to needs to have either (requires clarification)  crossdomain.xml and/or clientaccesspolicy.xml in the root of the webserver.

Resources/links:

The code:

private void RequestButton_Click(object sender, RoutedEventArgs e)
{
    try
    {
        // Create and being making the request/getting the response
        HttpWebRequest request = (HttpWebRequest)WebRequest.Create(new Uri("http://www.leggetter.co.uk/"));
        request.BeginGetResponse(HandleResponse, request);
    }
    catch (Exception ex)
    {
        Debug.WriteLine(ex);
    }
}

private void HandleResponse(IAsyncResult result)
{
    try
    {
        // Finish getting the response and then read the response in chunks
        HttpWebRequest request = (HttpWebRequest)result.AsyncState;
        HttpWebResponse response = (HttpWebResponse)request.EndGetResponse(result);

        StringBuilder readText = new StringBuilder();
        using (Stream responseStream = response.GetResponseStream())
        {
            byte[] buffer = new byte[1024];
            int read = responseStream.Read(buffer, 0, buffer.Length);
            while (read > 0)
            {
                readText.Append(Encoding.UTF8.GetString(buffer, 0, read));
                read = responseStream.Read(buffer, 0, buffer.Length);
            }
        }

        // Display the response text
        Dispatcher.BeginInvoke(delegate()
        {
            ResponseText.Text = readText.ToString();
        });
    }
    catch (Exception ex)
    {
        Debug.WriteLine(ex);
    }
}

Related posts:

1. Using Fiddler to trick Silverlight into allowing a crossdomain Web Request
2. Using Fiddler to help develop cross domain capable JavaScript web applications
3. Making cross domain JavaScript requests using XMLHttpRequest or XDomainRequest

string tempPath =
    System.IO.Path.GetTempPath();

Related posts:

1. How to install Caplin Systems’ Liberator Free Edition on Windows
2. Windows Azure – 503 Service Unavailable
3. Response.Redirect in Windows Azure