Рефакторинг кода с помощью встроенного JavaScript

Я пытаюсь реорганизовать этот w3schools для кода с переключаемыми вкладками, потому что все мы знаем, что использование встроенного JavaScript — очень плохая практика, поэтому я пытаюсь максимально их разделить, поэтому я выбрал tablinks и добавил прослушиватель событий. но я борюсь с названиями городов (взгляните на их код, и вы поймете, о чем я говорю)

любая помощь, пожалуйста, и спасибо заранее

HTML

<!-- Tab links -->
<div class = "tab">
  <button class = "tablinks" onclick = "openCity(event, 'London')">London</button>
  <button class = "tablinks" onclick = "openCity(event, 'Paris')">Paris</button>
  <button class = "tablinks" onclick = "openCity(event, 'Tokyo')">Tokyo</button>
</div>

<!-- Tab content -->
<div id = "London" class = "tabcontent">
  <h3>London</h3>
  <p>London is the capital city of England.</p>
</div>

<div id = "Paris" class = "tabcontent">
  <h3>Paris</h3>
  <p>Paris is the capital of France.</p> 
</div>

<div id = "Tokyo" class = "tabcontent">
  <h3>Tokyo</h3>
  <p>Tokyo is the capital of Japan.</p>
</div>

CSS

.tab {
  overflow: hidden;
  border: 1px solid #ccc;
  background-color: #f1f1f1;
}

/* Style the buttons that are used to open the tab content */
.tab button {
  background-color: inherit;
  float: left;
  border: none;
  outline: none;
  cursor: pointer;
  padding: 14px 16px;
  transition: 0.3s;
}

/* Change background color of buttons on hover */
.tab button:hover {
  background-color: #ddd;
}

/* Create an active/current tablink class */
.tab button.active {
  background-color: #ccc;
}

/* Style the tab content */
.tabcontent {
  display: none;
  padding: 6px 12px;
  border: 1px solid #ccc;
  border-top: none;
}

JavaScript

function openCity(evt, cityName) {
  // Declare all variables
  var i, tabcontent, tablinks;

  // Get all elements with class = "tabcontent" and hide them
  tabcontent = document.getElementsByClassName("tabcontent");
  for (i = 0; i < tabcontent.length; i++) {
    tabcontent[i].style.display = "none";
  }

  // Get all elements with class = "tablinks" and remove the class "active"
  tablinks = document.getElementsByClassName("tablinks");
  for (i = 0; i < tablinks.length; i++) {
    tablinks[i].className = tablinks[i].className.replace(" active", "");
  }

  // Show the current tab, and add an "active" class to the button that opened the tab
  document.getElementById(cityName).style.display = "block";
  evt.currentTarget.className += " active";
}