غير قادر على تعيين سعر الاشتراك إلى 19.99 دولارًا

[RGJ (Richard - Communiteq)]

• استخدام Discourse-Subscriptions
• محاولة إضافة خطة تسعير إلى منتج
• إدخال 19.99 في خانة السعر (هذا صعب بالفعل لأن الإدخال يتصرف بغرابة شديدة)
• محاولة الإرسال
• الحصول على الخطأ “Invalid integer 1998.9999999999998” (عدد صحيح غير صالح 1998.9999999999998)

لقد اختبرت هذا فقط على الإصدار المستقر 3.5.3 حتى الآن، لكن الكود هو نفسه في النسخة التجريبية (beta).

Code

  @computed("unit_amount")
  get amountDollars() {
    return parseFloat(this.get("unit_amount") / 100).toFixed(2);
  }

  set amountDollars(value) {
    const decimal = parseFloat(value) * 100;
    this.set("unit_amount", decimal);
  }

خبث أرقام الفاصلة العائمة في جافاسكريبت / IEEE-754:

-> parseFloat("19.99")*100
<- 1998.9999999999998

الحل المقترح: const decimal = Math.round(Number(value) * 100)

سؤالي الأكبر: لماذا لم يلاحظ أحد؟ ألم يقم أحد بتعيين السعر على 19.99 دولارًا من قبل؟

[pfaffman (Jay Pfaffman)]

@RGJ

Javascript / IEEE-754 float nastyness:

عرفت ذلك على الفور. شيء آخر تعلمته عن أجهزة الكمبيوتر منذ أكثر من 40 عامًا لم يتغير.

@RGJ

Suggested fix: const decimal = Math.round(Number(value) * 100)

أو استخدام أعداد صحيحة بفئة 1 سنت؟ أعتقد أن هذا ما تفعله Stripe، في الواقع. (لكن حلك يتطلب تغيير قدر أقل بكثير من التعليمات البرمجية، لذا أعتقد أنه يفوز.)

[RGJ (Richard - Communiteq)]

@pfaffman

أو استخدم أعدادًا صحيحة من السنت الواحد؟ أعتقد أن هذا ما يفعله Stripe في الواقع.

نعم، هذا ما يفعله، unit_amount هي سنتات و amountDollars هي قيمة إدخال “XX.XX”.
خطأ “العدد الصحيح غير الصالح” يأتي حتى من واجهة برمجة تطبيقات Stripe.

@pfaffman

عرفت على الفور. شيء آخر تعلمته عن أجهزة الكمبيوتر منذ أكثر من 40 عامًا لم يتغير.

أعتقد أننا كلينا كبيران بما يكفي لمعرفة خطأ FDIV

[sam (Sam Saffron)]

بصراحة، أعتقد أن تخزين هذا في عدد عشري (float) هو أمر مضلل، يجب أن يكون هذا العمود عددًا صحيحًا (int) ومخزنًا بالسنتات.

[RGJ (Richard - Communiteq)]

إنها كذلك، يتم استخدام parseFloat للتحويل من/إلى حقل الإدخال وهو “عملة بعلامتين عشريتين”. من الواضح أن خاصية الكائن unit_amount عديمة النوع، ومن هنا جاء اقتراحي باستخدام Math.round().

إنها ليست عمودًا من جانب الخادم، بل يتم تمريرها إلى واجهة برمجة تطبيقات Stripe.

[Bruce5051 (Bruce)]

بالنسبة للعملة، فإن النقطة العائمة BCD أفضل بشكل عام من النقطة العائمة الثنائية.

[zogstrip]

شكرًا @RGJ على التقرير - هذا يجب أن يصلحه

FIX: Round dollar-to-cents conversion in subscription plans (#37877)