Also gut. Auf jeden Fall stimmt die Logik ja.

Was wird mit $this.toggleClass('active'); getoggelt?

Dies hier
#accordion li.active { width: 450px; }

Die Klasse heisst ja aber .active, dewegen ja auch:

var totalActiveElements = $('.active').length;
UND
$('.active').each(function(e) {...}