Is this a DRY code?

var playList = [
    'I Did It My Way',
    'Respect',
    'Imagine',
    'Born to Run',
    'Louie Louie',
    'Maybellene'
];

function printlist( list ) { 
    var listHTML = "<ol>";
    for ( var i = 0; i < playList.length; i++ ) { 
        listHTML = listHTML + "<li>" + list[i] + "</li>" ; 
    } 
listHTML = listHTML + "</ol>"; 
document.write(listHTML);
}

printlist(playList);

You are definitely not repeating yourself, hence the use of the loop. You did make a mistake by using playList.length in your loop instead of list.length. The way you have it you are hardcoding the length of the list to be printed to just playlist. If you passed in another list that had a different length it would break.

If you were to use this function to create more lists it wouldn't be possible because you are simply writing a list to the DOM. You would want to make it so that you can pass in an argument for the id of the <ol> that you want to add list items to. Otherwise you would be creating another function to do the same thing again, but with a different list. Of course the exact structure and use case for how and what you are creating a list for would really depend on how the function is written. Here is one example that would accessing an <ol> that is already part of the DOM.

var playList = [
    'I Did It My Way',
    'Respect',
    'Imagine',
    'Born to Run',
    'Louie Louie',
    'Maybellene'
];
var myNewList = document.getElementById("newList");

function printlist( list, listId ) {
    var listHTML = "";
    for ( var i = 0; i < list.length; i++ ) {
        listHTML += "<li>" + list[i] + "</li>" ; 
    } 
    listId.innerHTML = listHTML;
}

printlist(playList, myNewList);

Hi Jake,

DRY means Don't Repeat Yourself. This function on its own doesn't repeat anything unnecessarily in terms of lines of code, maintaining this over an entire application is a bit tougher.

If you want to get a little DRYier you can omit a couple of words by using the syntactic sugar of +=. It's very minor though, and DRY doesn't mean "remove as many characters as you can" so much as "don't create more places to change things than necessary".

Here's the += version (excuse my difference in formatting):

function printlist(list) 
{
        var listHTML = "<ol>";

        for (var i = 0; i < list.length; i++) 
        {
                listHTML +=  "<li>" + list[i] + "</li>";
        }

        listHTML += "</ol>";

        document.write(listHTML);
}