Блеать, у тебя там столько ошибок, что разбирать заипешься. Готовое
<div class="menu-container"><ul id="menu" class="menu-22">
<li><a href="#">Добавить </a></li>
<li><a href="#">Контакты</a></li>
<li><a href="#">Полезное</a></li>
</ul></div>
<button type="button" class="form-control" id = "b"> Город </button>
<script>
var ff = document.getElementById('menu');
ff.style.display = "block"
document.getElementById("b").onclick = function myFunction() {
if (ff.style.display === "none") { ff.style.display = "block"; }
else { ff.style.display = "none";}
}
</script>