上面代码没有执行，重新优化一下

如果您的代码没有执行，可能是因为有以下几个问题：

1. 页面加载时，JavaScript 代码没有被执行，可以尝试将代码放在 `$(document).ready(function(){})` 或 `$(function(){})` 中，以确保页面加载完毕后再执行代码。

2. 选择器指定的元素不存在，可以检查一下页面中是否存在指定的元素，或者将选择器改为其他有效的选择器。

3. 代码中存在语法错误，可以使用浏览器的开发者工具查看控制台输出，以找到错误的位置和原因。

以下是优化后的代码，您可以尝试使用这个代码：

$(function() {
  var scrollWrap = $("#scroll-wrap");
  scrollWrap.on("scroll", function() {
    var scrollTop = scrollWrap.scrollTop();
    var scrollHeight = scrollWrap.prop("scrollHeight");
    var clientHeight = scrollWrap.prop("clientHeight");
    if (scrollTop + clientHeight >= scrollHeight - 1) {
      console.log("已滑到底部");
    }
  });
});

在这个方案中，使用了 `$(function(){})` 来确保页面加载完毕后再执行代码，同时使用了 `scrollTop()` 和 `prop()` 方法来获取容器的滚动位置和高度，代码也进行了简化。