Full Change Items System By HLP

[Storxy*]

المشاركة الأصلية كتبت بواسطة HLP عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

انا عامل سيستم ده من زمان اول لما كونت لسه بتعلم بس معنديش وقت اني اعيد صياغة برسيدور
علي فكرة انا ممكن اعملك برسيدور في كام سطر بطريقة ممتازة اتفضل

كود PHP:

USE [SRO_VT_SHARDLOG]
GO
/****** Object:  StoredProcedure [dbo].[_NewChangeSystem]    Script Date: 21/04/2016 06:22:45 ص ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER procedure [dbo].[_NewChangeSystem]
  @Charid  int,
  @Operation tinyint,
  @ItemRefid int 
AS
 DECLARE @IID int
        ,@RefID int
        ,@InvID int
        ,@Type varchar(50) = NULL
        ,@ID int
        ,@ID1 int
        ,@NID int 
        ,@Gender varchar(8)
        ,@DG varchar(50)
        ,@ItemID int 
        , @Slot int 
         set @Slot = '6'
IF @ItemRefID = '42099'
set @Type ='Sword'
IF @ItemRefID = '42100'
set @Type ='blade'
IF @ItemRefID = '42101'
set @Type ='Spear'
IF @ItemRefID = '42102'
set @Type ='TBALDE'
IF @ItemRefID = '42103'
set @Type ='Bow'
IF @ItemRefID = '42107'
Set @Type = 'Dagger'
IF @ItemRefID = '42108'
Set @Type = 'Xbow'
IF @ItemRefID = '42109'
Set @Type = 'Axe'
IF @ItemRefID = '42110'
Set @Type = '1Hand'
IF @ItemRefID = '42111'
Set @Type = '2Hand'
IF @ItemRefID = '42112'
Set @Type = 'Tstaff'
IF @ItemRefID = '42113'
Set @Type = 'Staff'
IF @ItemRefID = '42114'
Set @Type = 'DarkStaff'
IF @ItemRefID = '42115'
Set @Type = 'harp'


            SELECT @ItemID=I.ID64 , @RefID = I.RefItemID 
            FROM SRO_VT_SHARD.dbo._Items I INNER JOIN SRO_Vt_SHARD.dbo._Inventory Iv
            ON I.ID64 = Iv.ItemID
            WHERE Iv.Slot = @Slot AND Iv.CharID = @CharID

            select @ID = ID ,@DG = DG , @Gender =  Gender 
            FROM SRO_VT_SHARDLOG.dbo._SystemChangeByHLP
            WITH(NOLOCK) where ID = @RefID

            Select @NID = ID 
            FROM SRO_VT_SHARDLOG.dbo._SystemChangeByHLP  
            WITH(NOLOCK) WHERE  
            TYPE = @Type
             and Gender = @Gender 
             AND DG = @DG



                                                                      
IF (@ItemRefID Between 42099 and  42103 ) and (@Operation = 41) 
Begin
 IF @RefID =@ID 
begin
 Update SRO_VT_SHARD.._Items Set RefItemID=@NID,CreaterName='HLP' Where ID64 = @ItemID
END
end 


شكرا علي رايك

انا زى مقولتلك انا مبقولكش كدة تقليل منك ولا حاجة انا عندى اعتراف بان كل واحد بيبقى بكرة احسن انا نفسى كنت كدة عادى وهفضل كدة يعنى مستوايا انهاردة عمره مهيكون نفس مستوايا كمان سنة مثلا وانك مثلا تنزل بروسيدر زى ده وانت عارف اد ايه هو فيه عك سعتها انت هتكون بتدور نفسك لان مثلا المبتداء فى ال SQL هيقولك عشان طلاما السيستم شغال لاكن الفاهم ياما يرد وينتقدك وممكن الطريقة متعجبكش يا اما هياخد فكرة عنك وحشة فمتدقش من انى انتقدتك ونصيحة اخيرة هو انا مقرتش البروديسر بالنفصيل بس بدل ما تشتغل ب IF @XXXX SET @XXXX=كذااا اشتغلها ب WHEN و THEN احسن تقبل مرورى وسواء خدت بالنصيحة او لا حاجة ترجعلك وتحترم على ردك وحاول تهتم بال Performance على قد متقدر لان مش صعب انك تعمل سيستم او كدة لحد فاهم قد م صعب انك تعمله ب Performance عالية

[HLP]

المشاركة الأصلية كتبت بواسطة Storxy* عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

انا زى مقولتلك انا مبقولكش كدة تقليل منك ولا حاجة انا عندى اعتراف بان كل واحد بيبقى بكرة احسن انا نفسى كنت كدة عادى وهفضل كدة يعنى مستوايا انهاردة عمره مهيكون نفس مستوايا كمان سنة مثلا وانك مثلا تنزل بروسيدر زى ده وانت عارف اد ايه هو فيه عك سعتها انت هتكون بتدور نفسك لان مثلا المبتداء فى ال SQL هيقولك عشان طلاما السيستم شغال لاكن الفاهم ياما يرد وينتقدك وممكن الطريقة متعجبكش يا اما هياخد فكرة عنك وحشة فمتدقش من انى انتقدتك ونصيحة اخيرة هو انا مقرتش البروديسر بالنفصيل بس بدل ما تشتغل ب IF @XXXX SET @XXXX=كذااا اشتغلها ب WHEN و THEN احسن تقبل مرورى وسواء خدت بالنصيحة او لا حاجة ترجعلك وتحترم على ردك وحاول تهتم بال Performance على قد متقدر لان مش صعب انك تعمل سيستم او كدة لحد فاهم قد م صعب انك تعمله ب Performance عالية

تمام بس WHEN و THEN نفس فكرة Type
وانا سغال When و then بس في TRIGGER

كود PHP:

Select @Type = Case 
WHEN @ItemRefID = '42099' Then 'Sword'
WHEN @ItemRefID = '42100' Then 'blade'
WHEN @ItemRefID = '42101' Then 'Spear'
WHEN @ItemRefID = '42102' Then 'TBALDE'
WHEN @ItemRefID = '42103' Then 'Bow'
WHEN @ItemRefID = '42107' Then 'Dagger'
WHEN @ItemRefID = '42108' Then 'Xbow' 


[Storxy*]

المشاركة الأصلية كتبت بواسطة HLP عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

تمام بس WHEN و THEN نفس فكرة Type
وانا سغال When و then بس في TRIGGER

كود PHP:

Select @Type = Case 
WHEN @ItemRefID = '42099' Then 'Sword'
WHEN @ItemRefID = '42100' Then 'blade'
WHEN @ItemRefID = '42101' Then 'Spear'
WHEN @ItemRefID = '42102' Then 'TBALDE'
WHEN @ItemRefID = '42103' Then 'Bow'
WHEN @ItemRefID = '42107' Then 'Dagger'
WHEN @ItemRefID = '42108' Then 'Xbow' 


معاك ان الاتنين هيعملو المطلوب بس ك MS و Performance هيبقو احسن اقولك انا زى ايه مثلا

كود PHP:

SELECT CharName16 FROM _Char WHERE  CharID='5' 


كود PHP:

SELECT Top 1 [CharName16] FROM _Char WHERE CharID='5' 


الاتنين الفوق هيقومو بنفس المهمة هتقلى استخدم اى واحدة فيهم هقولك لا استخدم التانية هتقولى ليه هقولك نفرض التابل فى 1000000 لاين لواستخدمت الاولى هيطر يعمل Check للينات كلها بعدين يديك الى لقاها لكن التانى بقى اول ميلقيها هيبطل تدوير ويديك الى لفيها لان اصلا انت عارف ان مفيش غير اسم واحد الى ال CharID بتاعه 5

الفكرة مش فى انك تكتب بروديسر الفكرة فى انك تبقى صايع عشان يطلع ب Performance عالية و MS قليل
زى فكرة مثلا انك تعمل 5000 INSERT لاين فى بروديسر وانك تعمل لاين واحد عبارة عن INSERT ... SELECT

[HLP]

المشاركة الأصلية كتبت بواسطة Storxy* عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

معاك ان الاتنين هيعملو المطلوب بس ك MS و Performance هيبقو احسن اقولك انا زى ايه مثلا

كود PHP:

SELECT CharName16 FROM _Char WHERE  CharID='5' 


كود PHP:

SELECT Top 1 [CharName16] FROM _Char WHERE CharID='5' 


الاتنين الفوق هيقومو بنفس المهمة هتقلى استخدم اى واحدة فيهم هقولك لا استخدم التانية هتقولى ليه هقولك نفرض التابل فى 1000000 لاين لواستخدمت الاولى هيطر يعمل Check للينات كلها بعدين يديك الى لقاها لكن التانى بقى اول ميلقيها هيبطل تدوير ويديك الى لفيها لان اصلا انت عارف ان مفيش غير اسم واحد الى ال CharID بتاعه 5

الفكرة مش فى انك تكتب بروديسر الفكرة فى انك تبقى صايع عشان يطلع ب Performance عالية و MS قليل
زى فكرة مثلا انك تعمل 5000 INSERT لاين فى بروديسر وانك تعمل لاين واحد عبارة عن INSERT ... SELECT

انا معك في اللي انت بتقوله كل حاجة
في حالة اسكرول wpn ينفع علشان بيشتغيل علي slot واحد
انما علشان سكرول set مينفعش احدد slot واحد بس علشان انا عامل ان سكرولة واحدة تغير set كامل لازم lines تبقي كتير ولو فيه طريقة تانية منك نستفيد

[Storxy*]

المشاركة الأصلية كتبت بواسطة HLP عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

انا معك في اللي انت بتقوله كل حاجة
في حالة اسكرول wpn ينفع علشان بيشتغيل علي slot واحد
انما علشان سكرول set لازم احدد slot واحد بس علشان انا عامل ان سكرولة واحدة تغير set كامل لازم lines تبقي كتير ولو فيه طريقة تانية منك نستفيد

لا لا انتى فهمتنى غلط انا معلقتش على حكاية انك تعملها ب ال SLOTS فى SET انا كنت بثبتلك ان ممكن حاجتين يعملو نفس المهمة لاكن من نفس الادء تعليقا على حكاية WHEN و THEN و IF

[HLP]

المشاركة الأصلية كتبت بواسطة Storxy* عفوا ,,, لايمكنك مشاهده الروابط لانك غير مسجل لدينا [ للتسجيل اضغط هنا ]

لا لا انتى فهمتنى غلط انا معلقتش على حكاية انك تعملها ب ال SLOTS فى SET انا كنت بثبتلك ان ممكن حاجتين يعملو نفس المهمة لاكن من نفس الادء تعليقا على حكاية WHEN و THEN و IF

اه انا فاهم دي شكرا علي النصيحة كلنا بنتعلم من الاخطاء