Unexpected token ( in line 392
第一個function(b)那裡

移除掉後又會說Cannot read property 'click' of undefined
第二個for迴圈還會導致網頁整個當掉

調整了，先前說了，你的function(b) 要拆出來獨立，有問題你要自己處理，沒有環境沒辦法幫你修正，要連修正都不會的話，誰都幫不了你～

還是一樣Cannot read property 'click' of undefined.....看來apObj必須在click前宣告

我剛有試著自己寫一個獨立函式 現在只差動作跑不出來而已了

function clickEvent()
{
	$.getJSON("data/complex.json",
	function(b)
		{
			var request_json=b.requestlist;
			request_length=request_json.length;
			p=1;
			id=1;
			h=130;
			var coverObj = $("#cover-float");
			var firstfpath="cover/";
			var endfpath="Cover.jpg";
			request_html="";
			for(q=0;q<3;q++)
			{
				var apObj=$("#apDivp" + id);
				var pObj = $("#p" + id);
				var msgObj = $("#message-float" + id);
				var key=id-1;
				var coverfpath=encodeURIComponent(request_json[q].coverpath).replace(/[%A-F8]/g,"").substring(12,210);
				request_html+="<img id=p"+p+" height="+h+"px width="+h+"px src="+firstfpath+coverfpath+endfpath;
				request_html+="></img>";
				
				request_html+='<p id=message-float'+p+' style=display:none>歌手:'+request_json[q].artist+"<br>";
				request_html+="歌名:"+request_json[q].title+"<br>";
				request_html+='專輯:'+request_json[q].album+"<br>";
				request_html+="時間:"+request_json[q].length+"</p>";
				h=h-(10*p);
				p++;
				id++;
			}
			coverObj.html(request_html);
			
			apObj.click(function(){
			pObj.animate({key: pList[key]});
			msgObj.animate({opacity: msgList[key]});
			});
			
			var pList = [];
			pList[0] = {left: '10px', right: '110px', right: '10px'};
			pList[1] = {right: '230px', left: '10px', right: '10px'};
			pList[2] = {right: '230px', right: '110px', left: '10px'};
			var msgList = [];
			msgList[0] = [ 'show', 'hide', 'hide'];
			msgList[1] = [ 'hide', 'show', 'hide'];
			msgList[2] = [ 'hide', 'hide', 'show'];
			
				$(pObj).error(function() {
					coverObj.find(pStr).attr('src', '/img/no.png');
				});
		}
	)
}

是$.getJSON("data/complex.json" 這個不要跟處理綁在一起，id是動態的，怎麼會寫成id=1;，自己慢慢改吧，總之不要在function(b) 裡又寫一大串，不然你的code永遠難以維護～

所以你的apObj.click要放在哪才不會undefind又不會把網頁搞掛掉 還有至今我還沒看過他有一次動作過

還是把測試用檔案發給你好了這樣你也比較好debug

https://drive.google.com/file/d/1QYcFP0A19z68f1tGaIPFWa223xY2tXIM/view

說真的，我很懶得看你的code，因為實在太難幫你理清了，不過給你兩個建議：
1.ES6有模版，如果你一定要用ES5，你可以用類似下面的方式去替代。

'<img id="{0}" class="{1}">'.replace("{0}", "123").replace("{1}", "active")

2.動畫部分不要用jQ做，用css的keyframes去做，這樣你只要操作class就好。這樣第一更能精準的控制效果，第二更改效果時只要去改css就好。

如果讓你們看起來很亂那還真是抱歉阿 畢竟是自學的

自己也是自學，所以跟自學無關，是你沒有花時間在基礎上，一個code改上十遍，自然就會寫得好，程式要那麼好混，那還有什麼搞頭～

我還是姑且問一下好了是哪段讓你們覺得亂
什麼等號前加號前沒加空格除外
這段code我自己讀大致上都能了解其中函意 我寫code原則上會先以寫出功能出來為第一考量 之後才會想如何簡化 效能我也會考慮 像是要我選原生的for跟jq的each我就會選前者 因為each實在是跑太慢了

先前說過了，重覆的code或邏輯不要到處寫，這是最基本的，另外html跟js不要綁在一起寫，不然你的html調整，js就要跟著動，等於整個function都要重寫，不是寫的人看懂就好，要考慮之後的維護還有功能調整的問題，寫的爛之後的時候都在搞維護就飽了，更別說功能越加越多，最後會比重頭寫還麻煩～

我並沒有像你說的時常在維護
重複的code你是說request_html這個?
如果你有想到比request_html +=更好的也能告訴我啊
其實這個維護也沒像你所說的這麼難 因為html那都是套個id而已
id如果變了也只要在最後輸出結果的地方改就好
基本上現在主要會遇到的問題就是timeout沒了 不過上次問要如何改寫也沒人給個能參考的回覆 之前有個說要改的現在也音訊全無了

btw 現在首樓的code有經過簡化且功能大致上也有了 長這樣

function clickEvent()
{
	$.getJSON("data/complex.json",
		function(b)
		{
			var request_json=b.requestlist;
			request_length=request_json.length;
			p=1;
			h=130;
			checkn=0;
			request_html="";
			for(q=0;q<3;q++)
			{
				var firstpath="cover/";
				var coverpath=encodeURIComponent(request_json[q].coverpath).replace(/[%A-F8]/g,"").substring(12,210);
				var endpath="Cover.jpg";
				request_html+="<img id=p"+p+" height="+h+"px width="+h+"px src="+firstpath+coverpath+endpath;
				request_html+="></img>";
				
				request_html+='<p id=message-float'+p+' style=display:none>歌手:'+request_json[q].artist+"<br>";
				request_html+="歌名:"+request_json[q].title+"<br>";
				request_html+='專輯:'+request_json[q].album+"<br>";
				request_html+="時間:"+request_json[q].length+"</p>";
				h=h-(10*p);
				p++;
			}
			request_html+="";
			$("#cover-float").html(request_html);
			
			$("#apDivp1").click(function(){
				if(checkn!=1)
				{
					checkn=1;
					$("#p2,#p3").removeAttr('style');
					$("#p1").animate({left:'10px'},function(){$("#p2,#p3").removeAttr('style');});
					$("#message-float1").animate({opacity:'show'});
					$("#message-float2,#message-float3").animate({opacity:'hide'});
				}
			});
			$("#apDivp2").click(function(){
				if(checkn!=2)
				{
					checkn=2;
					$("#p1,#p3").removeAttr('style');
					$("#p2").animate({left:'10px'},function(){$("#p1,#p3").removeAttr('style');});
					$("#message-float1,#message-float3").animate({opacity:'hide'});
					$("#message-float2").animate({opacity:'show'});
				}
			});
			$("#apDivp3").click(function(){
				if (checkn!=3)
				{
					checkn=3;
					$("#p1,#p2").removeAttr('style');
					$("#p3").animate({left:'10px'},function(){$("#p1,#p2").removeAttr('style');});
					$("#message-float1,#message-float2").animate({opacity:'hide'});
					$("#message-float3").animate({opacity:'show'});
				}
			});
			$("#p1").error(function(){
			$("#cover-float").find("#p1").attr('src','/img/no.png');
			});
			$("#p2").error(function(){
			$("#cover-float").find("#p2").attr('src','/img/no.png');
			});
			$("#p3").error(function(){
			$("#cover-float").find("#p3").attr('src','/img/no.png');
			});
		}
	)
}

最後我要說一點:至少我用這code是有功能的而不是像你一樣沒功能 我說過了我都是以用出功能為優先 我不會讓沒功能的code佔據在我的點播系統上

有沒有功能自己到這試試最快:http://ac1011.hopto.org:4095/

已經把工具告訴你了...
https://pjchender.blogspot.com/2017/01/javascript-es6-template-literalstagged.html

http://www.w3school.com.cn/cssref/pr_keyframes.asp

這裡沒人欠你，不用這麼兇。

ES6現在不是還沒幾款瀏覽器能支援嗎
keyframe也是 這支援率
我沒辦法保證使用者一定是從IE以外的瀏覽器進來我的點播系統所以還是算了吧