Error switch quantity not an integer

[c++][switch] как юзать свитч для char *?

Коротко — никак. Только для фундаментальных вроде int/char/enum можно.

[c++][switch] как юзать свитч для char *?

switch(*a){

Ага, пробуем скомпилить:

#include <iostream>
using namespace std;

int main(){
const char *a = "b";


switch(*a){

    case "a":
        cout << "a = a" << endl; break;
    case "b":
        cout << "a = b" << endl; break;
    }
}

Получаем ошибки:
switch.cpp:10: error: case label does not reduce to an integer constant
switch.cpp:12: error: case label does not reduce to an integer constant

[c++][switch] как юзать свитч для char *?

А если строки нужно сравнивать, то можно по хешам свитч делать.

[c++][switch] как юзать свитч для char *?

> хешики сравнивай

Плохой совет. Хорошим может быть использование кодогенерации на темплейтах.

[c++][switch] как юзать свитч для char *?

>Хорошим может быть использование кодогенерации на темплейтах

вот и займись этим

jtootf ★★★★★

(06.11.09 23:32:34 MSK)

[c++][switch] как юзать свитч для char *?

Самое интересное, что в макроаасемблерах обычно больше можно было, и со сладким синтаксисисом.

madcore ★★★★★

(06.11.09 23:33:08 MSK)

[c++][switch] как юзать свитч для char *?

>> хешики сравнивай

>Плохой совет. Хорошим может быть использование кодогенерации на темплейтах.

Темплейты со стрингами не работают. Вообще, насколько я понимаю оптимально тут каким-то макаром интернить строки чтобы сравнивать их указатели.

Absurd ★★★

(06.11.09 23:33:19 MSK)

Re: [c++][switch] как юзать свитч для char *?

собственно компилятор тебе все говорит правильно

[c++][switch] как юзать свитч для char *?

Ладно, буду использовать нагромождения из конструкций if(), else if(). Проект не большой, просто учусь писать и использовать классы.

[c++][switch] как юзать свитч для char *?

Да нет, тебе просто паскаль понравился

madcore ★★★★★

(06.11.09 23:36:27 MSK)

[c++][switch] как юзать свитч для char *?

    void Npc::damage(int damage_type, int damage_sum){


    switch(damage_type){
        case 1:
        cout << "Урон SMG. Одно попадание забирает 5 очков. Попаданий:" << damage_sum  << endl;
        health -= (5 * damage_sum);
        break;

        case 2:
        cout << "Урон AR2. Одно попадание забирает 9 очков. Попаданий:" << damage_sum << endl;
        health -= (9 * damage_sum);
        break;

        default:
            cerr << "!!!!|       Оружие не известно! Ошибка" << endl << endl;
        break;
        }

}

Хотел как damage_type указывать не номера, а слова, например «ar2».

[c++][switch] как юзать свитч для char *?

В case ведь могут быть только константы.

[c++][switch] как юзать свитч для char *?

>Хотел как damage_type указывать не номера, а слова, например «ar2».

Enum для этого и сделан

[c++][switch] как юзать свитч для char *?

[c++][switch] как юзать свитч для char *?

Хватит быдлокодить. На самом деле в c++ switch для char не нужен.

madcore ★★★★★

(06.11.09 23:46:42 MSK)

[c++][switch] как юзать свитч для char *?

>это неправильное желание
Почему? ИМХО, так было бы легче юзать класс.

[c++][switch] как юзать свитч для char *?

switch не нужен. Юзай полиморфизм или ассоциативные контейнеры

yoghurt ★★★★★

(07.11.09 00:31:53 MSK)

[c++][switch] как юзать свитч для char *?

>> Хотел как damage_type указывать не номера, а слова, например «ar2».

>это неправильное желание

В Qt вроде бы по такому принципу сигналы диспатчатся.

Absurd ★★★

(07.11.09 00:35:12 MSK)

[c++][switch] как юзать свитч для char *?

Диспатч он вообще всегда так работает. Сравниением строк…

Xellos ★★★★★

(07.11.09 00:43:10 MSK)

[c++][switch] как юзать свитч для char *?

Потому что это сравнение строк. Которое требует дофига операций. Когда строки будут символов под сто, а условий будет под сотню, а вызовов в секунду будет…

Xellos ★★★★★

(07.11.09 00:44:31 MSK)

[c++][switch] как юзать свитч для char *?

Я не понял, а что, с одинарными кавычками не прокатит?

Re: [c++][switch] как юзать свитч для char *?

>Ладно, буду использовать нагромождения из конструкций if(), else if(). Проект не большой, просто учусь писать и использовать классы.

Не совсем ясна задача.

Если на входе слова из известного словаря и требуется очень быстрая обработка, то следует использовать любую библиотеку вычисления perfect hash.

Если требования по скорости более низкие или словарь заранее неизвестен, то можно использовать дерево поиска.

Если требований по скорости нет вообще — то можно использовать последовательный перебор строк. Например, можно хранить нужные слова в массиве вместе с указателеми на функцию обработки.

sign ★

(07.11.09 01:44:52 MSK)

[c++][switch] как юзать свитч для char *?

прочитал

тяжелый случай у 70%

нафиг так мучать С++

namezys ★★★★

(07.11.09 01:54:29 MSK)

[c++][switch] как юзать свитч для char *?

namezys ★★★★

(07.11.09 01:59:00 MSK)

[c++][switch] как юзать свитч для char *?

Вариант: взять дистанцию левенштейна, и свитч делать уже по ней

dannie

(07.11.09 03:47:32 MSK)

[c++][switch] как юзать свитч для char *?

Это development

svu ★★★★★

(07.11.09 04:19:08 MSK)

[c++][switch] как юзать свитч для char *?

По теме. В жабке такую фичу (свитч по строкам) обещают добавить в 1.7

svu ★★★★★

(07.11.09 04:20:25 MSK)

[c++][switch] как юзать свитч для char *?

>Как такая конструкция

мне вот просто интересно — а что ты, собственно, хотел этой конструкцией сказать? что switch — это такой goto? что в нём можно забыть написать break? или что-то третье?

jtootf ★★★★★

(07.11.09 04:33:52 MSK)

[c++][switch] как юзать свитч для char *?

вот и займись этим

Заколебали. Вот идея. Именно этот код не тестировал, но рабочая 100%:

#include <boost/mpl/vector.hpp>
#include <boost/mpl/begin.hpp
#include <boost/mpl/deref.hpp>

#define ADDCASE(Name, Label, Function) 
class Name
{
public:
String label_;
Name() : label_(Label){}
void execute() { Function(); return true; }
};

Потом в коде:

void function1()
{ printf("Hello from switch"); }

ADDCASE(CASEBLOCK1, "label1" , function1)

И так — сколько нужно кейсов.

Потом кладем кейсы в контейнер:

typedef boost::mpl::vector<CASEBLOCK1, CASEBLOCK2, CASEBLOCK3> CASES;

Сам свитч:

template <class Begin, class End>
struct SWITCH
{
static bool doswitch(const String& s)
{
  typedef typename boost::mpl::deref<Begin>::type deref;
  dereft ex;
  if( ex.label_ == s )
    return ex.execute();
  else
    return Iteratorhelper<typename boost::mpl::next<Begin>::type, End >::doswitch(s); 
}
};

template <class End>
struct SWITCH<End, End>
{
static  bool doswitch(const String& s)
{
  bool false;
}
};

template<typename Container>
static QString PERFORMSWITCH(const String& s)
{
return SWITCH<typename boost::mpl::begin<Container>::type, typename boost::mpl::end<Container>::type>::switch(s);
}

Как пользоваться:

PERFORMSWITCH<CASES>("blabla");

Теперь компилятор сгенерит нам статическую функцию PERFORMSWITCH<CASES>, которая будет то же самое, что много if/else,
но куда проще для расширения, доработки, создания новых кейсов.

[c++][switch] как юзать свитч для char *?

>которая будет то же самое

что и сравнение по хешикам, с той лишь разницей, что ты их считаешь статически (рукми объявляешь все CASEBLOCK’и). расширяемость, читаемость (как реализации так и использования), зависимости — всё на уровне плинтуса. вопорос дня: а в чём профит-то?

jtootf ★★★★★

(07.11.09 08:24:58 MSK)

[c++][switch] как юзать свитч для char *?

что и сравнение по хешикам

Нет, в данном случае будет вызываться оператор сравнения для строки. Можно и хэши сделать (если постараться, то и статические). В doswitch можно делать все, что хочешь.

Читаемость отличная. Профит в том, что компилятор напишет за тебя очень много кода (и сделает это оптимально), скорости, расширяемости. Сам свитч можно вынести в отдельный хидер. Тогда для написания новых свитчей тебе достаточно сделать

void function1() { printf("Hello from switch"); }
void function2() { printf("Hello from switch"); }

ADDCASE(CASEBLOCK1, "label1" , function1) 
ADDCASE(CASEBLOCK2, "label2" , function1) 
ADDCASE(CASEBLOCK3, "label3" , function2) 

void test( )
{
...
typedef boost::mpl::vector<CASEBLOCK1, CASEBLOCK2, CASEBLOCK3> CASES2;
String mystr("label3");
PERFORMSWITCH<CASES2>(mystr);
...
}

Отличная читаемость, отличная расширяемость. И самое главное — это только идея, можно очень красиво ее доработать.

[c++][switch] как юзать свитч для char *?

Это как? Так?

(defgeneric switch-test (switch-case value)
            (:method (x y) (equal x y))
            (:method ((switch-case (eql T)) value) T))

(defmacro switch (value &rest cases)
  (let ((value-var (gensym)))
    (labels ((process-switch (cases)
               (if (null cases)
                 nil
                 (destructuring-bind
                   ((current-case &body body) &rest cases) cases
                   `(if (switch-test ,current-case ,value-var)
                      (progn ,@body)
                      ,(process-switch cases))))))
      `(let ((,value-var ,value))
         ,(process-switch cases)))))

(use-package :cl-ppcre)

(defstruct regex (pattern ".*" :type string))

(set-dispatch-macro-character
  ## #" (lambda (s c n)
            (declare (ignore n))
            (unread-char c s)
            `(make-regex :pattern ,(read s t nil t))))

(defmethod switch-test ((case-of regex) (value string))
  (cl-ppcre:scan (regex-pattern case-of) value))

(defun main (s)
  (switch s
    (#"(?i)[A-Z]" (format t "String contains letters~%"))
    (#"[0-9]" (format t "String contains at least one digit and no letters~%"))
    (T (format t "The value is not a string"))))

[c++][switch] как юзать свитч для char *?

между вторым и третьим случаями туда неплохо бы «(#».*» (format t «String contains no letters or digits»))», но суть и так понятна, думаю

[c++][switch] как юзать свитч для char *?

если один символ, то

#include <iostream> 

using namespace std;

int main() 
{ 
    const char *a = "b";

    switch(*a) {                  /* или a[0] вместо *a */
      case 'a': 
        cout << "a = a" << endl; 
        break; 
      
      case 'b': 
        cout << "a = b" << endl; 
        break; 
    } 
}

ono ★

(07.11.09 09:48:04 MSK)

[c++][switch] как юзать свитч для char *?

> Отличная читаемость, отличная расширяемость.

Уййййоооооо…

kemm ★

(07.11.09 10:10:30 MSK)

[c++][switch] как юзать свитч для char *?

Оптимально построить finite state machine по данным строкам на этапе компиляции. Она будет небольшой, и поиск будет занимать O(N) (N — длина входной строки) или даже меньше.

[c++][switch] как юзать свитч для char *?

> Уййййоооооо…

Что не так?

Вдогонку: вместо
[code]
return Iteratorhelper<typename boost::mpl::next<Begin>::type, End >::doswitch(s);
[/code]
следует читать
[code]
return SWITCH<typename boost::mpl::next<Begin>::type, End >::doswitch(s);
[/code]

[c++][switch] как юзать свитч для char *?

> Оптимально построить finite state machine

Какие мы умные. Конечно можно, но делать ты это будешь приблизительно так же. Только добавится еще функция, возвращающая указатель на себя же.

[c++][switch] как юзать свитч для char *?

Как я понимаю, вариант в Tcl-е делает то же самое, ибо от правильных регулярных выражений производительность не пострадает.

[c++][switch] как юзать свитч для char *?

В лиспе можно сделать при желании примерно так:

(case (var)
  ("abc[de]+f" 'first-string)
  ("abc[fg]" 'second-string)
  (else 'third))

В С++ без препроцессора я бы не стал этого делать вообще (если только не очень жётские требования к производительности). В D можно сделать.

А говорить про твоё нагромождение капслока, что оно отлично читается — нельзя.

[c++][switch] как юзать свитч для char *?

Не путай интерпретацию с компиляцией, ладно?

[c++][switch] как юзать свитч для char *?

Капслок здесь необязателен, препроцессор для удобства. Это вообще демонстрация идеи. И неоптимальная, за счет создания временного объекта.

[c++][switch] как юзать свитч для char *?

Не смотрел реализацию тикля, но мой комментарий относился не конкретно к тиклю, и компиляции там ничего не мешает. Вообще реализация case-а по строкам с использованием FSM идеологически очень похожа на реализацию case-а с использованием таблицы переходов.

[c++][switch] как юзать свитч для char *?

В полку индусов прибыло. Зачем вы за этот switch зацепились. Нагородите всяких макросов, шаблонами все обложите, и тот кто будет разбирать ваш код после вас, будет яростно писать кипятком. Чем плох такой вариант?

inline bool _IsEqu(const char* s1,const char* s2) { return (strcmp(s1,s2)==0); };

void Process(const char* text)
{
	if(_IsEqu(text,"aaa"))
	{
		//do some ...
	}
	else if(_IsEqu(text,"bbb"))
	{
		//do some ...
	}
	else if(_IsEqu(text,"ccc"))
	{
		//do some ...
	}
	else
	{
		pAnIc();
	}
}

З.Ы. Без _IsEqu можно было обойтись, но мне кажется, что так легче читается, а назначение этой функции можно понять из названия.

[c++][switch] как юзать свитч для char *?

madcore ★★★★★

(07.11.09 15:24:47 MSK)