写了个demo,你可以参考一下：

https://jsfiddle.net/px7hbj8L/

试试 background-color .css("background-color","#9E9E9E");

再不然就是选择器有问题，确定 $("p[data-bg1]")这样能获得对应的对象，可以写 $("p[data-bg1]").hide()试试看是否会隐藏。

addClass方法中如果传入函数是需要这个函数返回字符串以代表类名的，removeClass同理

我不知道你这是什么用法

我建议，你定义一个颜色 比如

<style>
//橙色
.orange{
    background:#ff7e00;
}
//蓝色
.blue{
    background:#ff7e00;
}
//默认灰色
.default{
    background:#9E9E9E;
}
</style>
//支付选择方式
// 金额支付
$(".amount-payment").click(function(){//点击金额按钮
    $("p").removeClass("orange blue");//假设这是以上两条数据p
    $("p[data-bg1]").addClass("orange");//这是现金支付行
})
//积分支付
$(".exchange").click(function(){//点击积分按钮
    $("p").removeClass("orange blue");
    $("p[data-bg2]").addClass("blue");//这是兑换云卷行
})

大兄弟，你怎么知道你就一定是点的现金支付按钮？就一句$(".pay-tabs li").click(function(){

    $(this).addClass(function () {...}）；
    }）就能绑定到目标元素了？
    你知不知道$(".pay-tabs li")返回的是一个什么？数组！这意味着什么？要循环！怎么循环，jq的each函数自己去看看。就算你不循环，你也得$(".pay-tabs li").eq(0/1)吧？
    你知道你为什么没有效果？你点击一次它每次都执行removeClass(function () {
        //点击下一个按钮则删除之前"sel"
        $(this).removeClass('sel');
        点击下一个按钮，现金支付的框背景色变成灰色
        $("p[data-bg1]").css("background","#9E9E9E");
        点击下一个按钮，兑换云券的框背景色变成蓝色
        $("p[data-bg2]").css("background","#1863af");
    });，所以没有变化的。
    给你看下我的一个小例子（不过是用元生js写的，你自己换一下就Ok ,还比我的简单一些）https://jsfiddle.net/Jason_Ght92/63px88vt/1/

你这样切换不麻烦嘛？
直接更改class
点击谁就给谁加class 另外一个移除就可以了。